Skip to content
Permalink
Browse files

Autocomplete: Use .ui-front instead of .zIndex() for the suggestions …

…menu.
  • Loading branch information
scottgonzalez committed Feb 5, 2013
1 parent 7a49ee4 commit 7d5fe02dd9e3af3ebbb4fcc3092045139b166c48
Showing with 2 additions and 5 deletions.
  1. +2 −5 ui/jquery.ui.autocomplete.js
@@ -192,15 +192,14 @@ $.widget( "ui.autocomplete", {

this._initSource();
this.menu = $( "<ul>" )
.addClass( "ui-autocomplete" )
.addClass( "ui-autocomplete ui-front" )
.appendTo( this._appendTo() )
.menu({
// custom key handling for now
input: $(),
// disable ARIA support, the live region takes care of that
role: null
})
.zIndex( this.element.zIndex() + 1 )
.hide()
.data( "ui-menu" );

@@ -489,9 +488,7 @@ $.widget( "ui.autocomplete", {
},

_suggest: function( items ) {
var ul = this.menu.element
.empty()
.zIndex( this.element.zIndex() + 1 );
var ul = this.menu.element.empty();
this._renderMenu( ul, items );
this.menu.refresh();

3 comments on commit 7d5fe02

@StanleyMlha

This comment has been minimized.

Copy link

StanleyMlha replied Apr 18, 2013

It doesn´t work in IE8

@scottgonzalez

This comment has been minimized.

Copy link
Member Author

scottgonzalez replied Apr 18, 2013

What doesn't work? If you'd like to report a bug, please do so at http://bugs.jqueryui.com/newticket and make sure to include a reduced test case showing the problem.

@StanleyMlha

This comment has been minimized.

Copy link

StanleyMlha replied Apr 18, 2013

autocomplete in dialog box

Please sign in to comment.
You can’t perform that action at this time.