break screen: x11 keyboard locking: move threading

concentrate all the threading logic within the one method. everything
else runs on the main thread

Author: deltragon

safeeyes/ui/break_screen.py

@@ -143,7 +143,7 @@ def close(self) -> None:
             self.__release_keyboard_x11()
 
         # Destroy other windows if exists
-        GLib.idle_add(lambda: self.__destroy_all_screens())
+        self.__destroy_all_screens()
 
     def __show_break_screen(
         self,
@@ -298,18 +298,21 @@ def __lock_keyboard_x11(self) -> None:
                         event.detail == self.keycode_shortcut_skip
                         and self.show_skip_button
                     ):
-                        self.skip_break()
+                        utility.execute_main_thread(lambda: self.skip_break())
                         break
                     elif (
                         event.detail == self.keycode_shortcut_postpone
                         and self.show_postpone_button
                     ):
-                        self.postpone_break()
+                        utility.execute_main_thread(lambda: self.postpone_break())
                         break
             else:
                 # Reduce the CPU usage by sleeping for a second
                 time.sleep(1)
 
+        self.x11_display.ungrab_keyboard(X.CurrentTime)
+        self.x11_display.flush()
+
     def on_key_pressed_wayland(
         self, event_controller_key, keyval, keycode, state
     ) -> bool:
@@ -325,13 +328,8 @@ def on_key_pressed_wayland(
 
     def __release_keyboard_x11(self) -> None:
         """Release the locked keyboard."""
-        if self.x11_display is None:
-            return
-
         logging.info("Unlock the keyboard")
         self.lock_keyboard = False
-        self.x11_display.ungrab_keyboard(X.CurrentTime)
-        self.x11_display.flush()
 
     def __destroy_all_screens(self) -> None:
         """Close all the break screens."""