-
Notifications
You must be signed in to change notification settings - Fork 819
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
Clean up low-zoom levels #2933
Clean up low-zoom levels #2933
Conversation
* Render roads from z6 instead of z5 * Render national parks from z8 instead of z7 * Render railways from z8 instead of z7 This is in line with the discussion on use cases and USECASES.md. This contributes towards gravitystorm#2925 and gravitystorm#2688.
@kocio-pl As you are also working on the low-level stuff, what do you think of this PR? |
Sure, I plan to do it soon, just trying to collect all the pieces in my head... 😄 |
I like the idea of this changes. |
Preview of this (and other pending changes) is available at http://bl.ocks.org/math1985/raw/af7a602c222dbf1ff1a2c0d84ed755b7/ |
Preview helped me a lot, thanks for it! I approve all the changes:
|
Thanks, this is a great help! Btw, what is the best way to report issues? This page send to be not working on mobiles - attempting to zoom out/in results in moving to NaN/NaN coordinates. |
Thanks for reviewing! |
This is in line with the discussion on use cases and USECASES.md.
This contributes towards #2925 and #2688.