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
Updated Jeep #5011
Updated Jeep #5011
Conversation
With regard to the "®", you are, unfortunately, right about its inclusion; I'd missed the following paragraph on page 3 of this PDF when I originally added it:
Looks like there are a few excess points that can be cleaned up here:
And, as a follow-up to that last one, I wonder if we shouldn't create and supply our own versions of "®" & "™" for contributors to use in their SVGs as it's surprising the number of source files we come across where they are riddled with imperfections and/or excess points? |
Will pick this up tomorrow - I bashed these out pretty quickly yesterday - so figured there'd be a fair bit to tidy up.
I would've imagined there are default versions provided by the relevant trademark authorities, though of course there's likely to be optimizations we can make on our end. May be worth finding the 'best one' from our existing icons, and look at applying them across the board? |
@PeterShaggyNoble give that one a go - rebuilt from FCACI using a decent SVG found in the PDF. |
LGTM 👍🏼 Let me merge it. |
# New Icons - AirAsia (#4892) - AMP (#4962) - Amul (#4955) - Apostrophe (#4977) - Apple Arcade (#4944) - AssemblyScript (#4997) - AutoHotkey (#4911) - Baremetrics (#5118) - Bitwig (#4958) - CBS (#5122) - Etihad Airways (#4957) - Grammarly (#5045) - KakaoTalk (#5016) - Kasa Smart (#5114) - KnowledgeBase (#4922) - Max (#4981) - MyAnimeList (#4718) - Namebase (#5074) - Nano (#5063) - Nx (#4860) - PlayCanvas (#5103) - Plotly (#5111) - Ram (#5014) - reveal.js (#5127) - Rezgo (#5100) - S7 Airlines (#4888) - Supabase (#4858) - SurveyMonkey (#4790) - SUSE (#4905) - Swiggy (#4875) - ts-node (#5105) - Vaadin (#5007) # Updated Icons - Chrysler (#4991) - Fiat (#5008) - Jeep (#5011) - Peugeot (#5119) - Vauxhall (#5015)
Issue: Contributes to #4578
Alexa rank: 9,857
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
Updated source to FCACI now it's been updated with Stellantis branding.
Used SVG embedded in pg11 of the PDF, ® icon seems to be pretty clear on that one.
Given Jeep use the ® symbol EVERYWHERE, I figured we should probably look to include it ourselves. Happy to maintain existing icon and just update source if others prefer that.