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

Trans-Canada Highway shield #226

Merged
merged 1 commit into from
Mar 15, 2022
Merged

Trans-Canada Highway shield #226

merged 1 commit into from
Mar 15, 2022

Conversation

1ec5
Copy link
Member

@1ec5 1ec5 commented Mar 14, 2022

Added a route shield for the Trans-Canada Highway based on @TagaSanPedroAko’s recommendation in #218 (comment). The shield background is based very loosely on this public domain image on Wikimedia Commons, with numerous adjustments to make the shape better accommodate an inscribed number.

Portage La Prairie

Ontario inscribes road names on the TCH shield, similar to the Palisades Interstate Parkway shields in New York State. However, this PR keeps the named TCH shields blank because they only supplement the standard provincial route shields. It’s similar to how we’d treat some state scenic route shields like California’s. Unfortunately, OpenMapTiles sorts the TCH shield before the provincial route shields, probably because CA:transcanada:namedRoute sorts before CA:ON:primary.

Sault Ste. Marie

Nova Scotia puts the number on a rectangular plaque beneath the TCH shield, similar to how U.S. Bicycle Routes are signposted. For concision and consistency and expediency, the number has been inscribed onto the shield proper:

Amherst

@Pengor
Copy link
Member

Pengor commented Mar 14, 2022

Nice handling of the TCH shield, I've been thinking about this route for a while but never sat down and created any shield artwork. This was essentially the approach that I had been leaning toward though, so 👍 from me!

Copy link
Member

@ZeLonewolf ZeLonewolf left a comment

Choose a reason for hiding this comment

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

Looks good to me. Note that I am also seeing the blank shield in Quebec, so if that's not expected, we should take a look at that.

@TagaSanPedroAko
Copy link

That's also what I was looking to post here as well.

@1ec5
Copy link
Member Author

1ec5 commented Mar 15, 2022

Looks good to me. Note that I am also seeing the blank shield in Quebec, so if that's not expected, we should take a look at that.

Yes, it is expected. The TCH follows Autoroutes that have the standard red, white, and blue shield.

@1ec5 1ec5 merged commit e4b899b into main Mar 15, 2022
@1ec5 1ec5 deleted the 1ec5-ca-tch branch March 15, 2022 03:17
@1ec5 1ec5 mentioned this pull request Mar 30, 2022
@Pengor Pengor added this to the 1.0.0 milestone Apr 19, 2022
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.

4 participants