From 993d37af1d04d586b8b977daf0da3588d63714d9 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=B6rn=20Zaefferer?= Date: Thu, 5 May 2011 14:20:56 +0200 Subject: [PATCH] Popup: Rename open flag to isOpen to avoid name conflict with open method. --- tests/visual/menu/popup.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) 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 ); } -- 2.39.5