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

Updated Opel #5012

Merged
merged 3 commits into from
Feb 20, 2021
Merged

Updated Opel #5012

merged 3 commits into from
Feb 20, 2021

Conversation

adamrusted
Copy link
Member

opel

Issue: Contributes to #4578
Alexa rank: 128,956

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

Update to source, and a slight change to the SVG based on new source.

@github-actions github-actions bot added the update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both label Feb 13, 2021
@adamrusted adamrusted mentioned this pull request Feb 13, 2021
14 tasks
@PeterShaggyNoble
Copy link
Member

Could you take another look at this one, @adamrusted? I'm noticing some hairline differences, mainly in the "bolt", which may simply be a precision issue. There are also a couple of excess points top & bottom of the inside of the circle and on the very right of the "bolt".

@adamrusted
Copy link
Member Author

@PeterShaggyNoble had to bump it to 4 decimals, but should be good now.

@fbernhart
Copy link
Contributor

The icon looks good to me 👍🏼 But how about the color?

I haven't found any brand guidelines, but given that our current color (#F7D900) was based on #2361, where we've just used the yellow found on their website, we should change it.

The two websites linked below are now using #F7FF14 instead.

https://de-media.opel.com
https://www.opel.de

@adamrusted
Copy link
Member Author

@fbernhart happy to update colour. Give me a minute and should be ready for review 👌

@fbernhart
Copy link
Contributor

@adamrusted Great work! 🚗

@fbernhart fbernhart merged commit d4ee1ac into simple-icons:develop Feb 20, 2021
@adamrusted adamrusted deleted the Update/Opel branch February 20, 2021 12:32
ericcornelissen added a commit that referenced this pull request Feb 21, 2021
# New Icons

- Aerospike (#4994)
- Alibaba.com (#4953)
- ANA (#4821)
- Apache Hive (#4882)
- Aparat (#5046)
- British Airways (#4908)
- ChatBot (#4877)
- Codeberg (#4757)
- Cycling '74 (#4982)
- Databricks (#5037)
- Dogecoin (#4762)
- Ethiopian Airlines (#4909)
- Handshake (#5071)
- Handshake (Protocol) (#5072)
- HelpDesk (#4895)
- Hibernate (#4873)
- Hive (Blockchain) (#3008)
- IEEE (#4989)
- Juniper Networks (#4894)
- Linux Containers (#3155)
- Lion Air (#4959)
- Miro (#5077)
- MobX (#4986)
- MobX-State-Tree (#4984)
- MODX (#4893)
- Naver (#4987)
- OnlyFans (#5059)
- OpenFaaS (#4980)
- Podman (#4774)
- Prisma (#5052)
- Qatar Airways (#4869)
- Resurrection Remix OS (#4756)
- Rider (#5041)
- Sefaria (#4988)
- semantic-release (#4995)
- sharp (#4998)
- Shotcut (#4879)
- teratail (#4897)
- WebdriverIO (#5042)

# Updated Icons

- Citroën (#5005)
- DS Automobiles (#5006)
- GNU (#4945)
- GNU social (#4945)
- Google Play (#4505)
- IconJar (#3909)
- IntelliJ IDEA (#5041)
- Intercom (#5041)
- InVision (#5048)
- Kotlin (#5041)
- Maserati (#5009)
- Opel (#5012)
- Peugeot (#5013)
- PhpStorm (#5041)
- PyCharm (#5041)
- TeamCity (#5041)
- Textpattern (#5065)
- WebStorm (#5041)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants