fredsa / gwt-dnd

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

iOS: Drag with drag sensitivity > 0 starts at wrong location #171

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Does the issue occur in "quirks mode", "standards mode" or both? If you
don't know, does your HTML page contains a DOCTYPE declaration?
N/A

What version of GWT are you using? 1.4.60? 2.0.4? Other?
 * 2.5.0

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

What operating system(s) are you using? Windows? Linux? Mac?
 * Issue only applies to iPad (iOS). Can be demonstrated on iPad simulator.

Does the issue occur in web mode, development mode (formerly "hosted
mode"), both or don't know?
 * Web mode (development mode not available for iPad)

What browser(s) do you use? Chrome, Firefox, IE, Safari, other?
 * Mobile Safari. Also includes iOS app embedded browser (UIWebView), which is nearly (but not completely) identical to Safari.
What is the browser version (if you know) from Help->About?
 * Information not available for Mobile Safari. However, iOS version includes 6.1 and 5.1.

What steps will reproduce the problem? Please attach sample code if you
can.
 * Reproducible using your demo (http://allen-sauer.com/com.allen_sauer.gwt.dnd.demo.DragDropDemo/DragDropDemo.html#BinExample). 

1. Set Drag Sensitivity to non-zero.
2. Press and hold on widget.
3. Move to just the sensitivity amount.

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

Expected: Drag starts at current widget location.
Observed: Drag starts substantially offset, possibly outside of constrained 
boundary. (With the demo, if the widget is released, it will be misplaced).

Do you have a workaround?
* No workaround possible.

Please provide any additional information below.

Original issue reported on code.google.com by gmcdor...@skura.com on 14 May 2013 at 8:04

GoogleCodeExporter commented 9 years ago
Additional comment: It appears that removing the following code (lines 404-405 
of MouseDragHandler) fixes the issue. Unclear what other side effects may be, 
however.

          x += location.getLeft();
          y += location.getTop();

Original comment by gmcdor...@skura.com on 14 May 2013 at 9:05

GoogleCodeExporter commented 9 years ago
Patch that makes these changes attached.

Original comment by grmcdor...@gmail.com on 27 Jun 2013 at 3:36

Attachments: