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

Polish: Map zoom levels #2

Open
cjdd3b opened this issue Mar 1, 2018 · 2 comments
Open

Polish: Map zoom levels #2

cjdd3b opened this issue Mar 1, 2018 · 2 comments

Comments

@cjdd3b
Copy link

cjdd3b commented Mar 1, 2018

Some of the outstate map zoom levels are pretty useless. For example:

screen shot 2018-03-01 at 10 26 30 am

screen shot 2018-03-01 at 10 26 43 am

Not a high priority, but a couple potential solutions:

  • If a lat/lon is outside some bounding box established around the metro area, use a wider default zoom (9? 10?)
  • Manually set zoom levels on each map (a pain, but I can do the legwork if you have a tech solution to integrate it)
  • Set a slightly wider zoom level for all maps

Suppose it comes down to what these maps are for. To me they just offer a bit of geographic context, which is different in the metro vs. outstate, which is why the first solution might be the easiest.

But like I said, low priority.

@zzolo
Copy link
Contributor

zzolo commented Mar 2, 2018

A good idea. Will try to get to it, but feel free to give it a while.

I think the best place for it is to just add in the logic into the mapImage helper.
https://github.com/striblab/2018-summer-camp-guide/blob/master/components/camp.html#L137-L149

@zzolo
Copy link
Contributor

zzolo commented Mar 2, 2018

Do note that the map zoom levels may have been adjusted since you last looked at, so they may be ok where they are.

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

2 participants