From afaf4e39db993b6b03650732fe58f1a79c913bd2 Mon Sep 17 00:00:00 2001
From: Krinkle <krinkle@users.mediawiki.org>
Date: Sun, 22 May 2011 23:52:16 +0000
Subject: [PATCH] jquery.tabIndex.js was broken, not it works (for the first
 time?) * Defaulting to null * Setting to initial value during first iteration
 (instead of outside the loop), that way we dont have to set minTabIndex to
 some insane high value to let if (  .. > .. ) work * (bug 29048)
 jQuery.tabIndex: firstTabIndex() outputs the same as lastTabIndex() * Added
 extra <textarea> to test suite to make sure the 'correct answer' is not the
 first or last element

---
 resources/jquery/jquery.tabIndex.js           | 18 ++++++-----
 tests/qunit/index.html                        |  1 +
 .../resources/jquery/jquery.tabIndex.js       | 30 ++++++++++++-------
 3 files changed, 32 insertions(+), 17 deletions(-)

diff --git a/resources/jquery/jquery.tabIndex.js b/resources/jquery/jquery.tabIndex.js
index 9d4da4d9a0..ec5371d2d5 100644
--- a/resources/jquery/jquery.tabIndex.js
+++ b/resources/jquery/jquery.tabIndex.js
@@ -8,10 +8,12 @@
  * @return number Lowest tabindex on the page
  */
 $.fn.firstTabIndex = function() {
-	var minTabIndex = 0;
-	$(this).find( '[tabindex]' ).each( function() {
+	var minTabIndex = null;
+	$(this).find( '[tabindex]' ).each( function( i ) {
 		var tabIndex = parseInt( $(this).attr( 'tabindex' ), 10 );
-		if ( tabIndex > minTabIndex ) {
+		if ( i === 0 ) {
+			minTabIndex = tabIndex;
+		} else if ( tabIndex < minTabIndex ) {
 			minTabIndex = tabIndex;
 		}
 	} );
@@ -24,13 +26,15 @@ $.fn.firstTabIndex = function() {
  * @return number Highest tabindex on the page
  */
 $.fn.lastTabIndex = function() {
-	var maxTabIndex = 0;
-	$(this).find( '[tabindex]' ).each( function() {
+	var maxTabIndex = null;
+	$(this).find( '[tabindex]' ).each( function( i ) {
 		var tabIndex = parseInt( $(this).attr( 'tabindex' ), 10 );
-		if ( tabIndex > maxTabIndex ) {
+		if ( i === 0 ) {
+			maxTabIndex = tabIndex;
+		} else if ( tabIndex > maxTabIndex ) {
 			maxTabIndex = tabIndex;
 		}
 	} );
 	return maxTabIndex;
 };
-} )( jQuery );
\ No newline at end of file
+} )( jQuery );
diff --git a/tests/qunit/index.html b/tests/qunit/index.html
index a59cda514e..d5a1a2ae67 100644
--- a/tests/qunit/index.html
+++ b/tests/qunit/index.html
@@ -47,6 +47,7 @@
 	<script src="suites/resources/jquery/jquery.mwPrototypes.js"></script>
 	<script src="suites/resources/mediawiki.util/mediawiki.util.js"></script>
 	<script src="suites/resources/jquery/jquery.autoEllipsis.js"></script>
+	<script src="suites/resources/jquery/jquery.colorUtil.js"></script>
 	<script src="suites/resources/jquery/jquery.tabIndex.js"></script>
 
 	<!-- TestSwarm: If a test swarm is running this,
diff --git a/tests/qunit/suites/resources/jquery/jquery.tabIndex.js b/tests/qunit/suites/resources/jquery/jquery.tabIndex.js
index 32862cd33b..73a180062b 100644
--- a/tests/qunit/suites/resources/jquery/jquery.tabIndex.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tabIndex.js
@@ -12,15 +12,20 @@ test( 'firstTabIndex', function(){
 	var testEnvironment = 
 '<form>\
 	<input tabindex="7" />\
-	<input tabindex="2" />\
-	<textarea tabindex="9">Foobar</textarea>\
+	<input tabindex="9" />\
+	<textarea tabindex="2">Foobar</textarea>\
+	<textarea tabindex="5">Foobar</textarea>\
 </form>';
-	var $test = $( '<div />' ).html( testEnvironment ).appendTo( 'body' );
+	var $testA = $( '<div />' ).html( testEnvironment ).appendTo( 'body' );
 
-	deepEqual( $test.firstTabIndex(), 2, 'First tabindex should be 2 within this context.' );
+	deepEqual( $testA.firstTabIndex(), 2, 'First tabindex should be 2 within this context.' );
+
+	var $testB = $( '<div />' );
+
+	deepEqual( $testB.firstTabIndex(), null, 'Return null if none available.' );
 
 	// Clean up
-	$test.remove();
+	$testA.add( $testB).remove();
 });
 
 test( 'lastTabIndex', function(){
@@ -28,13 +33,18 @@ test( 'lastTabIndex', function(){
 	var testEnvironment = 
 '<form>\
 	<input tabindex="7" />\
-	<input tabindex="2" />\
-	<textarea tabindex="9">Foobar</textarea>\
+	<input tabindex="9" />\
+	<textarea tabindex="2">Foobar</textarea>\
+	<textarea tabindex="5">Foobar</textarea>\
 </form>';
-	var $test = $( '<div />' ).html( testEnvironment ).appendTo( 'body' );
+	var $testA = $( '<div />' ).html( testEnvironment ).appendTo( 'body' );
+
+	deepEqual( $testA.lastTabIndex(), 9, 'Last tabindex should be 9 within this context.' );
+
+	var $testB = $( '<div />' );
 
-	deepEqual( $test.lastTabIndex(), 9, 'Last tabindex should be 9 within this context.' );
+	deepEqual( $testB.lastTabIndex(), null, 'Return null if none available.' );
 
 	// Clean up
-	$test.remove();
+	$testA.add( $testB).remove();
 });
\ No newline at end of file
-- 
2.20.1