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

Add latin text highway shields for Japan #307

Merged
merged 4 commits into from
May 7, 2022
Merged

Add latin text highway shields for Japan #307

merged 4 commits into from
May 7, 2022

Conversation

ZeLonewolf
Copy link
Member

This PR is a subset of #219 which provides comprehensive shield rendering for Japan highways. Support is added for those highway shields which are consistent with international norms and prefixed with the JP country code. This allows us to add the shield to the map that we all agree should be rendered, while deferring the routes with more controversial network values.

@ZeLonewolf ZeLonewolf requested a review from 1ec5 May 6, 2022 23:56
Copy link
Member

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As in #219 (comment), I agree that we should move forward with these uncontroversial networks and resolve the urban expressways separately.

@@ -0,0 +1,3 @@
<svg width="20" height="20" viewBox="0 0 5.292 5.292" xmlns="http://www.w3.org/2000/svg">
<path d="m0.60707 0.13229c-0.26207 0-0.47478 0.21271-0.47478 0.47478v2.0388c0 1.3875 1.1261 2.5135 2.5135 2.5135 1.3875 0 2.5135-1.1261 2.5135-2.5135v-2.0388c0-0.26207-0.2127-0.47478-0.47478-0.47478h-2.0388z" fill="#006747" stroke="#fff" stroke-linejoin="round" stroke-width=".265"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This SVG is unused for now. Let’s leave it for #219.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants