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

scroll-to-zoom for 'map' data view? #114

Closed
JoeGermuska opened this issue Jun 1, 2015 · 3 comments
Closed

scroll-to-zoom for 'map' data view? #114

JoeGermuska opened this issue Jun 1, 2015 · 3 comments

Comments

@JoeGermuska
Copy link
Member

A UserVoice comment on a 'map' data view page said:

Hi! I can't zoom in using my mouse; that makes it difficult to look at a particular census tract without accidentally clicking on another census tract.

My response:

We chose to disable scroll-wheel zooming because there are other pages on Census Reporter where the likelihood of accidentally zooming the map when you were actually trying to move down the page was higher and we didn't want users to be frustrated.

Note that there are zoom controls in the upper right corner of the map.

For the map/data view from which you sent this feedback, admittedly there is no other scrolling, so allowing scroll-wheel-zooming might be worth turning on. We'll consider whether that inconsistency is minor and possibly enable it.

Thanks for the input.

should we enable scroll-to-zoom for this page alone?

@ryanpitts
Copy link
Member

I vote no. Not just because of the inconsistency it would introduce -- I also think the UX drawbacks of accidental scroll-wheel-zooming are much greater than the benefits of intentional use, especially when there's an alternate zoom control on the page.

@iandees
Copy link
Member

iandees commented Jun 1, 2015

I was going to vote no as well, but I think that it might be more important to allow scroll-zooming when the map consumes the whole page like in this example. It definitely doesn't make sense to allow scroll-zooming when the map is just the header of the profile page, for example.

That being said, I think the zoom controls solve the problem pretty well.

@JoeGermuska
Copy link
Member Author

the nos have it

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

No branches or pull requests

3 participants