Menu: skip over elements that aren't menu items.

Fixes #5188 - Refactor menu rendering for more customizing.

demos/autocomplete/categories.html

@@ -19,29 +19,6 @@
 	}
 	</style>
 	<script type="text/javascript">
-	$.extend( $.ui.menu.prototype, {
-		next: function() {
-			this.move("next", ".ui-menu-item:first");
-		},
-
-		previous: function() {
-			this.move("prev", ".ui-menu-item:last");
-		},
-
-		move: function(direction, edge) {
-			if (!this.active) {
-				this.activate(this.element.children(edge));
-				return;
-			}
-			var next = this.active[direction + "All"]('.ui-menu-item').eq( 0 );
-			if (next.length) {
-				this.activate(next);
-			} else {
-				this.activate(this.element.children(edge));
-			}
-		}
-	});
-
 	$.widget("custom.catcomplete", $.ui.autocomplete, {
 		_renderMenu: function( ul, items ) {
 			var self = this,

ui/jquery.ui.autocomplete.js

@@ -391,11 +391,11 @@ $.widget("ui.menu", {
 	},
 
 	next: function(event) {
-		this.move("next", "li:first", event);
+		this.move("next", ".ui-menu-item:first", event);
 	},
 
 	previous: function(event) {
-		this.move("prev", "li:last", event);
+		this.move("prev", ".ui-menu-item:last", event);
 	},
 
 	first: function() {
@@ -411,7 +411,7 @@ $.widget("ui.menu", {
 			this.activate(event, this.element.children(edge));
 			return;
 		}
-		var next = this.active[direction]();
+		var next = this.active[direction + "All"](".ui-menu-item").eq(0);
 		if (next.length) {
 			this.activate(event, next);
 		} else {