Opened 9 years ago
Closed 9 years ago
#10126 closed bug (notabug)
Accordion: Appending new items with all panels closed via 'active: none' opens added panel
Reported by: | jcwren | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | none |
Component: | ui.accordion | Version: | 1.10.4 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
If a set of panels is created with 'active: none' set, and a new panel is appended and the accordion refreshed, the new panel is opened. If another panel is already opened, then the new panel is not opened. The 'refresh' method appears to not be honoring the configured 'active' setting.
Demonstration -- http://jsfiddle.net/jcwren/MGKMH/
Change History (6)
comment:1 Changed 9 years ago by
Status: | new → open |
---|---|
Summary: | Appending new items with all panels closed via 'active: none' opens added panel → Accordion: Appending new items with all panels closed via 'active: none' opens added panel |
comment:2 follow-up: 3 Changed 9 years ago by
I spoke with @scott_gonsalez in #jqueryui-dev, and the behavior described in this ticket seems to be the expected behavior.
comment:3 Changed 9 years ago by
Replying to jperrault:
I spoke with @scott_gonsalez in #jqueryui-dev, and the behavior described in this ticket seems to be the expected behavior.
Well, that's... disappointing. And seemingly insane. It's easier to programmatically open it than have to find it and close it.
comment:4 follow-up: 5 Changed 9 years ago by
Scott also mentioned that 'active: none' is an invalid value, it should be 'active: false'. With the false setting, your first scenario results in the header remaining closed.
comment:5 Changed 9 years ago by
Replying to jperrault:
Scott also mentioned that 'active: none' is an invalid value, it should be 'active: false'. With the false setting, your first scenario results in the header remaining closed.
I guess that's the danger in trusting an accepted answer from StackOverlow.
comment:6 Changed 9 years ago by
Resolution: | → notabug |
---|---|
Status: | open → closed |
Here is a fiddle showing it working properly: http://jsfiddle.net/MGKMH/3/
Confirmed against master: http://jsfiddle.net/tj_vantoll/MGKMH/2/