Accordion: Handle negative values for active option on init.

scottgonzalez · scottgonzalez · commit d099e3a3b264 · 2011-02-03T09:59:05.000-05:00
diff --git a/tests/unit/accordion/accordion_options.js b/tests/unit/accordion/accordion_options.js
@@ -51,12 +51,11 @@ test( "{ active: Number }", function() {
 
 if ( $.uiBackCompat === false ) {
 	test( "{ active: -Number }", function() {
-		// TODO: fix initializing with negative value
 		var ac = $( "#list1" ).accordion({
-//			active: -1
+			active: -1
 		});
-//		equals( ac.accordion( "option", "active" ), 2 );
-//		state( ac, 0, 0, 1 );
+		equals( ac.accordion( "option", "active" ), 2 );
+		state( ac, 0, 0, 1 );
 
 		ac.accordion( "option", "active", -2 );
 		equals( ac.accordion( "option", "active" ), 1 );
diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
@@ -48,6 +48,10 @@ $.widget( "ui.accordion", {
 		if ( !options.collapsible && options.active === false ) {
 			options.active = 0;
 		}
+		// handle negative values
+		if ( options.active < 0 ) {
+			options.active += this.headers.length;
+		}
 		self.active = self._findActive( options.active )
 			.addClass( "ui-state-default ui-state-active" )
 			.toggleClass( "ui-corner-all" )
