[gnome-shell] AppMenuButton: Don't set the actor reactive if it's not really visible



commit 22e1abbaff2533b8e7d1e6426c5277b990acc8e6
Author: Rui Matos <tiagomatos gmail com>
Date:   Tue Mar 13 19:14:18 2012 +0100

    AppMenuButton: Don't set the actor reactive if it's not really visible
    
    Reactive means that the actor is reachable from keyboard
    navigation. If the target isn't current that means we are not tweening
    the actor to be visible so we shouldn't set it reactive either.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=671998

 js/ui/panel.js |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
---
diff --git a/js/ui/panel.js b/js/ui/panel.js
index 8f09190..eb93547 100644
--- a/js/ui/panel.js
+++ b/js/ui/panel.js
@@ -317,11 +317,12 @@ const AppMenuButton = new Lang.Class({
 
         this._visible = true;
         this.actor.show();
-        this.actor.reactive = true;
 
         if (!this._targetIsCurrent)
             return;
 
+        this.actor.reactive = true;
+
         Tweener.removeTweens(this.actor);
         Tweener.addTween(this.actor,
                          { opacity: 255,



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