Skip to content
Permalink
Browse files

Menu: Don't remove the icon elements on destroy since they exist in t…

…he original markup
  • Loading branch information...
kborchers committed Jun 1, 2012
1 parent e0fe788 commit 94d87c8ad61be23ba17836e964ec28099ac21669
Showing with 1 addition and 4 deletions.
  1. +1 −4 ui/jquery.ui.menu.js
@@ -157,10 +157,7 @@ $.widget( "ui.menu", {
.removeClass( "ui-corner-all ui-state-hover" )
.removeAttr( "tabIndex" )
.removeAttr( "role" )
.removeAttr( "aria-haspopup" )
// TODO: is this correct? Don't these exist in the original markup?
.children( ".ui-icon" )
.remove();
.removeAttr( "aria-haspopup" );

// unbind currentEventTarget click event handler
$( currentEventTarget ).unbind( "click.menu" );

0 comments on commit 94d87c8

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