#4645 closed bug (fixed)

Opened June 30, 2009 08:12AM UTC

Closed July 15, 2010 07:51AM UTC

Last modified November 19, 2010 06:26PM UTC

Accordion should only manipulate Icons that are children of ui-accordion-header

Reported by: ajpiano Owned by:
Priority: major Milestone: 1.8.3
Component: ui.accordion Version: 1.7.2
Keywords: Cc:
Blocked by: Blocking:
Description

Whether or not it is 100% wise to start having other icons in the accordion header, it should be possible to put icons up there without interference. Hence, this line in _clickHandler is too general:

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);
Attachments (0)
Change History (4)

Changed July 29, 2009 04:09PM UTC by scottgonzalez comment:1

milestone: 1.next → 1.8

Changed July 15, 2010 07:51AM UTC by jzaefferer comment:2

resolution: → fixed
status: new → closed

Fixed in 06f721b

Changed August 04, 2010 01:19AM UTC by scottgonzalez comment:3

milestone: 1.9 → 1.8.3

Changed November 19, 2010 06:26PM UTC by jzaefferer comment:4

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

Changeset: 06f721b74f94272baf5da96133e961f5600f5a90