Skip to content

Commit 94d87c8

Browse files
committed
Menu: Don't remove the icon elements on destroy since they exist in the original markup
1 parent e0fe788 commit 94d87c8

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

ui/jquery.ui.menu.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -157,10 +157,7 @@ $.widget( "ui.menu", {
157157
.removeClass( "ui-corner-all ui-state-hover" )
158158
.removeAttr( "tabIndex" )
159159
.removeAttr( "role" )
160-
.removeAttr( "aria-haspopup" )
161-
// TODO: is this correct? Don't these exist in the original markup?
162-
.children( ".ui-icon" )
163-
.remove();
160+
.removeAttr( "aria-haspopup" );
164161

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

0 commit comments

Comments
 (0)