fix(gui): delay window movement to make it more robust

dynobo · Jan 19, 2024 · a7c0f22 · a7c0f22
1 parent 669a51b
commit a7c0f22
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/normcap/gui/dbus.py b/normcap/gui/dbus.py
@@ -193,6 +193,7 @@ def move_windows_via_window_calls_extension(title_id: str, position: Rect) -> bo
                 window_title = response[0]
                 if window_title == title_id:
                     window_id = window["id"]
+                    break
 
             response = proxy.move_resize(
                 window_id,

diff --git a/normcap/gui/window.py b/normcap/gui/window.py
@@ -159,11 +159,13 @@ def set_fullscreen(self) -> None:
 
         self.showFullScreen()
 
-        if system_info.display_manager_is_wayland():
-            self._move_to_position_on_wayland()
-
         self.setFocus()
 
+        if system_info.display_manager_is_wayland():
+            # Movement is delayed to ensure the window is fully active and
+            # registered within the window manager
+            QtCore.QTimer.singleShot(20, lambda: self._move_to_position_on_wayland())
+
     def clear_selection(self) -> None:
         self.selection_rect = QtCore.QRect()
         self.ui_container.rect = self.selection_rect