FLUID-2339: Toggling Table of Contents on, throws an error: using IE

Metadata

Source
FLUID-2339
Type
Bug
Priority
Blocker
Status
Closed
Resolution
Fixed
Assignee
Justin Obara
Reporter
Justin Obara
Created
2009-03-12T12:43:51.000-0400
Updated
2009-03-25T11:49:25.000-0400
Versions
N/A
Fixed Versions
  1. 1.0
Component
  1. UI Options

Description

Toggling Table of Contents off, throws an error

Steps to reproduce:

1) Open the UI Options example from the daily build site
http://build.fluidproject.org/fluid/fluid-components/html/templates/UIOptions.html

2) Open the "Easier to See" accordion panel

3) Change the radio button selection for "Table of Contents" from "No" to "Yes"

4) Save and apply the changes

Notice that an error is thrown

Error: Invalid Argument
Line: 957
File: jquery-1.3.2.js

Environments

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

Comments

  • Justin Obara commented 2009-03-12T12:44:26.000-0400

    Bug Parade 1.0 release

  • Michelle D'Souza commented 2009-03-20T10:50:52.000-0400

    This seems to happen when adding the anchors to the document.

  • Antranig Basman commented 2009-03-24T02:52:15.000-0400

    Fixed at revision 6817.
    It seems that IE is extremely fussy, when dealing with multiple documents perhaps within iframes, that new markup nodes are created using the correct owner document. I spent a while writing raw DOM equivalents, but eventually discovered that jQuery supports syntax like the following:

    var node = $("<div></div>", container[0].ownerDocument);

    This approach was suggested by the following report: http://dev.jquery.com/ticket/978
    where it is reported that the "ownerDocument" property is extremely portable, with reference to QuirksMode at http://www.quirksmode.org/dom/w3c_core.html#t119

  • Jacob Farber commented 2009-03-25T11:28:22.000-0400

    Looks good.

  • Justin Obara commented 2009-03-25T11:38:17.000-0400

    Verified fix using:

    IE6 (Win XP)
    IE7 (Win Vista)

  • Anastasia Cheetham commented 2009-03-25T11:49:25.000-0400

    This fix looks fine to me.