Merge pull request #435 from ofourdan/xdotool-use-xtest-instead-of-xw…

…arppointer Use XTEST instead of XWarpPointer with a single screen
jordansissel · Oct 16, 2024 · e7a6f15 · e7a6f15
2 parents 5689dc0 + 8478e38
commit e7a6f15
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/xdo.c b/xdo.c
@@ -804,8 +804,12 @@ int xdo_move_mouse(const xdo_t *xdo, int x, int y, int screen)  {
    * seem to recommend XWarpPointer instead, ie;
    * https://bugzilla.redhat.com/show_bug.cgi?id=518803
    */
-  Window screen_root = RootWindow(xdo->xdpy, screen);
-  ret = XWarpPointer(xdo->xdpy, None, screen_root, 0, 0, 0, 0, x, y);
+  if (screen > 0) {
+    Window screen_root = RootWindow(xdo->xdpy, screen);
+    ret = XWarpPointer(xdo->xdpy, None, screen_root, 0, 0, 0, 0, x, y);
+  } else {
+    ret = XTestFakeMotionEvent(xdo->xdpy, 0, x, y, CurrentTime);
+  }
   XFlush(xdo->xdpy);
   return _is_success("XWarpPointer", ret == 0, xdo);
 }