Skip to content
Permalink
Browse files

Menu: Only look for .ui-menu-item elements to determine if a menu ite…

…m is the first or last element in the menu. Fixes #5528 - Autocomplete: Issue when try to navigate using up key by keyboard in IE6.
  • Loading branch information...
scottgonzalez committed Jul 19, 2010
1 parent a090faa commit 5311fe22f327d275f6f8d885da1ce6bbc2c4a681
Showing with 2 additions and 2 deletions.
  1. +2 −2 ui/jquery.ui.autocomplete.js
@@ -433,11 +433,11 @@ $.widget("ui.menu", {
},

first: function() {
return this.active && !this.active.prev().length;
return this.active && !this.active.prevAll(".ui-menu-item").length;
},

last: function() {
return this.active && !this.active.next().length;
return this.active && !this.active.nextAll(".ui-menu-item").length;
},

move: function(direction, edge, event) {

2 comments on commit 5311fe2

@jzaefferer

This comment has been minimized.

Copy link
Member

replied Jul 20, 2010

Are the other changes from the menu branch that need to be applied to autocomplete?

@scottgonzalez

This comment has been minimized.

Copy link
Member Author

replied Jul 20, 2010

I haven't looked over the menu branch yet. I've got a note about merging this change into the menu branch, but I'm not sure when we want to pull menu into master.

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