From: Scott González <scott.gonzalez@gmail.com>
Date: Wed, 14 Jul 2010 15:00:28 +0000 (-0400)
Subject: Accordion: Add disabled class to headers and panels to fix styling in IE. Fixes ... 
X-Git-Tag: 1.8.3~51
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=c71e63cb6907e4df994e6dfa2e52d3a2dfc19b2d;p=jquery-ui.git

Accordion: Add disabled class to headers and panels to fix styling in IE. Fixes #5332 - Accordion: Disabled accordion doesn't look disabled in IE.
---

diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
index 8699f8e12..6bda009e3 100644
--- a/ui/jquery.ui.accordion.js
+++ b/ui/jquery.ui.accordion.js
@@ -144,23 +144,21 @@ $.widget("ui.accordion", {
 
 		this.element
 			.removeClass("ui-accordion ui-widget ui-helper-reset")
-			.removeAttr("role")
-			.unbind('.accordion')
-			.removeData('accordion');
+			.removeAttr("role");
 
 		this.headers
 			.unbind(".accordion")
-			.removeClass("ui-accordion-header ui-helper-reset ui-state-default ui-corner-all ui-state-active ui-corner-top")
+			.removeClass("ui-accordion-header ui-accordion-disabled ui-helper-reset ui-state-default ui-corner-all ui-state-active ui-state-disabled ui-corner-top")
 			.removeAttr("role").removeAttr("aria-expanded").removeAttr("tabIndex");
 
 		this.headers.find("a").removeAttr("tabIndex");
 		this._destroyIcons();
-		var contents = this.headers.next().css("display", "").removeAttr("role").removeClass("ui-helper-reset ui-widget-content ui-corner-bottom ui-accordion-content ui-accordion-content-active");
+		var contents = this.headers.next().css("display", "").removeAttr("role").removeClass("ui-helper-reset ui-widget-content ui-corner-bottom ui-accordion-content ui-accordion-content-active ui-accordion-disabled ui-state-disabled");
 		if (o.autoHeight || o.fillHeight) {
 			contents.css("height", "");
 		}
 
-		return this;
+		return $.Widget.prototype.destroy.call( this );
 	},
 	
 	_setOption: function(key, value) {
@@ -175,7 +173,13 @@ $.widget("ui.accordion", {
 				this._createIcons();
 			}
 		}
-		
+		// #5332 - opacity doesn't cascade to positioned elements in IE
+		// so we need to add the disabled class to the headers and panels
+		if (key == "disabled") {
+			this.headers.add(this.headers.next())
+				[ value ? "addClass" : "removeClass"](
+					"ui-accordion-disabled ui-state-disabled" );
+		}
 	},
 
 	_keydown: function(event) {