From: kborchers <k_borchers@yahoo.com>
Date: Wed, 13 Jul 2011 20:33:17 +0000 (-0500)
Subject: Menu: Remove the ui-state-active class from the siblings of the newly activated menu... 
X-Git-Tag: 1.9m6~98^2~6
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=bc93b3ff080160e4e8e7a035e2e4421d1521e224;p=jquery-ui.git

Menu: Remove the ui-state-active class from the siblings of the newly activated menu item without a delay to fix a bug where mouseover of a menu item directly adjacent to the active item would cause a 1 pixel jump due to both items having a class with a border (ui-state-active and ui-state-focus)
---

diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js
index 770e0252a..5f9de6134 100644
--- a/ui/jquery.ui.menu.js
+++ b/ui/jquery.ui.menu.js
@@ -58,6 +58,8 @@ $.widget( "ui.menu", {
 				}
 				var target = $( event.target ).closest( ".ui-menu-item" );
 				if ( target.length ) {
+					//Remove ui-state-active class from siblings of the newly focused menu item to avoid a jump caused by adjacent elements both having a class with a border
+					target.siblings().children( ".ui-state-active" ).removeClass( "ui-state-active" );
 					self.focus( event, target );
 				}
 			})