Menu: Update styles to use ui-widget-header for categories

kborchers · Mar 16, 2012 · cf6fa85 · cf6fa85 · kborchers · Mar 16, 2012
1 parent ee79303
commit cf6fa85
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/tests/visual/menu/menu.html b/tests/visual/menu/menu.html
@@ -79,7 +79,6 @@
 	<li><a href="#">Ada</a></li>
 	<li><a href="#">Adamsville</a></li>
 	<li><a href="#">Addyston</a></li>
-	<li></li>
 	<li>Group B</li>
 	<li>
 		<a href="#">Delphi</a>

diff --git a/themes/base/jquery.ui.menu.css b/themes/base/jquery.ui.menu.css
@@ -10,8 +10,7 @@
 .ui-menu { list-style:none; padding: 2px; margin: 0; display:block; outline: none; }
 .ui-menu .ui-menu { margin-top: -3px; position: absolute; }
 .ui-menu .ui-menu-item, .ui-menu .ui-menu-category { margin: 0; padding: 0; zoom: 1; width: 100%; }
-.ui-menu .ui-menu-category { font-weight: bold; border-bottom: 1px solid #AAA; line-height: 1.5; zoom: 1; }
-.ui-menu .ui-menu-category span { display: block; padding: 2px .4em; }
+.ui-menu .ui-widget-header { padding: 3px; }
 .ui-menu .ui-menu-item a { text-decoration: none; display: block; padding: 2px .4em; line-height: 1.5; zoom: 1; font-weight: normal; }
 .ui-menu .ui-menu-item a.ui-state-focus,
 .ui-menu .ui-menu-item a.ui-state-active { font-weight: normal; margin: -1px; }

diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js
@@ -282,9 +282,8 @@ $.widget( "ui.menu", {
 		menus.children( ":not( .ui-menu-item )" ).each( function() {
 			var item = $( this );
 			item
-				.addClass( "ui-menu-category" )
+				.addClass( "ui-widget-header" )
 				.attr( "role", "group" );
-			item.contents().wrapAll( "<span>" );
 		});
 
 		submenus.each( function() {