Don't arrange, but only limit the position on screen dimension changes.

bbidulock · May 30, 2024 · b74955d · b74955d
1 parent 313f4e3
commit b74955d
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/src/wmframe.cc b/src/wmframe.cc
@@ -2106,7 +2106,7 @@ void YFrameWindow::maybeFocus() {
     }
 }
 
-void YFrameWindow::focus(bool canWarp) {
+void YFrameWindow::limitPos() {
     if (limitPosition &&
         (x() >= int(desktop->width()) - borderX() ||
          y() >= int(desktop->height()) - borderY() - titleY() ||
@@ -2125,6 +2125,10 @@ void YFrameWindow::focus(bool canWarp) {
             newY = int(- borderY());
         setCurrentPositionOuter(newX, newY);
     }
+}
+
+void YFrameWindow::focus(bool canWarp) {
+    limitPos();
     manager->setFocus(this, canWarp);
     if (raiseOnFocus && manager->isRunning()) {
         wmRaise();

diff --git a/src/wmframe.h b/src/wmframe.h
@@ -54,6 +54,7 @@ class YFrameWindow:
     void createPointerWindows();
     void grabKeys();
 
+    void limitPos();
     void maybeFocus();
     void focus(bool canWarp = false);
     void activate(bool canWarp = false, bool curWork = true);

diff --git a/src/wmmgr.cc b/src/wmmgr.cc
@@ -3877,7 +3877,11 @@ void YWindowManager::updateScreenSize(XEvent *event) {
 
         /// TODO #warning "make something better"
         if (arrangeWindowsOnScreenSizeChange && resize) {
-            wmActionListener->actionPerformed(actionArrange, 0);
+            for (YFrameIter frame(focusedIterator()); ++frame; ) {
+                if (frame->visible()) {
+                    frame->limitPos();
+                }
+            }
         }
         if (resize) {
             arrangeIcons();