[orca/gnome-3-4] Don't let focused objects in gnome-shell dialogs preempt presentation of the dialog itself.



commit 0164f596d212d85e423a7b7b16aa221067c01a77
Author: Joanmarie Diggs <jdiggs igalia com>
Date:   Sun Apr 15 11:23:11 2012 -0400

    Don't let focused objects in gnome-shell dialogs preempt presentation of the dialog itself.

 src/orca/scripts/toolkits/CALLY/script.py |   27 +++++++++++++++++++++++++++
 1 files changed, 27 insertions(+), 0 deletions(-)
---
diff --git a/src/orca/scripts/toolkits/CALLY/script.py b/src/orca/scripts/toolkits/CALLY/script.py
index fc27b7f..49fd448 100644
--- a/src/orca/scripts/toolkits/CALLY/script.py
+++ b/src/orca/scripts/toolkits/CALLY/script.py
@@ -180,6 +180,33 @@ class Script(default.Script):
 
         return default.Script.checkKeyboardEventData(self, keyboardEvent)
 
+    def skipObjectEvent(self, event):
+        """Gives us, and scripts, the ability to decide an event isn't
+        worth taking the time to process under the current circumstances.
+
+        Arguments:
+        - event: the Event
+
+        Returns True if we shouldn't bother processing this object event.
+        """
+
+        try:
+            role = event.source.getRole()
+        except:
+            return default.Script.skipObjectEvent(self, event)
+
+        # Currently dialogs appearing do not result in window:activate events;
+        # instead, they claim focus. But often after that, a button within that
+        # dialog does the same thing before we have presented the dialog. Since
+        # we normally do not want to present things which are no longer focused,
+        # we normally skip the old event from the dialog (and never present it).
+        # That is bad.
+        if role == pyatspi.ROLE_DIALOG and event.detail1 \
+           and event.type.startswith('object:state-changed:focused'):
+            return False
+
+        return default.Script.skipObjectEvent(self, event)
+
     # NOTE: right now this is being redefined just for GNOME Shell, so
     # the proper place would be a GNOME Shell script. Anyway, as right
     # now GNOME Shell is the only Clutter application we care, for the



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]