commit 62993d9391469e87c3c69592d436122d73db8b07
author Dominique Martinet <asmadeus@codewreck.org> Fri Oct 26 13:22:10 2018 +0900
committer linkmauve <linkmauve@linkmauve.fr> Sat Nov 03 14:17:39 2018 +0100
tree e30006d6b023ba66a3e12ac2fc73de57c5e939ce
parent da8f3bc174a38198529792defb5d3b5faf3719f9

x11 window: update cursor position on enter event

click events would have an incorrect position after changing workspace,
if the mouse didn't move in between.
(Another example where this matters is a new window, if it appears under
the cursor, clicking would lead the application to think the user clicked
at 0,0)

src/x11_window.c

diff --git a/src/x11_window.c b/src/x11_window.c
index 0ae3b4c..5e91610 100644
--- a/src/x11_window.c
+++ b/src/x11_window.c
@@ -1455,12 +1455,20 @@
 
         case EnterNotify:
         {
+            // XEnterWindowEvent is XCrossingEvent
+            const int x = event->xcrossing.x;
+            const int y = event->xcrossing.y;
+
             // HACK: This is a workaround for WMs (KWM, Fluxbox) that otherwise
             //       ignore the defined cursor for hidden cursor mode
             if (window->cursorMode == GLFW_CURSOR_HIDDEN)
                 updateCursorImage(window);
 
             _glfwInputCursorEnter(window, GLFW_TRUE);
+            _glfwInputCursorPos(window, x, y);
+
+            window->x11.lastCursorPosX = x;
+            window->x11.lastCursorPosY = y;
             return;
         }