aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Nagel <info@felixnagel.com>2010-11-29 00:13:49 +0100
committerFelix Nagel <info@felixnagel.com>2010-11-29 00:13:49 +0100
commit58085b49f6377e874cef540d2f4d68f113d2d651 (patch)
tree8edb8ff9f6702f2ff28ce32e208817c3c414ce50
parent158a966b56fb91470d371baee37d8c8847150492 (diff)
downloadjquery-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.js20
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