fredsa / gwt-dnd

Library providing easy to use mouse or touch based drag-and-drop capabilities to GWT
43 stars 41 forks source link

PickupDragController with VerticalPanel : Widget drags vetoed with VetoDragException in a DragHandler are often returned to an incorrect position in the panel #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of GWT are you using? 1.3.3? 1.4.60? Other?

Self-built from public svn repository @5880.

What version of the gwt-dnd jar file or library file are you using?

Self-built  from public svn repository @933.

What operating system(s) are you using? Windows? Linux? Mac?

Mac OS X 10.5.7

Does the issue occur in web mode, hosted mode, both or don't know?

Both

What browser(s) do you use? Firefox, IE, Safari, other?

Firefox 3.5.1 (seen in other browsers too)

What is the browser version (if you know) from Help->About?

Firefox 3.5.1

What steps will reproduce the problem? Please attach sample code if you
can.

1. Create a VerticalPanel, and a PickupDragController. 
2. Add a new DragHandler to the drag controller that always throws
VetoDragException on onPreviewDragEnd.
3. Create a new IndexedDropController associated with the vertical panel
and register it with the drag controller. 
4. Add some Label widgets to the vertical panel, having made each one
draggable with the drag controller.

What is the expected output? What do you see instead?

Run the app. Notice that when you drag a label to a position after the
original position in the list, the drag veto event returns the label to the
correct position in the list. 

Notice that when you drag a label to a position before its original
position in the list, the drag veto event causes the label to be placed the
incorrect position, seemingly always one position too high.

Do you have a workaround?

No

Original issue reported on code.google.com by jimblack...@gmail.com on 5 Aug 2009 at 7:44

Attachments:

GoogleCodeExporter commented 9 years ago
Fixed in r965

Original comment by fredsa on 11 Aug 2009 at 6:52