From 019dcc26e307614681a6dfd0fcd9665548cb6ab6 Mon Sep 17 00:00:00 2001 From: kborchers Date: Mon, 9 Jul 2012 23:20:43 -0500 Subject: [PATCH] Menu: Remove incorrect scroll check behavior from focus event handler --- ui/jquery.ui.menu.js | 22 +++++----------------- 1 file changed, 5 insertions(+), 17 deletions(-) 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 ) { -- 2.39.5