Skip to content
Permalink
Browse files

Menu: Remove display property on hidden submenus instead of setting d…

…isplay:block. Surfaced by domEqual now checking for style property (see d687a1b).
  • Loading branch information
jzaefferer committed Dec 3, 2012
1 parent d687a1b commit 53b940fa27fc50b4663eef77249a9afce4129240
Showing with 3 additions and 1 deletion.
  1. +3 −1 ui/jquery.ui.menu.js
@@ -148,7 +148,9 @@ $.widget( "ui.menu", {
.removeAttr( "aria-hidden" )
.removeAttr( "aria-disabled" )
.removeUniqueId()
.show();
.css({
display: ""
});

// Destroy menu items
this.element.find( ".ui-menu-item" )

2 comments on commit 53b940f

@mikesherov

This comment has been minimized.

Copy link
Member

mikesherov replied Dec 4, 2012

why?

@scottgonzalez

This comment has been minimized.

Copy link
Member

scottgonzalez replied Dec 4, 2012

Reverted in 8072099.

Please sign in to comment.
You can’t perform that action at this time.