Error in Interface Accordion currentPanel option

[thibault]

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();

[semoi]

I had the same problem. I agree with your solution, but it isn't complete. The lines :

154	.eq(0)
155	.addClass(el.accordionCfg.activeClass)
156	.end();

should be replaced by the following :

154	.eq(el.accordionCfg.currentPanel)
155	.addClass(el.accordionCfg.activeClass)
156	.end();

With this two modifications, the plugin works perfectly.

Hope this two fixes will be included in the next Interface release. Great plugin !

Cedric

[scott.gonzalez]

Interface is no longer supported; consider switching to jQuery UI.