-
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
Zooming with scroll wheel during pan ease resets pan action. #6668
Comments
Probably the same root cause, but wanted to mention this on the issue as well. When initiating an animation programatically, for example with map.easeTo(), mouse wheel zooming does not interrupt the animation, which causes weird UX. Demonstrated here (start mouse wheel/touch pad zooming during animation): https://codepen.io/anon/pen/ZRevPa This may be more of an upgrade blocker for applications than the original issue, especially for applications heavily dependent on animations. |
This seems to be a regression not present in <0.45.0. |
It may be related to #5743 |
This is no longer reproducible in v0.47 (although I don't remember the exact PR that fixed it). |
I'm on v0.47 and I still have the issue. |
I might have misunderstood the issue — I can no longer reproduce the jerking motion when zooming during panning inertia, it just stops the inertia and zooms. Zoom and inertia are not cumulative, like the original issue requests, but I think they shouldn't be. |
This issue is not present in 0.32.0, so I believe it's a regression caused by pan/zoom ease handling.
mapbox-gl-js version:
0.45.0
browser:
Confirmed in Chrome, Firefox. Expect bug is not browser specific.
Steps to Trigger Behavior
Link to Demonstration
Can be replicated in any demos which allow panning and zooming.
Expected Behavior
Zoom and pan animations should be cumulative
Actual Behavior
The pan animation will stop while the zoom is in progress, and then jerk to the new eased position
The text was updated successfully, but these errors were encountered: