-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
touchmove event changed in v1.10, not fired when map is moved #9676
Comments
Thanks @ansis this is exactly what I am talking about. But you Switches the
titles of your links. The first points to the 1.10 version and the second
to 1.9 ;)
Andreas
Karim Naaji <notifications@github.com> schrieb am Mo. 11. Mai 2020 um 23:52:
… @fcandi <https://github.com/fcandi> Thanks for the report, I created a
comparison example to identify whether this is a regression.
- v1.9.0 <https://jsfiddle.net/kanaa/crL210mt/17/embedded/result>
- v1.10.0 <https://jsfiddle.net/kanaa/crL210mt/16/embedded/result>
This part of the code seems covered by unit tests, but I can confirm this
is a regression in v1.10.0 as I just tested and compared on iOS.
cc @ansis <https://github.com/ansis>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#9676 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABFPZSGOYZZYXRXUXSUXLDRRBXTLANCNFSM4M5WQ24Q>
.
|
@fcandi Will be available as part of v1.10.1, thanks for the report! |
karimnaaji
pushed a commit
that referenced
this issue
May 13, 2020
karimnaaji
pushed a commit
that referenced
this issue
May 14, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
mapbox-gl-js version: 1.10.0
browser: Chrome
Steps to Trigger Behavior
touchmove event is not fired when moving the map on touch devices
Expected Behavior
Touchmove event should be fired whenever the map moves on touch devices, as it is in all versions prior to 1.10.0
Actual Behavior
The touchmove event is only fired when a draggable marker is moved on touch devices
The text was updated successfully, but these errors were encountered: