Menu: use the _classes method.

Fixes #7053

Author: petersendidit

tests/unit/menu/menu_common.js

@@ -1,5 +1,12 @@
 TestHelpers.commonWidgetTests( "menu", {
 	defaults: {
+		classes: {
+			"ui-menu": "ui-corner-all",
+			"ui-menu-icons": null,
+			"ui-menu-icon": null,
+			"ui-menu-item": null,
+			"ui-menu-divider": null
+		},
 		disabled: false,
 		icons: {
 			submenu: "ui-icon-carat-1-e"

tests/unit/menu/menu_core.js

@@ -3,9 +3,10 @@
 module( "menu: core" );
 
 test( "markup structure", function() {
-	expect( 6 );
+	expect( 7 );
 	var element = $( "#menu1" ).menu();
 	ok( element.hasClass( "ui-menu" ), "main element is .ui-menu" );
+	ok( element.hasClass( "ui-corner-all" ), "main element is .ui-corner-all" );
 	element.children().each(function( index ) {
 		ok( $( this ).hasClass( "ui-menu-item" ), "child " + index + " is .ui-menu-item" );
 	});

ui/menu.js

@@ -30,6 +30,13 @@ return $.widget( "ui.menu", {
 	defaultElement: "<ul>",
 	delay: 300,
 	options: {
+		classes: {
+			"ui-menu": "ui-corner-all",
+			"ui-menu-icons": null,
+			"ui-menu-icon": null,
+			"ui-menu-item": null,
+			"ui-menu-divider": null
+		},
 		icons: {
 			submenu: "ui-icon-carat-1-e"
 		},
@@ -55,8 +62,8 @@ return $.widget( "ui.menu", {
 		this.mouseHandled = false;
 		this.element
 			.uniqueId()
-			.addClass( "ui-menu ui-widget ui-widget-content" )
-			.toggleClass( "ui-menu-icons", !!this.element.find( ".ui-icon" ).length )
+			.addClass( this._classes( "ui-menu" ) + " ui-widget ui-widget-content" )
+			.toggleClass( this._classes( "ui-menu-icons" ), !!this.element.find( ".ui-icon" ).length )
 			.attr({
 				role: this.options.role,
 				tabIndex: 0
@@ -154,7 +161,7 @@ return $.widget( "ui.menu", {
 		this.element
 			.removeAttr( "aria-activedescendant" )
 			.find( ".ui-menu" ).addBack()
-				.removeClass( "ui-menu ui-widget ui-widget-content ui-menu-icons ui-front" )
+				.removeClass( this._classes( "ui-menu ui-menu-icons" ) + " ui-widget ui-widget-content ui-front" )
 				.removeAttr( "role" )
 				.removeAttr( "tabIndex" )
 				.removeAttr( "aria-labelledby" )
@@ -166,7 +173,7 @@ return $.widget( "ui.menu", {
 
 		// Destroy menu items
 		this.element.find( ".ui-menu-item" )
-			.removeClass( "ui-menu-item" )
+			.removeClass( this._classes( "ui-menu-item" ) )
 			.removeAttr( "role" )
 			.removeAttr( "aria-disabled" )
 			.removeUniqueId()
@@ -182,7 +189,8 @@ return $.widget( "ui.menu", {
 			});
 
 		// Destroy menu dividers
-		this.element.find( ".ui-menu-divider" ).removeClass( "ui-menu-divider ui-widget-content" );
+		this.element.find( ".ui-menu-divider" )
+			.removeClass( this._classes( "ui-menu-divider" ) + " ui-widget-content" );
 	},
 
 	_keydown: function( event ) {
@@ -285,7 +293,7 @@ return $.widget( "ui.menu", {
 
 		// Initialize nested menus
 		submenus.filter( ":not(.ui-menu)" )
-			.addClass( "ui-menu ui-widget ui-widget-content ui-front" )
+			.addClass( this._classes( "ui-menu" ) + " ui-widget ui-widget-content ui-front" )
 			.hide()
 			.attr({
 				role: this.options.role,
@@ -296,7 +304,7 @@ return $.widget( "ui.menu", {
 				var menu = $( this ),
 					item = menu.parent(),
 					submenuCarat = $( "<span>" )
-						.addClass( "ui-menu-icon ui-icon " + icon )
+						.addClass( that._classes( "ui-menu-icon" ) + " ui-icon " + icon )
 						.data( "ui-menu-submenu-carat", true );
 
 				item
@@ -312,13 +320,13 @@ return $.widget( "ui.menu", {
 		items.not( ".ui-menu-item" ).each(function() {
 			var item = $( this );
 			if ( that._isDivider( item ) ) {
-				item.addClass( "ui-widget-content ui-menu-divider" );
+				item.addClass( that._classes( "ui-menu-divider" ) + " ui-widget-content" );
 			}
 		});
 
 		// Don't refresh list items that are already adapted
 		items.not( ".ui-menu-item, .ui-menu-divider" )
-			.addClass( "ui-menu-item" )
+			.addClass( this._classes( "ui-menu-item" ) )
 			.uniqueId()
 			.attr({
 				tabIndex: -1,