Skip to content

Commit 44ef35e

Browse files
committed
Menu: Add dividers using empty list items
1 parent c0f5db5 commit 44ef35e

File tree

3 files changed

+10
-3
lines changed

3 files changed

+10
-3
lines changed

tests/visual/menu/menu.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
<li><a href="#">Ada</a></li>
7979
<li><a href="#">Adamsville</a></li>
8080
<li><a href="#">Addyston</a></li>
81+
<li></li>
8182
<li>
8283
<a href="#">Delphi</a>
8384
<ul>

themes/base/jquery.ui.menu.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
.ui-menu { list-style:none; padding: 2px; margin: 0; display:block; outline: none; }
1111
.ui-menu .ui-menu { margin-top: -3px; position: absolute; }
1212
.ui-menu .ui-menu-item { margin: 0; padding: 0; zoom: 1; width: 100%; }
13+
.ui-menu .ui-menu-divider { margin: 5px -2px 5px -2px; height: 0; font-size: 0; line-height: 0; }
1314
.ui-menu .ui-menu-item a { text-decoration: none; display: block; padding: 2px .4em; line-height: 1.5; zoom: 1; font-weight: normal; }
1415
.ui-menu .ui-menu-item a.ui-state-focus,
1516
.ui-menu .ui-menu-item a.ui-state-active { font-weight: normal; margin: -1px; }

ui/jquery.ui.menu.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,8 @@ $.widget( "ui.menu", {
260260

261261
refresh: function() {
262262
// initialize nested menus
263-
var menuId,
263+
var menus,
264+
menuId = this.menuId,
264265
submenus = this.element.find( this.options.menus + ":not( .ui-menu )" )
265266
.addClass( "ui-menu ui-widget ui-widget-content ui-corner-all" )
266267
.hide()
@@ -271,8 +272,9 @@ $.widget( "ui.menu", {
271272
});
272273

273274
// don't refresh list items that are already adapted
274-
menuId = this.menuId;
275-
submenus.add( this.element ).children( ":not( .ui-menu-item ):has( a )" )
275+
menus = submenus.add( this.element );
276+
277+
menus.children( ":not( .ui-menu-item ):has( a )" )
276278
.addClass( "ui-menu-item" )
277279
.attr( "role", "presentation" )
278280
.children( "a" )
@@ -283,6 +285,9 @@ $.widget( "ui.menu", {
283285
return menuId + "-" + i;
284286
});
285287

288+
// initialize unlinked menu-items as dividers
289+
menus.children( ":not( .ui-menu-item )" ).addClass( "ui-widget-content ui-menu-divider" );
290+
286291
submenus.each( function() {
287292
var menu = $( this ),
288293
item = menu.prev( "a" );

0 commit comments

Comments
 (0)