Menu: Filter out non-items when typing

Fixes #10571
Closes gh-1329

2 files changed, 26 insertions, 0 deletions

diff --git a/tests/unit/menu/menu.html b/tests/unit/menu/menu.html
index 05fdcee11..2aa286c69 100644
--- a/tests/unit/menu/menu.html
+++ b/tests/unit/menu/menu.html
@@ -306,6 +306,15 @@
 	<li>Amesville</li>
 </ul>
 
+<ul id="menu8">
+	<li class="foo">Aberdeen</li>
+	<li class="foo ui-state-disabled">Ada</li>
+	<li class="foo">Adamsville</li>
+	<li class="foo">Addyston</li>
+	<li class="foo">-</li>
+	<li class="foo">-Saarland</li>
+</ul>
+
 </div>
 </body>
 </html>
diff --git a/tests/unit/menu/menu_events.js b/tests/unit/menu/menu_events.js
index 8fa77ab95..0a02cc01b 100644
--- a/tests/unit/menu/menu_events.js
+++ b/tests/unit/menu/menu_events.js
@@ -644,4 +644,21 @@ test( "#9469: Stopping propagation in a select event should not suppress subsequ
 	equal( logOutput(), "1,2", "Both select events were not triggered." );
 });
 
+asyncTest( "#10571: When typing in a menu, only menu-items should be focused", function() {
+	expect( 3 );
+
+	var element = $( "#menu8" ).menu({
+		focus: function( event, ui ) {
+			equal( ui.item.length, 1, "There should only be one match when filtering" );
+			ok( ui.item.hasClass( "ui-menu-item" ), "element is .ui-menu-item" );
+			equal( ui.item.text(), "-Saarland", "element has correct text" );
+		}
+	});
+
+	setTimeout(function() {
+		element.menu( "widget" ).simulate( "keydown", { keyCode: "-".charCodeAt( 0 ) } );
+		start();
+	});
+});
+
 })( jQuery );