Bug Tracker

Ticket #1378 (closed bug: wontfix)

Opened 1 year ago

Last modified 5 months ago

Error in Interface Accordion currentPanel option

Reported by: thibault Assigned to: stefan
Type: bug Priority: major
Milestone: Component: interface
Version: 1.1.2 Keywords: Accordion currentPanel
Cc: Accordion Needs: Patch

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

Change History

Changed 1 year 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

Changed 5 months ago by scott.gonzalez

  • status changed from new to closed
  • resolution set to wontfix

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

Note: See TracTickets for help on using tickets.