Accordion does not function in IE7/WinXP

[mkornblum]

Ref: Ticket 4331 (​http://dev.jqueryui.com/ticket/4331)

This problem still occurs in IE7 using JQuery 1.3.2 and JQuery UI 1.7.1, even after adding:

.ui-helper-reset { border: 0; }

To my site's CSS.

test case here: ​http://arts.cornish.edu/test/accordion.html (not the most minimal!)

Specifying "animated: false" when initializing the accordion fixes the issue, but is not as pretty.

Test here without animation here: ​http://cornish.edu/drums/day-one

[mkornblum]

Also should mention: works fine (with animation) in IE6; it is only IE7 that is still broken.

[wewals]

The linked test case works for me under IE7 on Windows XP. Can anyone reproduce?

[smadep]

could reproduce this on IE7 7.0.5730.13 after first area was opened no other area could be opened

[DiRN]

If this is the error that I'm experiencing, the problem is a divide by 0 error.

jQuery UI 1.7.2 (ui.core.js), line 3435 (the line number will vary by version)

percentDone = (settings.now - settings.start) / (settings.end - settings.start);

should be something along the lines of

percentDone = (settings.end - settings.start) != 0 ? (settings.now - settings.start) / (settings.end - settings.start) : 0;

With the correct code, percentDone ends up with a value of NaN. When using NaN, the following lines result in an error in IE7.

options.toShow[0].style[settings.prop] =
	(percentDone * showProps[settings.prop].value) + showProps[settings.prop].unit;

[abaumhau]

Replying to DiRN:

If this is the error that I'm experiencing, the problem is a divide by 0 error.

I can confirm that this is a divide by zero error in percentDone. The problem exists on IE8 when not in compatibility mode. The settings.{now|start|end} are all zero, and percentDone = NaN. This causes an invalid option on line 486 in jquery.ui.accordion.js. The fix listed does work. Thanks much.

Andy

[Scott González]

Fixed in r3703.