From: kborchers Date: Thu, 14 Jul 2011 15:20:47 +0000 (-0500) Subject: Menu: Changed closeAll to collapseAll and updated in Menubar, also updated a referenc... X-Git-Tag: 1.9m6~98^2~2 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=6d0859f71600c3776913f90a39e9cfe086454e11;p=jquery-ui.git Menu: Changed closeAll to collapseAll and updated in Menubar, also updated a reference to left in Menubar to collapse. --- diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js index 66601ef38..f6012de4f 100644 --- a/ui/jquery.ui.menu.js +++ b/ui/jquery.ui.menu.js @@ -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 ); } }); diff --git a/ui/jquery.ui.menubar.js b/ui/jquery.ui.menubar.js index e28ea3e3c..fd6d92d6b 100644 --- a/ui/jquery.ui.menubar.js +++ b/ui/jquery.ui.menubar.js @@ -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" );