Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#8998 closed bug (fixed)

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.

Change History (4)

comment:1 Changed 4 years ago by scottgonzalez

  • Status changed from new to open
  • Summary changed from Accordion create event data no in line with api documentation to 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.

comment:2 Changed 4 years ago by Scott González

  • Resolution set to fixed
  • Status changed from open to closed

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

Changeset: 10c873ab6cf2333f7e51784e84c03a0fbb975e1f

comment:3 Changed 4 years ago by scottgonzalez

  • Milestone changed from none to 1.10.1
Note: See TracTickets for help on using tickets.