From: Felix Nagel Date: Thu, 17 Feb 2011 00:23:02 +0000 (+0100) Subject: fixed: removed useless _refreshPosition, thx to juzna, see https://github.com/fnagel... X-Git-Tag: selectmenu_v1.1.0~30 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=bf6fc206653763ae159f276b06cc1eddd059a98c;p=jquery-ui.git fixed: removed useless _refreshPosition, thx to juzna, see https://github.com/fnagel/jquery-ui/issues/#issue/76 --- 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();