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

Restore admin-text rendering for admin_level 3 to 5 #4126

Merged
merged 1 commit into from
Apr 26, 2020

Conversation

jeisenbe
Copy link
Collaborator

Fixes a mistake in PR #4100
Changes proposed in this pull request:

  • Add back admin_level=3, =4, =5 admin-text rendering along border lines from z11

The rendering of admin-text labels for these zoom levels was accidentally removed in PR#4100

Test rendering with links to the example places:

Before
admin-text-fix-before

After
admin-text-fix-after

@jeisenbe jeisenbe added the bug label Apr 25, 2020
@jeisenbe jeisenbe requested a review from pnorman April 25, 2020 18:09
@jeisenbe
Copy link
Collaborator Author

Sorry about this mistake, next time I'll remember to look at all the text features more carefully.

@jeisenbe jeisenbe merged commit b65ae26 into gravitystorm:master Apr 26, 2020
@jeisenbe jeisenbe deleted the admin-text branch April 26, 2020 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants