Menu: Changed closeAll to collapseAll and updated in Menubar, also updated a reference to left in Menubar to collapse.

This commit is contained in:
kborchers 2011-07-14 10:20:47 -05:00 committed by Jörn Zaefferer
parent 11d24ee84c
commit 6d0859f716
2 changed files with 7 additions and 7 deletions

View File

@ -176,7 +176,7 @@ $.widget( "ui.menu", {
this._bind( document, {
click: function( event ) {
if ( !$( event.target ).closest( ".ui-menu" ).length ) {
this.closeAll();
this.collapseAll();
}
}
});
@ -336,7 +336,7 @@ $.widget( "ui.menu", {
.position( position );
},
closeAll: function() {
collapseAll: function() {
this.element
.find( "ul" )
.hide()
@ -466,11 +466,11 @@ $.widget( "ui.menu", {
},
select: function( event ) {
// save active reference before closeAll triggers blur
// save active reference before collapseAll triggers blur
var ui = {
item: this.active
};
this.closeAll();
this.collapseAll();
this._trigger( "select", event, ui );
}
});

View File

@ -151,7 +151,7 @@ $.widget( "ui.menubar", {
});
that._bind( {
keydown: function( event ) {
if ( event.keyCode == $.ui.keyCode.ESCAPE && that.active && that.active.menu( "left", event ) !== true ) {
if ( event.keyCode == $.ui.keyCode.ESCAPE && that.active && that.active.menu( "collapse", event ) !== true ) {
var active = that.active;
that.active.blur();
that._close( event );
@ -206,7 +206,7 @@ $.widget( "ui.menubar", {
if ( !this.active || !this.active.length )
return;
this.active
.menu( "closeAll" )
.menu( "collapseAll" )
.hide()
.attr( "aria-hidden", "true" )
.attr( "aria-expanded", "false" );
@ -226,7 +226,7 @@ $.widget( "ui.menubar", {
// TODO refactor, almost the same as _close above, but don't remove tabIndex
if ( this.active ) {
this.active
.menu( "closeAll" )
.menu( "collapseAll" )
.hide()
.attr( "aria-hidden", "true" )
.attr( "aria-expanded", "false" );