From: jzaefferer <joern.zaefferer@gmail.com>
Date: Thu, 15 Jul 2010 07:51:20 +0000 (+0200)
Subject: Use .children() instead of .find() to manipulate accordion icons. Fixes #4645 - Accor... 
X-Git-Tag: 1.8.3~40
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=06f721b74f94272baf5da96133e961f5600f5a90;p=jquery-ui.git

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

diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
index 5d31a9fec..fd6331026 100644
--- a/ui/jquery.ui.accordion.js
+++ b/ui/jquery.ui.accordion.js
@@ -129,7 +129,7 @@ $.widget("ui.accordion", {
 		var o = this.options;
 		if (o.icons) {
 			$("<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");
 		}
 	},
@@ -279,7 +279,7 @@ $.widget("ui.accordion", {
 			if (!o.collapsible)
 				return;
 			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');
 			var toHide = this.active.next(),
 				data = {
@@ -309,10 +309,10 @@ $.widget("ui.accordion", {
 
 		// switch classes
 		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) {
 			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');
 		}