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

Single finger pan events overlaps with multi touch pan #7196

Closed
pakastin opened this issue Aug 24, 2018 · 2 comments · Fixed by #8100
Closed

Single finger pan events overlaps with multi touch pan #7196

pakastin opened this issue Aug 24, 2018 · 2 comments · Fixed by #8100
Labels

Comments

@pakastin
Copy link
Contributor

There is no touch point count filtering in single finger pan. When you start multi touch pan there’s single and multi touch pan functions doing DOM updates simultaneously.

I noticed this when was fixing the multi touch center issue. Will debug more if there’s an easy fix. I’m not sure how the events should work in such case, so didn’t want to rush with a fix..

@pakastin pakastin changed the title Single finger pan overlaps with multi touch pan Single finger pan events overlaps with multi touch pan Aug 30, 2018
@pakastin
Copy link
Contributor Author

Here's a good description how it affects the UX: #6722 (comment)

@jliebrand
Copy link

sub

vakila pushed a commit that referenced this issue Sep 18, 2019
- Stop an active single-touch drag if the user adds additional fingers
  and touchZoomRotate takes over the interaction (fixes #7196) 
- Start a single-touch drag if the user removes all but one finger (fixes #6900)
- Add tests for expected single vs. multi-touch interactions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants