From 22e1abbaff2533b8e7d1e6426c5277b990acc8e6 Mon Sep 17 00:00:00 2001
From: Rui Matos <tiagomatos@gmail.com>
Date: Tue, 13 Mar 2012 19:14:18 +0100
Subject: [PATCH] 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 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/js/ui/panel.js b/js/ui/panel.js
index 8f0919042..eb9354768 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,