From: jzaefferer <joern.zaefferer@gmail.com>
Date: Thu, 17 Mar 2011 14:37:27 +0000 (+0100)
Subject: Menubar: Another fix for tab/focus handling
X-Git-Tag: 1.9m5~204
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cb17d6d209e8fa82c920042ce8407cd592c2e798;p=jquery-ui.git

Menubar: Another fix for tab/focus handling
---

diff --git a/tests/visual/menu/menubar.js b/tests/visual/menu/menubar.js
index 35d0cc24f..3f4dba723 100644
--- a/tests/visual/menu/menubar.js
+++ b/tests/visual/menu/menubar.js
@@ -41,8 +41,8 @@ $.widget("ui.menubar", {
 					event.preventDefault();
 					break;
 				};
-			}).blur(function() {
-				self._close();
+			}).blur(function( event ) {
+				self._close( event );
 			});
 		});
 		items.each(function() {
@@ -97,7 +97,9 @@ $.widget("ui.menubar", {
 		});
 		self._bind(document, {
 			click: function(event) {
-				self.open && !$(event.target).closest(".ui-menubar").length && self._close();
+				if (self.open && !$(event.target).closest(".ui-menubar").length) {
+					self._close();
+				}
 			}
 		})
 		self._bind({
@@ -125,10 +127,10 @@ $.widget("ui.menubar", {
 	},
 	
 	_open: function(event, menu) {
-		// TODO refactor with _close
+		// almost the same as _close above, but don't remove tabIndex
 		if (this.active) {
 			this.active.menu("closeAll").hide();
-			this.active.prev().removeClass("ui-state-active").removeAttr("tabIndex");
+			this.active.prev().removeClass("ui-state-active");
 		}
 		clearTimeout(this.timer);
 		this.open = true;