FLUID-572: Reorderer: calls to document.writeln() while moving an item can cause the page to break.

Metadata

Source
FLUID-572
Type
Sub-task
Priority
Blocker
Status
Closed
Resolution
Fixed
Assignee
Justin Obara
Reporter
Michelle D'Souza
Created
2008-05-09T14:15:25.000-0400
Updated
2008-09-08T10:40:05.000-0400
Versions
  1. 0.1
  2. 0.3
  3. 0.4beta1
  4. 0.4
Fixed Versions
  1. 0.5
Component
  1. Reorderer

Description

This issue came up when the Reorderer was added to aTutor for reordering sections of the UI. The 'menu' section had script in it that did a 'document.writeln'. This code was executed at the same time as the item was being moved (using jQuery.after). The result was an empty page that didn't finish loading.

dev-iteration43

Attachments

Comments

  • Michelle D'Souza commented 2008-05-09T14:27:20.000-0400

    There is a work around for this issue: using jQuery.append instead of document.writeln worked as expected.

  • Michelle D'Souza commented 2008-05-12T10:49:18.000-0400

    I'm uploading jquery-tabs.html which contains a small modification to the jquery reorderable tabs example from the Fluid samples that shows the issue.

  • Michelle D'Souza commented 2008-05-16T15:53:57.000-0400

    There is now a manual test that shows the problem clearly.
    http://build.fluidproject.org/fluid/tests/fluid-tests/manual/jQueryDnD/writeln.html

    To recreate the issue, try to drag 'Item 1'.

    I've opened a ticket in jQuery's Trac: http://dev.jquery.com/ticket/2887

  • Anastasia Cheetham commented 2008-07-03T19:51:05.000-0400

    Update: In upgrading to jQuery 1.2.6 and UI 1.5.1, I checked this jQuery ticket: It is still open, and the bug still seems to be present.

  • Colin Clark commented 2008-07-31T16:18:13.000-0400

    infusion-05-wishlist

  • Colin Clark commented 2008-07-31T17:47:18.000-0400

    Apparently calls to document.writeln() is standard practice for almost everything Google has written.

  • Antranig Basman commented 2008-09-07T18:03:52.000-0400

    Drop Manager, Dom Permutation work merged back into trunk at revision 5476