From: Scott González <scott.gonzalez@gmail.com>
Date: Tue, 13 Jul 2010 13:10:43 +0000 (-0400)
Subject: Accordion: Don't handle hover/focus when disabled. Fixes #5330 - Accordion headers... 
X-Git-Tag: 1.8.3~56
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=551bf6e1e7844dc3ea4ca62a3bf0e7cb4c18744b;p=jquery-ui.git

Accordion: Don't handle hover/focus when disabled. Fixes #5330 - Accordion headers still show rollover when disabled.
---

diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
index 07fad3130..c8025cbbf 100644
--- a/ui/jquery.ui.accordion.js
+++ b/ui/jquery.ui.accordion.js
@@ -42,11 +42,24 @@ $.widget("ui.accordion", {
 		// in lack of child-selectors in CSS we need to mark top-LIs in a UL-accordion for some IE-fix
 		this.element.children("li").addClass("ui-accordion-li-fix");
 
-		this.headers = this.element.find(o.header).addClass("ui-accordion-header ui-helper-reset ui-state-default ui-corner-all")
-			.bind("mouseenter.accordion", function(){ $(this).addClass('ui-state-hover'); })
-			.bind("mouseleave.accordion", function(){ $(this).removeClass('ui-state-hover'); })
-			.bind("focus.accordion", function(){ $(this).addClass('ui-state-focus'); })
-			.bind("blur.accordion", function(){ $(this).removeClass('ui-state-focus'); });
+		this.headers = this.element.find(o.header)
+			.addClass("ui-accordion-header ui-helper-reset ui-state-default ui-corner-all")
+			.bind("mouseenter.accordion", function() {
+				if (o.disabled) { return; }
+				$(this).addClass('ui-state-hover');
+			})
+			.bind("mouseleave.accordion", function() {
+				if (o.disabled) { return; }
+				$(this).removeClass('ui-state-hover');
+			})
+			.bind("focus.accordion", function() {
+				if (o.disabled) { return; }
+				$(this).addClass('ui-state-focus');
+			})
+			.bind("blur.accordion", function() {
+				if (o.disabled) { return; }
+				$(this).removeClass('ui-state-focus');
+			});
 
 		this.headers
 			.next()