Autocomplete: Only prevent the default action for enter when a menu item is active. Fixes #6038 - Autocomplete: Allow default behaviour on enter when menu is open but inactive.

Thanks Ján Suchal.

1 files changed, 31 insertions, 2 deletions

diff --git a/tests/unit/autocomplete/autocomplete_core.js b/tests/unit/autocomplete/autocomplete_core.js
index 9b8709abb..fcd835d1a 100644
--- a/tests/unit/autocomplete/autocomplete_core.js
+++ b/tests/unit/autocomplete/autocomplete_core.js
@@ -23,7 +23,7 @@ test("close-on-blur is properly delayed", function() {
 		same( $(".ui-menu:visible").length, 0 );
 		start();
 	}, 200);
-})
+});
 
 test("close-on-blur is cancelled when starting a search", function() {
 	var ac = $("#autocomplete").autocomplete({
@@ -38,6 +38,35 @@ test("close-on-blur is cancelled when starting a search", function() {
 		same( $(".ui-menu:visible").length, 1 );
 		start();
 	}, 200);
-})
+});
+
+test( "prevent form submit on enter when menu is active", function() {
+	var event;
+	var ac = $( "#autocomplete" ).autocomplete({
+		source: [ "java", "javascript" ]
+	}).val( "ja" ).autocomplete( "search" );
+	
+	event = $.Event( "keydown" );
+	event.keyCode = $.ui.keyCode.DOWN;
+	ac.trigger( event );
+	same( $( ".ui-menu-item:has(.ui-state-hover)" ).length, 1, "menu item is active" );
+	
+	event = $.Event( "keydown" );
+	event.keyCode = $.ui.keyCode.ENTER;
+	ac.trigger( event );
+	ok( event.isDefaultPrevented(), "default action is prevented" );
+});
+
+test( "allow form submit on enter when menu is not active", function() {
+	var event;
+	var ac = $( "#autocomplete" ).autocomplete({
+		source: [ "java", "javascript" ]
+	}).val( "ja" ).autocomplete( "search" );
+	
+	event = $.Event( "keydown" );
+	event.keyCode = $.ui.keyCode.ENTER;
+	ac.trigger( event );
+	ok( !event.isDefaultPrevented(), "default action is prevented" );
+});
 
 })(jQuery);