Accordion: Update active option when programmatically collapsing the accordion.

scottgonzalez · scottgonzalez · commit 2e17770e5466 · 2011-02-03T10:04:40.000-05:00
diff --git a/tests/unit/accordion/accordion_options.js b/tests/unit/accordion/accordion_options.js
@@ -92,10 +92,9 @@ test( "{ collapsible: true }", function() {
 		collapsible: true
 	});
 
-	// TODO: fix setting active to false
-//	ac.accordion( "option", "active", false );
-//	equal( ac.accordion( "option", "active" ), false );
-//	state( ac, 0, 0, 0 );
+	ac.accordion( "option", "active", false );
+	equal( ac.accordion( "option", "active" ), false );
+	state( ac, 0, 0, 0 );
 
 	ac.accordion( "option", "active", 1 );
 	equal( ac.accordion( "option", "active" ), 1 );
diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
@@ -285,6 +285,7 @@ $.widget( "ui.accordion", {
 			return;
 		}
 
+		this.options.active = false;
 		this.active
 			.removeClass( "ui-state-active ui-corner-top" )
 			.addClass( "ui-state-default ui-corner-all" )

Original file line number	Diff line number	Diff line change
`@@ -285,6 +285,7 @@ $.widget( "ui.accordion", {`
`285`	`285`	`return;`
`286`	`286`	`}`
`287`	`287`
	`288`	`+ this.options.active = false;`
`288`	`289`	`this.active`
`289`	`290`	`.removeClass( "ui-state-active ui-corner-top" )`
`290`	`291`	`.addClass( "ui-state-default ui-corner-all" )`