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

Do not order low zoom roads by layer #2820

Merged
merged 1 commit into from
Sep 11, 2017

Conversation

matthijsmelissen
Copy link
Collaborator

On low zoom levels (up to z9) it is not important which road crosses
on top of the other. Instead, it is important to always show important
roads on top, so lower classes roads do not hide motorways.

On low zoom levels (up to z9) it is not important which road crosses
on top of the other. Instead, it is important to always show important
roads on top, so lower classes roads do not hide motorways.
@matthijsmelissen
Copy link
Collaborator Author

matthijsmelissen commented Sep 10, 2017

Rendering differences are typically minimal, except in particular circumstances with lower classed roads on top of tunnels of higher classed roads, such as here.

Before:
screen shot 2017-09-11 at 00 35 53

After:
screen shot 2017-09-11 at 00 34 53

@sommerluk
Copy link
Collaborator

I support this. On this zoom levels, z_order is indeed more meaningfull than layer.

@kocio-pl
Copy link
Collaborator

I don't see the problem on the attached renderings (probably scaling would be good here to show the problem fully), but I agree with layering being not adequate on lower zoom.

@matthijsmelissen
Copy link
Collaborator Author

I don't see the problem on the attached renderings

Best to look at the location of the marker in the linked map, and then to look again in the attached renderings how that part is rendered.

@matthijsmelissen
Copy link
Collaborator Author

Scaled images:
screen shot 2017-09-11 at 01 47 42
screen shot 2017-09-11 at 01 47 51

@pnorman pnorman merged commit 9d47f84 into gravitystorm:master Sep 11, 2017
@matthijsmelissen matthijsmelissen deleted the roads-low-zoom-order branch September 14, 2017 23:08
imagico added a commit to imagico/osm-carto-alternative-colors that referenced this pull request Sep 27, 2017
@matthijsmelissen matthijsmelissen restored the roads-low-zoom-order branch November 2, 2017 22:09
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.

4 participants