From 03d6a64695d86e103eb5c8464aeb231e651c4256 Mon Sep 17 00:00:00 2001
From: Richard Worth <rdworth@gmail.com>
Date: Fri, 21 Nov 2008 04:58:53 +0000
Subject: [PATCH] accordion: formatting changes - moved some things around for
 consistency. _init and destroy at the top. Sorted defaults alphabetically.

---
 ui/ui.accordion.js | 36 +++++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/ui/ui.accordion.js b/ui/ui.accordion.js
index 5cc6a5cc7..d30bd9217 100644
--- a/ui/ui.accordion.js
+++ b/ui/ui.accordion.js
@@ -13,6 +13,7 @@
 (function($) {
 
 $.widget("ui.accordion", {
+
 	_init: function() {
 		var options = this.options;
 
@@ -96,6 +97,19 @@ $.widget("ui.accordion", {
 		}
 	},
 
+	destroy: function() {
+		this.options.headers.parent().andSelf().removeClass(this.options.selectedClass);
+		this.options.headers.prev(".ui-accordion-left").remove();
+		this.options.headers.children(".ui-accordion-right").remove();
+		this.options.headers.next().css("display", "");
+		if ( this.options.fillSpace || this.options.autoHeight ) {
+			this.options.headers.next().css("height", "");
+		}
+		$.removeData(this.element[0], "accordion");
+
+		this.element.removeClass("ui-accordion").unbind(".accordion");
+	},
+
 	_keydown: function(event) {
 		if (this.options.disabled || event.altKey || event.ctrlKey)
 			return;
@@ -135,20 +149,8 @@ $.widget("ui.accordion", {
 		clickHandler.call(this.element[0], {
 			target: findActive( this.options.headers, index )[0]
 		});
-	},
-
-	destroy: function() {
-		this.options.headers.parent().andSelf().removeClass(this.options.selectedClass);
-		this.options.headers.prev(".ui-accordion-left").remove();
-		this.options.headers.children(".ui-accordion-right").remove();
-		this.options.headers.next().css("display", "");
-		if ( this.options.fillSpace || this.options.autoHeight ) {
-			this.options.headers.next().css("height", "");
-		}
-		$.removeData(this.element[0], "accordion");
-
-		this.element.removeClass("ui-accordion").unbind(".accordion");
 	}
+
 });
 
 function scopeCallback(callback, scope) {
@@ -329,16 +331,16 @@ function findActive(headers, selector) {
 $.extend($.ui.accordion, {
 	version: "@VERSION",
 	defaults: {
-		selectedClass: "selected",
+		autoHeight: true,
 		alwaysOpen: true,
 		animated: 'slide',
 		event: "click",
 		header: "a",
-		autoHeight: true,
-		running: 0,
 		navigationFilter: function() {
 			return this.href.toLowerCase() == location.href.toLowerCase();
-		}
+		},
+		running: 0,
+		selectedClass: "selected"
 	},
 	animations: {
 		slide: function(options, additions) {
-- 
2.39.5