New icon: La Poste #824
Merged
New icon: La Poste #824
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We don't have guideline for that because it hasn't happen. For now it's non-issue. If it happens, it's gonna be case-by-case based (for possible example, France version will renamed to "La Poste FR" and Swiss version as "La Poste CH"). But more likely, Swiss's "La Poste" will be named as "Swiss Post", because of Wikipedia: https://en.wikipedia.org/wiki/Swiss_Post |
|
Should all be fixed now |
krisu5
approved these changes
Jan 29, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
URL of the site: https://www.laposte.fr/
Proof of the Aegis-compatible 2FA: 2FA Doc
Logo / icon info: Direct URL of the logo file (taken from this page)
Background color info:
Took the most dark yellow from the gradient of the logo here
Background color is from the logo (
#e6af32).How should post services from different countries with the same name be handled? For example, French and Swiss post services are both named "La Poste" but with different logos and are entirely separate entities.
For now I added the country between parentheses after the name.