all repos — fluxbox @ 25a9a3158530075b606dc9ba28b7a30d64d624e2

custom fork of the fluxbox windowmanager 

fix _NET_WM_MOVERESIZE to interpret x_root and y_root correctly
markt markt
commit

25a9a3158530075b606dc9ba28b7a30d64d624e2

parent

f9ade8947c08a7f1a4485b44da2403a55a3c04e1

2 files changed, 9 insertions(+), 3 deletions(-)

jump to
M ChangeLog → ChangeLog 
@@ -1,6 +1,8 @@ 
  (Format: Year/Month/Day)
 Changes for 1.0rc3:
 *06/07/20:
+   * Fix _NET_WM_MOVERESIZE, bug #1525542 (Mark)
+     Ewmh.cc
    * Properly revert focus when two windows close simultaneously (Mark)
      WinClient.cc/hh FbTk/FbWindow.hh fluxbox.cc Window.cc
 *06/07/19:
M src/Ewmh.cc → src/Ewmh.cc 
@@ -891,9 +891,13 @@ case _NET_WM_MOVERESIZE_SIZE_BOTTOM:
         case _NET_WM_MOVERESIZE_SIZE_BOTTOMLEFT:
         case _NET_WM_MOVERESIZE_SIZE_LEFT:
         case _NET_WM_MOVERESIZE_SIZE_KEYBOARD:
-            winclient->fbwindow()->startResizing(ce.data.l[0], ce.data.l[1],
-                                                 static_cast<FluxboxWindow::ResizeDirection>
-                                                 (ce.data.l[2]));
+            // startResizing uses relative coordinates
+            winclient->fbwindow()->startResizing(ce.data.l[0] -
+                winclient->fbwindow()->x() -
+                winclient->fbwindow()->frame().window().borderWidth(),
+                ce.data.l[1] - winclient->fbwindow()->y() -
+                winclient->fbwindow()->frame().window().borderWidth(),
+                static_cast<FluxboxWindow::ResizeDirection>(ce.data.l[2]));
             break;
         case _NET_WM_MOVERESIZE_MOVE:
         case _NET_WM_MOVERESIZE_MOVE_KEYBOARD: