library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Use .children() instead of .find() to manipulate accordion icons. Fixes #4645 - Accordion should only manipulate Icons that are children of ui-accordion-header

Browse Source
This commit is contained in:
jzaefferer 2010-07-15 09:51:20 +02:00
parent 56aecac811
commit 06f721b74f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ui/jquery.ui.accordion.js vendored
View File

@ -129,7 +129,7 @@ $.widget("ui.accordion", {
var o = this.options; var o = this.options;
if (o.icons) { if (o.icons) {
$("<span/>").addClass("ui-icon " + o.icons.header).prependTo(this.headers); $("<span/>").addClass("ui-icon " + o.icons.header).prependTo(this.headers);
this.active.find(".ui-icon").toggleClass(o.icons.header).toggleClass(o.icons.headerSelected); this.active.children(".ui-icon").toggleClass(o.icons.header).toggleClass(o.icons.headerSelected);
this.element.addClass("ui-accordion-icons"); this.element.addClass("ui-accordion-icons");
} }
}, },
@ -279,7 +279,7 @@ $.widget("ui.accordion", {
if (!o.collapsible) if (!o.collapsible)
return; return;
this.active.removeClass("ui-state-active ui-corner-top").addClass("ui-state-default ui-corner-all") this.active.removeClass("ui-state-active ui-corner-top").addClass("ui-state-default ui-corner-all")
.find(".ui-icon").removeClass(o.icons.headerSelected).addClass(o.icons.header); .children(".ui-icon").removeClass(o.icons.headerSelected).addClass(o.icons.header);
this.active.next().addClass('ui-accordion-content-active'); this.active.next().addClass('ui-accordion-content-active');
var toHide = this.active.next(), var toHide = this.active.next(),
data = { data = {
@ -309,10 +309,10 @@ $.widget("ui.accordion", {
// switch classes // switch classes
this.active.removeClass("ui-state-active ui-corner-top").addClass("ui-state-default ui-corner-all") this.active.removeClass("ui-state-active ui-corner-top").addClass("ui-state-default ui-corner-all")
.find(".ui-icon").removeClass(o.icons.headerSelected).addClass(o.icons.header); .children(".ui-icon").removeClass(o.icons.headerSelected).addClass(o.icons.header);
if (!clickedIsActive) { if (!clickedIsActive) {
clicked.removeClass("ui-state-default ui-corner-all").addClass("ui-state-active ui-corner-top") clicked.removeClass("ui-state-default ui-corner-all").addClass("ui-state-active ui-corner-top")
.find(".ui-icon").removeClass(o.icons.header).addClass(o.icons.headerSelected); .children(".ui-icon").removeClass(o.icons.header).addClass(o.icons.headerSelected);
clicked.next().addClass('ui-accordion-content-active'); clicked.next().addClass('ui-accordion-content-active');
} }