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

Add Eclipse Jetty #5461

Merged

Conversation

PeterShaggyNoble
Copy link
Member

Eclipse Jetty

Issue: Closes #5102
Alexa rank: n/a (Google Trends comparison with some of our other Eclipse icons)

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

This was going to be my morning warm-up today but the day got away from me! Went for the darker shade of orange as it's the one used in the full logo on their website.

@PeterShaggyNoble PeterShaggyNoble added the new icon Issues or pull requests for adding a new icon label Apr 15, 2021
Copy link
Member

@service-paradis service-paradis left a comment

Choose a reason for hiding this comment

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

Good job @PeterShaggyNoble! I agree with the color!
The only difference I can see is at the bottom of the j. Can you take another look?
image

@PeterShaggyNoble
Copy link
Member Author

Thanks, @service-paradis; probably Inkscape being a jerk when converting the stroke to a path again. I've fixed it up as best I could manually but you'll need to double check the whole icon again as, as soon as I started fiddling with those nodes, Inkscape changed the dimnsions of the whole icon slightly.

Copy link
Member

@service-paradis service-paradis left a comment

Choose a reason for hiding this comment

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

Spot on, thank you @PeterShaggyNoble 💯

@service-paradis service-paradis merged commit 9e7f8c9 into simple-icons:develop May 11, 2021
@PeterShaggyNoble PeterShaggyNoble deleted the add/eclipsejetty branch May 12, 2021 14:39
ericcornelissen added a commit that referenced this pull request May 16, 2021
# New Icons

- Alacritty (#5581)
- AllTrails (#5624)
- BookStack (#5141)
- CKEditor 4 (#5159)
- Data Version Control (#5674)
- Eclipse Jetty (#5461)
- Expensify (#5670)
- Grid.ai (#5423)
- JUnit5 (#5644)
- LOT Polish Airlines (#5221)
- osu! (#5347)
- Processing Foundation (#4963)
- PyTorch Lightning (#5663)
- R3 (#5551)
- RxDB (#5622)
- Simkl (#5039)
- Zerodha (#5669)

# Updated Icons

- Apple Music (#5668)
- Consul (#5650)
- del.icio.us (#5047)
- Discord (#5692)
- FARFETCH (#5539)
- Fedora (#5678)
- Feedly (#5539)
- Ferrari (#5539)
- Fido Alliance (#5539)
- FileZilla (#5539)
- Flask (#5539)
- Flickr (#5539)
- Fluentd (#5539)
- Flutter (#5539)
- G2A (#5601)
- Game Jolt (#5601)
- Garmin (#5601)
- Gatsby (#5601)
- Gentoo (#5601)
- Geocaching (#5601)
- Ghost (#5601)
- Ghostery (#5601)
- GIMP (#5601)
- GIPHY (#5601)
- GitBook (#5601)
- Gitee (#5601)
- Glassdoor (#5601)
- GNU Bash (#5601)
- Go (#5601)
- Godot Engine (#5601)
- Google (#5601)
- Google Ads (#5601)
- Google Cast (#5601)
- Google Chrome (#5601)
- Google Fit (#5601)
- Google Hangouts (#5601)
- Google Lens (#5601)
- Google News (#5601)
- Google Pay (#5601)
- Google Photos (#5601)
- Google Play (#5601)
- Google Street View (#5601)
- Gradle (#5601)
- GraphQL (#5601)
- Gravatar (#5601)
- Gridsome (#5601)
- Groupon (#5601)
- Grunt (#5601)
- gulp (#5601)
- HackerRank (#5659)
- Indeed (#5691)
- InfluxDB (#5697)
- Inkscape (#5686)
- Internet Archive (#5698)
- Qgis (#5388)
- Qi (#5388)
- Qiita (#5388)
- Qt (#5388)
- Qualtrics (#5388)
- Quantcast (#5388)
- Quasar (#5388)
- Quest (#5388)
- QuickBooks (#5388)
- Super User (#5694)
- Terraform (#5650)
- Ubiquiti (#5408)
- Ubuntu (#5408)
- Unreal Engine (#5408)
- Untangle (#5408)
- Vagrant (#5650)
- Vault (#5650)
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.

Request: Eclipse Jetty Icon
2 participants