[gnome-shell/gnome-3-10] messageTray: Only attempt to grab the summary box pointer after showing it



commit 6eb6887a2facb1b79657d41bc11eddc050a09822
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Tue Dec 3 19:21:08 2013 -0500

    messageTray: Only attempt to grab the summary box pointer after showing it
    
    For mysterious reasons I'm not sure of myself, navigate_focus will only focus
    mapped actors. So, make sure the widget is showing before navigating to it.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=709853

 js/ui/messageTray.js |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 7a8403c..d290a3d 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -2847,13 +2847,13 @@ const MessageTray = new Lang.Class({
                                                                                   Lang.bind(this, 
this._onSourceDoneDisplayingContent));
 
         this._summaryBoxPointer.bin.child = child;
-        this._grabHelper.grab({ actor: this._summaryBoxPointer.bin.child,
-                                onUngrab: Lang.bind(this, this._onSummaryBoxPointerUngrabbed) });
-
         this._summaryBoxPointer.actor.opacity = 0;
         this._summaryBoxPointer.actor.show();
         this._adjustSummaryBoxPointerPosition();
 
+        this._grabHelper.grab({ actor: this._summaryBoxPointer.bin.child,
+                                onUngrab: Lang.bind(this, this._onSummaryBoxPointerUngrabbed) });
+
         this._summaryBoxPointerState = State.SHOWING;
         this._summaryBoxPointer.show(BoxPointer.PopupAnimation.FULL, Lang.bind(this, function() {
             this._summaryBoxPointerState = State.SHOWN;


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