FLUID-1123: 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

Environments

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

Attachments

• FLUID-1123.patch

Comments

• Justin Obara commented 2008-08-11T15:16:15.000-0400

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

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

  All of the above sites have the same issue.

• Jacob Farber commented 2008-08-12T12:09:45.000-0400

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

• Anastasia Cheetham commented 2008-08-12T12:18:26.000-0400

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

  I've committed the fix.

• Justin Obara commented 2008-08-13T08:55:48.000-0400

  Verified fix using:

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