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

fix: Avoid map-panning on mobile using two fingers navigation #2340

Conversation

fttriquet
Copy link

If scrollWheelZoom option is false we deactivate leaflet dragging options only on mobile

@fttriquet fttriquet force-pushed the fix-948-map-panning-with-two-fingers branch from ce96615 to 13cc938 Compare December 5, 2024 13:04
If scrollWheelZoom option is false we deactivate leaflet dragging options only on mobile
@fttriquet fttriquet force-pushed the fix-948-map-panning-with-two-fingers branch from 13cc938 to acbdfda Compare December 5, 2024 13:11
@yohanboniface
Copy link
Member

Thanks!

I'm not sure to understand: while in mobile and scrollWheelZoom if off, this will disabled dragging the map with one finger, but it will still be possible with two fingers ?

@yohanboniface yohanboniface merged commit b7649a5 into umap-project:master Dec 5, 2024
@yohanboniface
Copy link
Member

I guess answer is no (for two fingers scrolling), but let's go like this and see how it goes!

yohanboniface added a commit that referenced this pull request Dec 6, 2024
… is false

Changing map options at this stage is too late.

cf #2340
yohanboniface added a commit that referenced this pull request Dec 7, 2024
… is false (#2344)

Changing map options at this stage is too late.

cf #2340
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants