FLUID-1484: All moved elements will flash after performing a DnD operation

Metadata

Source
FLUID-1484
Type
Bug
Priority
Major
Status
Closed
Resolution
Fixed
Assignee
Justin Obara
Reporter
Justin Obara
Created
2008-09-08T12:18:44.000-0400
Updated
2008-09-09T09:14:41.000-0400
Versions
  1. 0.5beta1
Fixed Versions
  1. 0.5
Component
  1. Image Reorderer
  2. Reorderer

Description

All moved elements will flash after performing a DnD operation.

Steps to reproduce:

1) Open the lightbox demo from the daily build site:
http://build.fluidproject.org/fluid/sample-code/reorderer/generic-lightbox/generic-lightbox.html

2) Move an image to a new location

3) Notice that the images which are reordererd briefly flash on the screen after the DnD operation is complete.

Environments

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

Comments

  • Antranig Basman commented 2008-09-08T17:49:58.000-0400

    I have tried returning to JQuery manipulation rather than raw DOM operations, but unfortunately the flashing is not dispelled. It is hard to explain, since it was not present in the 0.5 beta. My guess at the moment is that this is some form of CSS application mistiming issue - and may be related to the other kinds of corruption we can now see, such as 1482 (I have seen some kinds of CSS corruption of this kind, but not this particular issue)

  • Antranig Basman commented 2008-09-08T18:12:29.000-0400

    I think I have found the cause of this issue, and apparently quite a number of other awfulnesses that I saw.

    This line,
    //jQuery(thatReorderer.activeItem).removeClass(options.styles.selected);
    in Reorderer.js drag stop, has now been commented out, which makes everything seemingly much more humane. This is odd, since this line was carried over from the drag stop operation we had before. In general I don't really understand the CSS transition rules we have right now, which are responsible for problems like FLUID-676. In any case I think this particular change is a step in the right direction.

  • Justin Obara commented 2008-09-09T09:14:41.000-0400

    Verified fix using:

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