diff options
| author | John Ahlroos <john@vaadin.com> | 2013-08-20 13:30:47 +0300 |
|---|---|---|
| committer | Vaadin Code Review <review@vaadin.com> | 2013-08-20 12:47:50 +0000 |
| commit | 7220ca8758e44ae84ae5c9ef0623dc9585f6d171 (patch) | |
| tree | 0e7fca68a7ac22012c29a8662af9a0ad984fb065 /client | |
| parent | b5a212a8c87a7370c385aeec22ea98a58fc7b051 (diff) | |
| download | vaadin-framework-7220ca8758e44ae84ae5c9ef0623dc9585f6d171.tar.gz vaadin-framework-7220ca8758e44ae84ae5c9ef0623dc9585f6d171.zip | |
Fixed DragAndDropWrapper using wrong drop target in IE8 #12406
VDragAndDropManager was assuming that the target element will always be
inside the cloned "drag image" element while dragging. This assumption
is false since the "drag image" can be 0x0px or transparent effectivly
disabling dragging.
Since Testbench 2 is also very flaky in using the Vaadin locators with
the drag/drop commands I replaced the locators with shorter locators
using a debug id to make the test more readable and stable.
Change-Id: I2cc9683d11e982521e74418c74dd3e81ee617ac5
Diffstat (limited to 'client')
| -rw-r--r-- | client/src/com/vaadin/client/ui/dd/VDragAndDropManager.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/client/src/com/vaadin/client/ui/dd/VDragAndDropManager.java b/client/src/com/vaadin/client/ui/dd/VDragAndDropManager.java index dd838fdeff..b4cf008a38 100644 --- a/client/src/com/vaadin/client/ui/dd/VDragAndDropManager.java +++ b/client/src/com/vaadin/client/ui/dd/VDragAndDropManager.java @@ -93,9 +93,7 @@ public class VDragAndDropManager { targetElement = targetNode.getParentElement(); } - if (Util.isTouchEvent(nativeEvent) - || (dragElement != null && dragElement - .isOrHasChild(targetElement))) { + if (Util.isTouchEvent(nativeEvent) || dragElement != null) { // to detect the "real" target, hide dragelement temporary and // use elementFromPoint String display = dragElement.getStyle().getDisplay(); |