Skip to content

Commit baf28d2

Browse files
committed
Menu: Cleanup contextmenu demo and fix aria unit test to ignore empty
string values
1 parent 540b78d commit baf28d2

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

demos/menu/contextmenu.html

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,7 @@
2323
select: function(event, ui) {
2424
$(this).hide();
2525
$("#log").append("<div>Selected " + ui.item.text() + "</div>");
26-
},
27-
input: $(this)
26+
}
2827
}).hide();
2928
}).click(function(event) {
3029
var menu = $(this).next();

tests/unit/menu/menu_core.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ test("accessibility", function () {
1414

1515
ok( ac.hasClass("ui-menu ui-widget ui-widget-content ui-corner-all"), "menu class");
1616
equals( ac.attr("role"), "listbox", "main role");
17-
equals( ac.attr("aria-activedescendant"), undefined, "aria attribute not yet active");
17+
ok( !ac.attr("aria-activedescendant"), "aria attribute not yet active");
1818
var item = ac.find( "li:first" ).find( "a" ).attr( "id", "xid" ).end();
1919
ac.menu( "focus", $.Event(), item );
2020
equals( ac.attr("aria-activedescendant"), "xid", "aria attribute, id from dom");

0 commit comments

Comments
 (0)