[FIXED JENKINS-17050] Close pop-up menu when clicking outside.

The _activationButtonPressed get's set to false onMouseUp, so can not be used to decide if the menu is active or not.
jenkinsci · Apr 19, 2014 · 0b49ff9 · 0b49ff9
1 parent dd7d837
commit 0b49ff9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/war/src/main/webapp/scripts/yui/button/button-debug.js b/war/src/main/webapp/scripts/yui/button/button-debug.js
@@ -1525,7 +1525,7 @@ version: 2.9.0
             }
 
 
-            if (this._activationButtonPressed || this._bOptionPressed) {
+            if (this.isActive() || this._bOptionPressed) {
 
                 Event.removeListener(document, "mouseup", this._onDocumentMouseUp);
 
@@ -1583,7 +1583,7 @@ version: 2.9.0
             }
 
 
-            if (this._activationButtonPressed || this._bOptionPressed) {
+            if (this.isActive() || this._bOptionPressed) {
 
                 Event.on(document, "mouseup", this._onDocumentMouseUp, null, this);