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

added jet icon #1574

Merged
merged 3 commits into from Aug 15, 2019
Merged

added jet icon #1574

merged 3 commits into from Aug 15, 2019

Conversation

vigonotion
Copy link
Contributor

Issue:

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

The logo consists of two colors, but yellow is the dominant one and used extensively on the website: https://www.jet-tankstellen.de/

@davidklebanoff davidklebanoff added the new icon Issues or pull requests for adding a new icon label Aug 6, 2019
Copy link
Contributor

@ericcornelissen ericcornelissen left a comment

Choose a reason for hiding this comment

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

I agree with how you made the logo monochromatic, but I see other possible approaches. I will leave it to one of the other maintainers to decide if they want to merge it like this. Everything else looks just fine, good job 👍

@birjj
Copy link
Contributor

birjj commented Aug 15, 2019

👍 This seems like the most appropriate monochromization, matching the way they present it.

Thanks for the work on this @vigonotion, it is much appreciated!

@birjj birjj merged commit 55bbd58 into simple-icons:develop Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants