From: jzaefferer <joern.zaefferer@gmail.com>
Date: Thu, 28 Oct 2010 17:22:57 +0000 (+0200)
Subject: Menu: Further refactoring of next/previousPage
X-Git-Tag: 1.9m3~2^2~2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=8959057ecced5287d94d1edd219ec00cf4117ef8;p=jquery-ui.git

Menu: Further refactoring of next/previousPage
---

diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js
index 58127566d..a67bc34bf 100644
--- a/ui/jquery.ui.menu.js
+++ b/ui/jquery.ui.menu.js
@@ -192,18 +192,14 @@ $.widget("ui.menu", {
 				height = this.element.height(),
 				result;
 			this.active.nextAll( ".ui-menu-item" ).each( function() {
-				var close = $( this ).offset().top - base - height;
-				if (close >= 0) {
-					result = $( this );
-					return false;
-				}
+				result = $( this );
+				return $( this ).offset().top - base - height < 0;
 			});
 
-			this.activate( event, result || this.element.children( ".ui-menu-item" ).last() );
+			this.activate( event, result );
 		} else {
 			this.activate( event, this.element.children( ".ui-menu-item" )
-				// TODO use .first()/.last()
-				.filter( !this.active || this.last() ? ":first" : ":last" ) );
+				[ !this.active || this.last() ? "first" : "last" ]() );
 		}
 	},
 
@@ -219,18 +215,14 @@ $.widget("ui.menu", {
 				height = this.element.height(),
 				result;
 			this.active.prevAll( ".ui-menu-item" ).each( function() {
-				var close = $(this).offset().top - base + height;
-				if (close <= 0) {
-					result = $( this );
-					return false;
-				}
+				result = $( this );
+				return $(this).offset().top - base + height > 0;
 			});
 
-			this.activate( event, result || this.element.children( ".ui-menu-item" ).first() );
+			this.activate( event, result );
 		} else {
 			this.activate( event, this.element.children( ".ui-menu-item" )
-				// TODO use .first()/.last()
-				.filter( !this.active || this.first() ? ":last" : ":first" ) );
+				[ !this.active || this.first() ? ":last" : ":first" ]() );
 		}
 	},