Skip to content

Commit bf6fc20

Browse files
committed
fixed: removed useless _refreshPosition, thx to juzna, see https://github.com/fnagel/jquery-ui/issues/#issue/76
1 parent 14756d9 commit bf6fc20

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

ui/jquery.ui.selectmenu.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -492,9 +492,9 @@ $.widget("ui.selectmenu", {
492492

493493
open: function(event) {
494494
var self = this;
495-
var disabledStatus = this.newelement.attr("aria-disabled");
496-
if ( disabledStatus != 'true' ) {
497-
this._refreshPosition();
495+
if ( this.newelement.attr("aria-disabled") != 'true' ) {
496+
// TODO: seems to be useless
497+
// this._refreshPosition();
498498
this._closeOthers(event);
499499
this.newelement
500500
.addClass('ui-state-active');
@@ -678,6 +678,7 @@ $.widget("ui.selectmenu", {
678678

679679
_refreshPosition: function() {
680680
var o = this.options;
681+
console.log("_refreshPosition");
681682
// if its a native pop-up we need to calculate the position of the selected li
682683
if (o.style == "popup" && !o.positionOptions.offset) {
683684
var selected = this._selectedOptionLi();

0 commit comments

Comments
 (0)