From: Krinkle <krinkle@users.mediawiki.org>
Date: Mon, 6 Jun 2011 21:51:18 +0000 (+0000)
Subject: Alright, IE7 is fixed by r89613. IE6 stays problematic, for some reason it contains... 
X-Git-Tag: 1.31.0-rc.0~29648
X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=e0e6adbb2e0fb89d8362662d47d1ba8f15e77500;p=lhc%2Fweb%2Fwiklou.git

Alright, IE7 is fixed by r89613. IE6 stays problematic, for some reason it contains non-nodes as well (presumably whitespace/textnodes). This oughta fix it forever (poke r89605)
---

diff --git a/resources/jquery/jquery.tabIndex.js b/resources/jquery/jquery.tabIndex.js
index 4a42a7e96d..4c406772a4 100644
--- a/resources/jquery/jquery.tabIndex.js
+++ b/resources/jquery/jquery.tabIndex.js
@@ -15,7 +15,7 @@ $.fn.firstTabIndex = function() {
 		// becuase it has a default value for tabIndex in IE6/IE7 of 0
 		// (rather than null/undefined). Therefore check "> 0" as well
 		if ( tabIndex > 0 ) {
-			if ( i === 0 ) {
+			if ( minTabIndex === null ) {
 				minTabIndex = tabIndex;
 			} else if ( tabIndex < minTabIndex ) {
 				minTabIndex = tabIndex;
@@ -34,7 +34,7 @@ $.fn.lastTabIndex = function() {
 	var maxTabIndex = null;
 	$(this).find( '[tabindex]' ).each( function( i ) {
 		var tabIndex = parseInt( $(this).attr( 'tabindex' ), 10 );
-		if ( i === 0 ) {
+		if ( maxTabIndex === null ) {
 			maxTabIndex = tabIndex;
 		} else if ( tabIndex > maxTabIndex ) {
 			maxTabIndex = tabIndex;