Fluid Infusion

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

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 0.1, 0.3, 0.4beta1, 0.4
  • Fix Version/s: 0.5
  • Component/s: Reorderer
  • Labels:
    None

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

Activity

Hide
Michelle D'Souza added a comment -
There is a work around for this issue: using jQuery.append instead of document.writeln worked as expected.
Show
Michelle D'Souza added a comment - There is a work around for this issue: using jQuery.append instead of document.writeln worked as expected.
Hide
Michelle D'Souza added a comment -
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.
Show
Michelle D'Souza added a comment - 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.
Hide
Michelle D'Souza added a comment - - edited
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
Show
Michelle D'Souza added a comment - - edited 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
Hide
Anastasia Cheetham added a comment -
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.
Show
Anastasia Cheetham added a comment - 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.
Hide
Colin Clark added a comment -
infusion-05-wishlist
Show
Colin Clark added a comment - infusion-05-wishlist
Hide
Colin Clark added a comment -
Apparently calls to document.writeln() is standard practice for almost everything Google has written.
Show
Colin Clark added a comment - Apparently calls to document.writeln() is standard practice for almost everything Google has written.
Hide
Antranig Basman added a comment -
Drop Manager, Dom Permutation work merged back into trunk at revision 5476
Show
Antranig Basman added a comment - Drop Manager, Dom Permutation work merged back into trunk at revision 5476

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
3d
Original Estimate - 3d
Remaining:
3d
Remaining Estimate - 3d
Logged:
Not Specified
Time Spent - Not Specified