Autocomplete: Re-add wrappers to menu items

Author: tjvantoll

demos/autocomplete/custom-data.html

@@ -68,7 +68,7 @@
 		})
 		.autocomplete( "instance" )._renderItem = function( ul, item ) {
 			return $( "<li>" )
-				.append( "<a>" + item.label + "<br>" + item.desc + "</a>" )
+				.append( "<div>" + item.label + "<br>" + item.desc + "</div>" )
 				.appendTo( ul );
 		};
 	});

tests/unit/autocomplete/autocomplete_core.js

@@ -16,7 +16,7 @@ test( "prevent form submit on enter when menu is active", function() {
 	event = $.Event( "keydown" );
 	event.keyCode = $.ui.keyCode.DOWN;
 	element.trigger( event );
-	equal( menu.find( ".ui-menu-item.ui-state-focus" ).length, 1, "menu item is active" );
+	equal( menu.find( ".ui-menu-item .ui-state-focus" ).length, 1, "menu item is active" );
 
 	event = $.Event( "keydown" );
 	event.keyCode = $.ui.keyCode.ENTER;

tests/unit/autocomplete/autocomplete_options.js

@@ -77,7 +77,7 @@ function autoFocusTest( afValue, focusedLength ) {
 		open: function() {
 			equal(
 				element.autocomplete( "widget" )
-					.children( ".ui-menu-item.ui-state-focus" )
+					.find( ".ui-menu-item .ui-state-focus" )
 					.length,
 				focusedLength,
 				"first item is " + (afValue ? "" : "not") + " auto focused" );

ui/autocomplete.js

@@ -542,7 +542,9 @@ $.widget( "ui.autocomplete", {
 	},
 
 	_renderItem: function( ul, item ) {
-		return $( "<li>" ).text( item.label ).appendTo( ul );
+		return $( "<li>" )
+			.append( $( "<div>" ).text( item.label ) )
+			.appendTo( ul );
 	},
 
 	_move: function( direction, event ) {