Tabs: Handle extraneous list items when using ui-tabs-active to set the initially active tab. Fixes #8568 - jQuery ui tabs: wrong default active li if ul contains extraneous elements.

scottgonzalez · scottgonzalez · commit fa62f21e5ad0 · 2012-10-09T09:07:11.000-04:00
diff --git a/ui/jquery.ui.tabs.js b/ui/jquery.ui.tabs.js
@@ -91,7 +91,7 @@ $.widget( "ui.tabs", {
 
 			// check for a tab marked active via a class
 			if ( active === null ) {
-				active = this.tabs.filter( ".ui-tabs-active" ).index();
+				active = this.tabs.index( this.tabs.filter( ".ui-tabs-active" ) );
 			}
 
 			// no active tab, set to false

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ $.widget( "ui.tabs", {`
`91`	`91`
`92`	`92`	`// check for a tab marked active via a class`
`93`	`93`	`if ( active === null ) {`
`94`		`- active = this.tabs.filter( ".ui-tabs-active" ).index();`
	`94`	`+ active = this.tabs.index( this.tabs.filter( ".ui-tabs-active" ) );`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`// no active tab, set to false`