From bf6fc206653763ae159f276b06cc1eddd059a98c Mon Sep 17 00:00:00 2001 From: Felix Nagel Date: Thu, 17 Feb 2011 01:23:02 +0100 Subject: [PATCH] fixed: removed useless _refreshPosition, thx to juzna, see https://github.com/fnagel/jquery-ui/issues/#issue/76 --- ui/jquery.ui.selectmenu.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js index bc875145a..6df51df9a 100644 --- a/ui/jquery.ui.selectmenu.js +++ b/ui/jquery.ui.selectmenu.js @@ -492,9 +492,9 @@ $.widget("ui.selectmenu", { open: function(event) { var self = this; - var disabledStatus = this.newelement.attr("aria-disabled"); - if ( disabledStatus != 'true' ) { - this._refreshPosition(); + if ( this.newelement.attr("aria-disabled") != 'true' ) { + // TODO: seems to be useless + // this._refreshPosition(); this._closeOthers(event); this.newelement .addClass('ui-state-active'); @@ -678,6 +678,7 @@ $.widget("ui.selectmenu", { _refreshPosition: function() { var o = this.options; + console.log("_refreshPosition"); // if its a native pop-up we need to calculate the position of the selected li if (o.style == "popup" && !o.positionOptions.offset) { var selected = this._selectedOptionLi(); -- 2.39.5