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

Same line wrap of amenitys for all zoom levels #2788

Merged
merged 1 commit into from
Sep 9, 2017

Conversation

sommerluk
Copy link
Collaborator

As proposed by @math1985 at #2730 (comment) here is a PR that unifies the line wrap for amenity points over all zoom levels. I think that indeed it makes sense, because amenity points are rather small (at difference to other features like cities or lakes that have larger line wrap at higher zoom levels because they are much bigger), and anyway the difference is not big compared to the current code. This PR will make zoom-in/zoom-out less surprising, because you get always the same line wrap for amenities.

@kocio-pl kocio-pl merged commit 5eae5fc into gravitystorm:master Sep 9, 2017
@matthijsmelissen
Copy link
Collaborator

Thanks, I should have done it done it like this from the beginning!

@matthijsmelissen
Copy link
Collaborator

I created also an issue at Mapnik, requesting to be able to specify wrap in em natively: mapnik/mapnik#3763

@sommerluk sommerluk deleted the amenitywrap01 branch September 29, 2017 22:15
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.

3 participants