Ticket #4601: disactivate_function_keys.patch

src/jarabe/controlpanel/gui.py

@@ -365 +365 @@
         self._update(query)
 
     def __stop_clicked_cb(self, widget):
+        # Now, since we are exiting the "modal" "My Settings",
+        # re-enable the Function-key-handlers.
+        from jarabe.view.keyhandler import set_key_handlers_active
+        set_key_handlers_active(True)
+
         self.destroy()
 
     def __close_request_cb(self, widget, event=None):

src/jarabe/view/buddymenu.py

@@ -180 +180 @@
         frame = jarabe.frame.get_view()
         frame.hide()
 
+        # Now, since we are entering the "modal" "My Settings",
+        # disable the Function-key-handlers.
+        from jarabe.view.keyhandler import set_key_handlers_active
+        set_key_handlers_active(False)
+
         # show the control panel
         panel = ControlPanel()
         panel.show()

src/jarabe/view/keyhandler.py

@@ -74 +74 @@
         self._key_pressed = None
         self._keycode_pressed = 0
         self._keystate_pressed = 0
+        self._key_handlers_active = True
 
         self._key_grabber = SugarExt.KeyGrabber()
         self._key_grabber.connect('key-pressed',
@@ -163 +164 @@
         journalactivity.get_journal().show_journal()
 
     def _key_pressed_cb(self, grabber, keycode, state, event_time):
+        if not self._key_handlers_active:
+            return
+
         key = grabber.get_key(keycode, state)
         logging.debug('_key_pressed_cb: %i %i %s', keycode, state, key)
         if key is not None:
@@ -197 +201 @@
         return False
 
     def _key_released_cb(self, grabber, keycode, state, event_time):
+        if not self._key_handlers_active:
+            return
+
         logging.debug('_key_released_cb: %i %i', keycode, state)
         if self._tabbing_handler.is_tabbing():
             # We stop tabbing and switch to the new window as soon as the
@@ -215 +222 @@
         del _instance
 
     _instance = KeyHandler(frame)
+
+
+def set_key_handlers_active(active):
+    """
+    The setup(frame) is already run at sugar-session startup.
+    So, we can safely assume the "_instance" is fully-grown up.
+    """
+
+    _instance._key_handlers_active = active