Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Items can only be moved when pressing hard on touch device (Microsoft Surface) #54

Closed
dominic-simplan opened this issue Feb 6, 2018 · 3 comments
Labels
intended behavior This describes correct behavior -- it is working as designed

Comments

@dominic-simplan
Copy link

dominic-simplan commented Feb 6, 2018

In the latest version (1.8.11) items can only be moved via touch gesture on a Microsoft Surface Tablet when I press quite hard on the item (both palette and modelling area).

Until 1.8.2 and between 1.8.8 and 1.8.10 (e.g. https://gojs.net/1.8.10/samples/flowchart.html), I just needed to touch an item slightly to be able to move it around.
In 1.8.11 (e.g. https://gojs.net/1.8.11/samples/flowchart.html) and from 1.8.3 to 1.8.7, I need to press quite hard else the item gets only selected and the diagram is scrolled.

This seems to be related to #52: In versions where the drag into the diagram area works correctly, a hard press is needed.

@simonsarris
Copy link
Collaborator

Thanks, we'll take a look.

@simonsarris
Copy link
Collaborator

I do see what you mean. This is actually intentional.

For touch events, we have a delay in the dragging tool on starting it in order to allow easier panning. If you do not like this simply set the DraggingTool.delay to 0:

    myDiagram = $(go.Diagram, "myDiagramDiv",
                  {
                    "draggingTool.delay": 0,
                     ...

Some background:

The versions you are getting "immediate" behavior in are not using PointerEvents in IE, instead they're using IE's simulated Mouse events. These versions have some issues, the most obvious one being dragging between Diagrams (you can see that you can't drop from the Palette in 1.8.10's flowchart).

Chrome and Firefox don't emit simulated mouse events, instead they emit touch events, so they already work this way, with the delay.

@dominic-simplan
Copy link
Author

I see, so it wasn't about pressing hard after all 😃 .
I have changed the delay to 20 and now it's what I've expected. Thanks also for the background information!

@WalterNorthwoods WalterNorthwoods added the intended behavior This describes correct behavior -- it is working as designed label Jun 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
intended behavior This describes correct behavior -- it is working as designed
Projects
None yet
Development

No branches or pull requests

3 participants