Bug Tracker

Opened 13 years ago

Closed 12 years ago

Last modified 11 years ago

#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 13 years ago by 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

comment:2 Changed 12 years ago by scott.gonzal

Resolution: wontfix
Status: newclosed

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

Note: See TracTickets for help on using tickets.