From: kborchers <kris.borchers@gmail.com>
Date: Tue, 10 Jul 2012 04:20:43 +0000 (-0500)
Subject: Menu: Remove incorrect scroll check behavior from focus event handler
X-Git-Tag: 1.9.0-rc.1~63
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=019dcc26e307614681a6dfd0fcd9665548cb6ab6;p=jquery-ui.git

Menu: Remove incorrect scroll check behavior from focus event handler
---

diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js
index 127a09bb5..3658bd8d5 100644
--- a/ui/jquery.ui.menu.js
+++ b/ui/jquery.ui.menu.js
@@ -101,25 +101,13 @@ $.widget( "ui.menu", {
 			"mouseleave .ui-menu": "collapseAll",
 			focus: function( event ) {
 				var menuTop,
-					menu = this.element,
-					// Default to focusing the first item
-					item = menu.children( ".ui-menu-item" ).eq( 0 );
+					item,
+					menu = this.element;
 
 				// If there's already an active item, keep it active
-				if ( this.active ) {
-					item = this.active;
-				// If there's no active item and the menu is scrolled,
-				// then find the first visible item
-				} else if ( this._hasScroll() ) {
-					menuTop = menu.offset().top;
-					menu.children().each(function() {
-						var currentItem = $( this );
-						if ( currentItem.offset().top - menuTop >= 0 ) {
-							item = currentItem;
-							return false;
-						}
-					});
-				}
+				// If not, activate the first item
+				item = this.active || menu.children( ".ui-menu-item" ).eq( 0 );
+
 				this.focus( event, item );
 			},
 			blur: function( event ) {