Skip to content

Commit fe6d244

Browse files
committed
Revert "Menubar: Properly track opening and closing of submenus for proper keyboard navigation"
This reverts commit 26e3d3b.
1 parent 76338ca commit fe6d244

File tree

1 file changed

+0
-16
lines changed

1 file changed

+0
-16
lines changed

ui/jquery.ui.menubar.js

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -168,9 +168,6 @@ $.widget( "ui.menubar", {
168168
clearTimeout( that.closeTimer );
169169
}
170170
});
171-
172-
// Keep track of open submenus
173-
this.openSubmenus = 0;
174171
},
175172

176173
_destroy : function() {
@@ -221,7 +218,6 @@ $.widget( "ui.menubar", {
221218
.removeAttr( "tabIndex" );
222219
this.active = null;
223220
this.open = false;
224-
this.openSubmenus = 0;
225221
},
226222

227223
_open: function( event, menu ) {
@@ -259,22 +255,10 @@ $.widget( "ui.menubar", {
259255
},
260256

261257
next: function( event ) {
262-
if ( this.open && this.active.data( "menu" ).active.has( ".ui-menu" ).length ) {
263-
// Track number of open submenus and prevent moving to next menubar item
264-
this.openSubmenus++;
265-
return;
266-
}
267-
this.openSubmenus = 0;
268258
this._move( "next", "first", event );
269259
},
270260

271261
previous: function( event ) {
272-
if ( this.open && this.openSubmenus ) {
273-
// Track number of open submenus and prevent moving to previous menubar item
274-
this.openSubmenus--;
275-
return;
276-
}
277-
this.openSubmenus = 0;
278262
this._move( "prev", "last", event );
279263
},
280264

0 commit comments

Comments
 (0)