#8998 closed bug (fixed)

Opened January 22, 2013 02:00PM UTC

Closed January 22, 2013 02:19PM UTC

Last modified January 22, 2013 03:13PM UTC

Accordion: Create event uses content instead of panel.

Reported by: Shikyo Owned by:
Priority: minor Milestone: 1.10.1
Component: ui.accordion Version: 1.10.0
Keywords: Cc:
Blocked by: Blocking:
Description

http://api.jqueryui.com/accordion/#event-create

The api states the 'ui' object has the properties header and panel but it's actually header and content. I believe the api docs are correct and the code is wrong.

Fiddle: http://jsfiddle.net/JB9Pr/

look at the console output.

Attachments (0)
Change History (4)

Changed January 22, 2013 02:07PM UTC by scottgonzalez comment:1

status: new → open
summary: Accordion create event data no in line with api documentation → Accordion: Create event uses content instead of panel.

I'm not sure how to set the milestone for this. It's a breaking change, but it's also a fairly big mistake.

Changed January 22, 2013 02:19PM UTC by Scott González comment:2

resolution: → fixed
status: open → closed

Accordion: Add panel in addition to content for create event. Fixes #8998 - Accordion: Create event uses content instead of panel.

Changeset: 10c873ab6cf2333f7e51784e84c03a0fbb975e1f

Changed January 22, 2013 02:19PM UTC by scottgonzalez comment:3

milestone: none → 1.10.1

Changed January 22, 2013 03:13PM UTC by scottgonzalez comment:4

Added to the 1.10 upgrade guide: https://github.com/jquery/jqueryui.com/commit/26d9724795739ce5dceb6163cf3366d69a573d36