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

kborchers · jzaefferer · commit 6d0859f71600 · 2011-07-14T11:38:03.000-04:00
diff --git 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
@@ -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" );
