From 551bf6e1e7844dc3ea4ca62a3bf0e7cb4c18744b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Tue, 13 Jul 2010 09:10:43 -0400 Subject: [PATCH] Accordion: Don't handle hover/focus when disabled. Fixes #5330 - Accordion headers still show rollover when disabled. --- ui/jquery.ui.accordion.js | 23 ++++++++++++++++++----- 1 file changed, 18 insertions(+), 5 deletions(-) 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() -- 2.39.5