Bug Tracker

#1378 closed bug (wontfix)

Opened July 09, 2007 11:54PM UTC

Closed March 31, 2008 01:26AM UTC

Last modified August 07, 2009 07:05AM UTC

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();
Attachments (0)
Change History (2)

Changed August 30, 2007 11:47AM UTC by semoi comment:1

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

Changed March 31, 2008 01:26AM UTC by scott.gonzal comment:2

resolution: → wontfix
status: new → closed

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