#1378 closed bug (wontfix)
Error in Interface Accordion currentPanel option
Reported by: | thibault | Owned by: | stefan |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | interface | Version: | 1.1.2 |
Keywords: | Accordion currentPanel | Cc: | Accordion |
Blocked by: | Blocking: |
Description
The plugin Accordion doesn't use the configuration property "currentPanel". It always uses the first pannel as current pannel.
I found the error :
69 el.accordionCfg.panels 70 .hide() 71 .css('height', '1px') 72 .eq(0) 73 .css( 74 { 75 height: el.accordionCfg.panelHeight + 'px', 76 display: 'block' 77 } 78 ) 79 .end();
should be replaced by :
69 el.accordionCfg.panels 70 .hide() 71 .css('height', '1px') 72 .eq(el.accordionCfg.currentPanel) 73 .css( 74 { 75 height: el.accordionCfg.panelHeight + 'px', 76 display: 'block' 77 } 78 ) 79 .end();
Change History (2)
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Interface is no longer supported; consider switching to jQuery UI.
Note: See
TracTickets for help on using
tickets.
I had the same problem. I agree with your solution, but it isn't complete. The lines :
should be replaced by the following :
With this two modifications, the plugin works perfectly.
Hope this two fixes will be included in the next Interface release. Great plugin !
Cedric