Reorderer broken, throws errors in IE

Description

When loading a page containing reorderable elements in IE, the following errors are thrown.

Generic Lightbox example:
---------------------------------------
Line: 2
Char: 1608
Error: Object doesn't support this property or method
Code: 0

Reorderable jQuery Tabs Demo:
---------------------------------------------
Line: 2
Char: 8913
Error: Expected Identifier, string or number
Code: 0

Line: 48
Char: 5
Error: Object expected
Code: 0

Sortable Vertical List:
--------------------------------
Line: 17
Char: 5
Error: Object doesn't support this property or method
Code: 0

Uportal demo:

Line: 2
Char: 1608
Error: Object doesn't support this property or method
Code: 0

Line: 2
Char: 584
Error: Object doesn't support this property or method
Code: 0

Environment

IE 6, IE7 (Win XP) IE7 (Win Vista)

Attachments

1

Activity

Show:

Justin Obara August 13, 2008 at 12:55 PM

Verified fix using:

IE6, IE7 (Win XP)
IE7 (Win Vista)

Anastasia Cheetham August 12, 2008 at 4:18 PM

I've tested this patch, and it does fix the error - good catch, Jacob!

I've committed the fix.

Jacob Farber August 12, 2008 at 4:09 PM

Extra comma removed - IE dies when a list contains a trailing comma, whereas FF wont.

Justin Obara August 11, 2008 at 7:16 PM

Updated debugging info
====================

File: Reorderer.js
Line: 483
Error: Expected identifier, string or number

All of the above sites have the same issue.

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created August 11, 2008 at 1:23 PM
Updated August 13, 2008 at 12:55 PM
Resolved August 12, 2008 at 4:28 PM