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 Metro de Madrid #3153
Add Metro de Madrid #3153
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, @mondeja - SVG looks perfect to me and I agree with the use of colour which is heavily used on their website.
The only change I'd suggest is in the naming; we recently add Métro de Paris under the original, French name and, where it makes sense, I think that's something we should continue to do for all languages using the Latin alphabet so could you update the name of this to "Metro de Madrid", please?
@PeterShaggyNoble It's done. |
Thanks, @mondeja. Just need to capitalise "Madrid" in the JSON and change the |
Thanks for your work on this one, @mondeja 👍 |
# New Icons - GeeksforGeeks (#3072) - Jasmine (#3152) - KTM (#2997) - Metro de la Ciudad de México (#3156) - Metro de Madrid (#3153) - N26 (#3133) - Prime Video (#3025) - Scratch (#2872) - Shenzhen Metro (#3158) - Tokyo Metro (#3157) - TuneIn (#3115) - VirusTotal (#3182) # Updated Icons - Bitwarden (#3150) - SpaceX (#3159)
Issue: #2822
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description