Accordion: Remove deprecated height options. Fixes #5868 - Accordion: Remove clearStyle option. Fixes #5872 - Accordion: Remove autoHeight and fillSpace options.

2 files changed, 1 insertions, 78 deletions

diff --git a/tests/unit/accordion/accordion_common_deprecated.js b/tests/unit/accordion/accordion_common_deprecated.js
index 22d5e1289..23324a0ad 100644
--- a/tests/unit/accordion/accordion_common_deprecated.js
+++ b/tests/unit/accordion/accordion_common_deprecated.js
@@ -3,14 +3,11 @@ TestHelpers.commonWidgetTests( "accordion", {
 		active: 0,
 		animate: null,
 		animated: "slide",
-		autoHeight: true,
-		clearStyle: false,
 		collapsible: false,
 		disabled: false,
 		event: "click",
-		fillSpace: false,
 		header: "> li > :first-child,> :not(li):even",
-		heightStyle: null,
+		heightStyle: "auto",
 		icons: {
 			"activeHeader": null,
 			"header": "ui-icon-triangle-1-e",
diff --git a/tests/unit/accordion/accordion_deprecated.js b/tests/unit/accordion/accordion_deprecated.js
index 2068fef95..b8f499542 100644
--- a/tests/unit/accordion/accordion_deprecated.js
+++ b/tests/unit/accordion/accordion_deprecated.js
@@ -104,80 +104,6 @@ test( "{ active: jQuery Object }", function() {
 
 
 
-
-module( "accordion (deprecated) - height options", setupTeardown() );
-
-test( "{ autoHeight: true }, default", function() {
-	expect( 3 );
-	equalHeight( $( "#navigation" ).accordion({ autoHeight: true }), 105 );
-});
-
-test( "{ autoHeight: false }", function() {
-	expect( 3 );
-	var element = $( "#navigation" ).accordion({ autoHeight: false }),
-		sizes = [];
-	element.find( ".ui-accordion-content" ).each(function() {
-		sizes.push( $(this).height() );
-	});
-	ok( sizes[0] >= 70 && sizes[0] <= 105, "was " + sizes[0] );
-	ok( sizes[1] >= 98 && sizes[1] <= 126, "was " + sizes[1] );
-	ok( sizes[2] >= 42 && sizes[2] <= 54, "was " + sizes[2] );
-});
-
-test( "{ fillSpace: true }", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element, 455 );
-});
-
-test( "{ fillSapce: true } with sibling", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30
-		})
-		.prependTo( "#navigationWrapper" );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element , 355 );
-});
-
-test( "{ fillSpace: true } with multiple siblings", function() {
-	expect( 3 );
-	$( "#navigationWrapper" ).height( 500 );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30
-		})
-		.prependTo( "#navigationWrapper" );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 50,
-			marginTop: 20,
-			marginBottom: 30,
-			position: "absolute"
-		})
-		.prependTo( "#navigationWrapper" );
-	$( "<p>Lorem Ipsum</p>" )
-		.css({
-			height: 25,
-			marginTop: 10,
-			marginBottom: 15
-		})
-		.prependTo( "#navigationWrapper" );
-	var element = $( "#navigation" ).accordion({ fillSpace: true });
-	equalHeight( element, 305 );
-});
-
-
-
-
-
 module( "accordion (deprecated) - icons", setupTeardown() );
 
 test( "icons, headerSelected", function() {