From 0bbd1569182bc03e8dc4f5f8aa203e8edbe15f99 Mon Sep 17 00:00:00 2001 From: Jörn Zaefferer Date: Mon, 19 Jan 2015 17:00:57 +0100 Subject: Menu: Use ui-state-active consistently So far we were using ui-state-active on active parent items, ui-state-focus on active child items. The theme update highlighted the visual inconsistency. With this change, only ui-state-active is used. Fixes #10692 --- tests/unit/autocomplete/autocomplete_core.js | 2 +- tests/unit/autocomplete/autocomplete_options.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'tests/unit/autocomplete') diff --git a/tests/unit/autocomplete/autocomplete_core.js b/tests/unit/autocomplete/autocomplete_core.js index fa3f9abb0..1208d89f1 100644 --- a/tests/unit/autocomplete/autocomplete_core.js +++ b/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-wrapper.ui-state-focus" ).length, 1, + equal( menu.find( ".ui-menu-item-wrapper.ui-state-active" ).length, 1, "menu item is active" ); event = $.Event( "keydown" ); diff --git a/tests/unit/autocomplete/autocomplete_options.js b/tests/unit/autocomplete/autocomplete_options.js index 226db29ed..8215dbe61 100644 --- a/tests/unit/autocomplete/autocomplete_options.js +++ b/tests/unit/autocomplete/autocomplete_options.js @@ -77,7 +77,7 @@ function autoFocusTest( afValue, focusedLength ) { open: function() { equal( element.autocomplete( "widget" ) - .find( ".ui-menu-item-wrapper.ui-state-focus" ) + .find( ".ui-menu-item-wrapper.ui-state-active" ) .length, focusedLength, "first item is " + (afValue ? "" : "not") + " auto focused" ); -- cgit v1.2.3