From: Jörn Zaefferer Date: Thu, 5 May 2011 12:20:56 +0000 (+0200) Subject: Popup: Rename open flag to isOpen to avoid name conflict with open method. X-Git-Tag: 1.9m5~86 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=993d37af1d04d586b8b977daf0da3588d63714d9;p=jquery-ui.git Popup: Rename open flag to isOpen to avoid name conflict with open method. --- diff --git a/tests/visual/menu/popup.js b/tests/visual/menu/popup.js index dc81b694c..08088c994 100644 --- a/tests/visual/menu/popup.js +++ b/tests/visual/menu/popup.js @@ -45,7 +45,7 @@ $.widget( "ui.popup", { }, click: function( event ) { event.preventDefault(); - if (this.open) { + if (this.isOpen) { // let it propagate to close return; } @@ -75,7 +75,7 @@ $.widget( "ui.popup", { this._bind(document, { click: function( event ) { - if (this.open && !$(event.target).closest(".ui-popup").length) { + if (this.isOpen && !$(event.target).closest(".ui-popup").length) { this.close( event ); } } @@ -116,7 +116,7 @@ $.widget( "ui.popup", { // take trigger out of tab order to allow shift-tab to skip trigger this.options.trigger.attr("tabindex", -1); - this.open = true; + this.isOpen = true; this._trigger( "open", event ); }, @@ -128,7 +128,7 @@ $.widget( "ui.popup", { this.options.trigger.attr("tabindex", 0); - this.open = false; + this.isOpen = false; this._trigger( "close", event ); }