Ticket #4100 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

IE js error when autoHeight is false

Reported by: kam99 Owned by:
Priority: major Milestone: 1.7
Component: ui.accordion Version: 1.6rc6
Keywords: IE accordion autoHeight js error Cc: alexey.koryakov@…
Blocking: Blocked by:

Description

Tested against IE6, IE7.

Error: Invalid argument. Accordion doesn't work if autoHeight is set to false.

$("#accordion").accordion({ clearStyle: true, autoHeight: false});

<div id="accordion">
	<a href="#" class="accordionHeader">Header 1</a>
	<div class="accordionContent">
		Container 1 <br />
		Container 1
	</div>
	<a href="#" class="accordionHeader">Header 2</a>
	<div class="accordionContent" >
		Container 2
	</div>
</div>

Quick fix I made locally: before (n is NaN, cause error)

j.toShow[0].style[m.prop]=n+"px"

after

j.toShow[0].style[m.prop]=(isNaN(n)?0:n)+"px"

Change History

comment:1 Changed 6 years ago by rdworth

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from 1.next to 1.7

This issue has been fixed in the trunk.

comment:2 Changed 6 years ago by harlley

The issue continues when I use an accordion inside another accordion.

Note: See TracTickets for help on using tickets.