summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Nagel <info@felixnagel.com>2011-06-01 21:00:55 +0200
committerFelix Nagel <info@felixnagel.com>2011-06-01 21:00:55 +0200
commitd81640d440d80d0a9f2c1fe60050c87b0f8c8c30 (patch)
tree2572b590abfeca4dc10f7e1a17edf84efad4ea44
parentd5958c1b437a3324b5352f8617bfa0735499ba1b (diff)
downloadjquery-ui-d81640d440d80d0a9f2c1fe60050c87b0f8c8c30.tar.gz
jquery-ui-d81640d440d80d0a9f2c1fe60050c87b0f8c8c30.zip
revert: problems with popup style in multiple browsers because of this pull request: https://github.com/fnagel/jquery-ui/pull/94
-rw-r--r--ui/jquery.ui.selectmenu.js8
1 files changed, 4 insertions, 4 deletions
diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js
index 3447bc08d..a70dc6bc6 100644
--- a/ui/jquery.ui.selectmenu.js
+++ b/ui/jquery.ui.selectmenu.js
@@ -505,13 +505,13 @@ $.widget("ui.selectmenu", {
this.list.appendTo('body');
}
- this.list.addClass(self.widgetBaseClass + '-open').attr('aria-hidden', false);
- // FIX IE: Refreshing position before focusing the element, prevents IE from scrolling to the focused element before it is in position.
- this._refreshPosition();
- this.list.find('li:not(.' + self.widgetBaseClass + '-group):eq(' + this._selectedIndex() + ') a')[0].focus();
+ this.list.addClass(self.widgetBaseClass + '-open')
+ .attr('aria-hidden', false)
+ .find('li:not(.' + self.widgetBaseClass + '-group):eq(' + this._selectedIndex() + ') a')[0].focus();
if ( this.options.style == "dropdown" ) {
this.newelement.removeClass('ui-corner-all').addClass('ui-corner-top');
}
+ this._refreshPosition();
this._trigger("open", event, this._uiHash());
}
},