Selectmenu: Do not copy accesskey attributes

Author: fnagel

tests/unit/selectmenu/selectmenu_core.js

@@ -4,17 +4,10 @@ module( "selectmenu: core" );
 
 asyncTest( "accessibility", function() {
 	var links, button, menu,
-		element = $( "#speed" )
-			.attr({
-				accesskey: "s",
-				title: "A demo title"
-			});
+		element = $( "#speed" ).attr( "title", "A demo title" );
 
 	element.find( "option" ).each(function( index ) {
-		$( this ).attr({
-			accesskey: index,
-			title: "A demo title #" + index
-		});
+		$( this ).attr( "title", "A demo title #" + index );
 	});
 
 	element.selectmenu();
@@ -24,7 +17,7 @@ asyncTest( "accessibility", function() {
 	button.simulate( "focus" );
 	links = menu.find( "li.ui-menu-item" );
 
-	expect( 15 + links.length * 4 );
+	expect( 13 + links.length * 3 );
 
 	setTimeout(function() {
 		equal( button.attr( "role" ), "combobox", "button role" );
@@ -38,8 +31,6 @@ asyncTest( "accessibility", function() {
 			"button link aria-labelledby"
 		);
 		equal( button.attr( "tabindex" ), 0, "button link tabindex" );
-		ok( !element.attr( "accesskey" ), "element accesskey" );
-		equal( button.attr( "accesskey" ), "s", "button accesskey" );
 		equal( button.attr( "title" ), "A demo title", "button title" );
 
 		equal( menu.attr( "role" ), "listbox", "menu role" );
@@ -55,7 +46,6 @@ asyncTest( "accessibility", function() {
 			var link = $( this );
 			equal( link.attr( "role" ), "option", "menu link #" + index +" role" );
 			equal( link.attr( "tabindex" ), -1, "menu link #" + index +" tabindex" );
-			equal( link.attr( "accesskey" ), index, "menu link #" + index + " accesskey" );
 			equal( link.attr( "title" ), "A demo title #" + index, "menu link #" + index + " title" );
 		});
 		start();

ui/selectmenu.js

@@ -110,12 +110,6 @@ return $.widget( "ui.selectmenu", {
 		this._setText( this.buttonText, this.element.find( "option:selected" ).text() );
 		this._resizeButton();
 
-		this.accesskey = this.element.attr( "accesskey" );
-		if ( this.accesskey ) {
-			this.button.attr( "accesskey", this.accesskey );
-			this.element.removeAttr( "accesskey" );
-		}
-
 		this._on( this.button, this._buttonEvents );
 		this.button.one( "focusin", function() {
 
@@ -303,7 +297,6 @@ return $.widget( "ui.selectmenu", {
 
 	_renderItem: function( ul, item ) {
 		var li = $( "<li>", {
-			accesskey: item.accesskey,
 			title: item.element.attr( "title" )
 		});
 
@@ -564,11 +557,9 @@ return $.widget( "ui.selectmenu", {
 				index: index,
 				value: option.attr( "value" ),
 				label: option.text(),
-				accesskey: option.attr( "accesskey" ),
 				optgroup: optgroup.attr( "label" ) || "",
 				disabled: optgroup.prop( "disabled" ) || option.prop( "disabled" )
 			});
-			option.removeAttr( "accesskey" );
 		});
 		this.items = data;
 	},
@@ -579,14 +570,6 @@ return $.widget( "ui.selectmenu", {
 		this.element.show();
 		this.element.removeUniqueId();
 		this.label.attr( "for", this.ids.element );
-		if ( this.accesskey ) {
-			this.element.attr( "accesskey", this.accesskey );
-		}
-		$.each( this.items, function() {
-			if ( this.accesskey ) {
-				this.element.attr( "accesskey", this.accesskey );
-			}
-		});
 	}
 });