Menu: Bind to mouseenter instead of mouseover to avoid use of stopImmediatePropagation

kborchers · kborchers · commit 00306d6bbdf6 · 2012-04-19T23:49:33.000-05:00
diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js
@@ -87,8 +87,7 @@ $.widget( "ui.menu", {
 					}
 				}
 			},
-			"mouseover .ui-menu-item": function( event ) {
-				event.stopImmediatePropagation();
+			"mouseenter .ui-menu-item": function( event ) {
 				var target = $( event.currentTarget );
 				// 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

Original file line number	Diff line number	Diff line change
`@@ -87,8 +87,7 @@ $.widget( "ui.menu", {`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`	`},`
`90`		`- "mouseover .ui-menu-item": function( event ) {`
`91`		`- event.stopImmediatePropagation();`
	`90`	`+ "mouseenter .ui-menu-item": function( event ) {`
`92`	`91`	`var target = $( event.currentTarget );`
`93`	`92`	`// Remove ui-state-active class from siblings of the newly focused menu item`
`94`	`93`	`// to avoid a jump caused by adjacent elements both having a class with a border`