diff options
author | Felix Nagel <info@felixnagel.com> | 2010-11-29 00:13:49 +0100 |
---|---|---|
committer | Felix Nagel <info@felixnagel.com> | 2010-11-29 00:13:49 +0100 |
commit | 58085b49f6377e874cef540d2f4d68f113d2d651 (patch) | |
tree | 8edb8ff9f6702f2ff28ce32e208817c3c414ce50 | |
parent | 158a966b56fb91470d371baee37d8c8847150492 (diff) | |
download | jquery-ui-58085b49f6377e874cef540d2f4d68f113d2d651.tar.gz jquery-ui-58085b49f6377e874cef540d2f4d68f113d2d651.zip |
fixed: selectmenu widget now works correct with jQuery UI dialog widegt, thx to dannycohn, see https://github.com/fnagel/jquery-ui/issues#issue/23
-rw-r--r-- | ui/jquery.ui.selectmenu.js | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js index 4b32e5af6..d160f28d8 100644 --- a/ui/jquery.ui.selectmenu.js +++ b/ui/jquery.ui.selectmenu.js @@ -530,20 +530,24 @@ $.widget("ui.selectmenu", { this.list.attr('aria-activedescendant', activeID); }, _refreshPosition: function(){ - var o = this.options; + var o = this.options; // 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.list.find('li:not(.ui-selectmenu-group):eq('+this._selectedIndex()+')'); // var _offset = "0 -" + (selected.outerHeight() + selected.offset().top - this.list.offset().top); var _offset = "0 -" + (selected.outerHeight() + selected.offset().top - this.list.offset().top); } - this.list.position({ - // set options for position plugin - of: o.positionOptions.of || this.newelement, - my: o.positionOptions.my, - at: o.positionOptions.at, - offset: o.positionOptions.offset || _offset - }); + this.list + .css({ + zIndex: this.element.zIndex() + }) + .position({ + // set options for position plugin + of: o.positionOptions.of || this.newelement, + my: o.positionOptions.my, + at: o.positionOptions.at, + offset: o.positionOptions.offset || _offset + }); } }); })(jQuery);
\ No newline at end of file |