Accordion: Update active option when programmatically collapsing the …

…accordion.
jquery · Feb 3, 2011 · 2e17770 · 2e17770
1 parent d099e3a
commit 2e17770
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
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" )