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

zoom maps with mouse wheel #20

Closed
JocelynDelalande opened this issue Jun 12, 2015 · 4 comments
Closed

zoom maps with mouse wheel #20

JocelynDelalande opened this issue Jun 12, 2015 · 4 comments

Comments

@JocelynDelalande
Copy link
Owner

No description provided.

@JocelynDelalande JocelynDelalande changed the title scroll with mouse wheel zoom maps with mouse wheel Jun 12, 2015
@opi
Copy link
Collaborator

opi commented Jun 13, 2015

Not really kind of zooming map with mouse wheel, because if we enable this feature, we cannot scroll down the page while beeing over the map. On desktop it could be ok, but not on a mobile/tablet experience.

That's really my feeling, and changing this behavior is no more complicated than setting scrollWheelZoom to true in https://github.com/JocelynDelalande/wifi-with-me/blob/master/assets/map.js#L29

@kheops2713
Copy link
Collaborator

Agreed with @opi : I find it annoying when a page stops scrolling because my pointer suddenly fell onto the map that has wheelzoom.

@JocelynDelalande
Copy link
Owner Author

Agreed with @opi : I find it annoying when a page stops scrolling because my pointer suddenly fell onto the map that has wheelzoom.

At least on the big map, which you don't have to scroll ()

On desktop it could be ok, but not on a mobile/tablet experience.

I was told that by a coleague of mine the same.

Something I don't get : are not scroll and zoom (pinch) different events on mobile ?

@bikepunk
Copy link
Collaborator

bikepunk commented May 6, 2016

@bikepunk bikepunk closed this as completed May 6, 2016
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

No branches or pull requests

4 participants