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 Stencyl (#2124) #2184

Merged
merged 3 commits into from Dec 19, 2019
Merged

Add Stencyl (#2124) #2184

merged 3 commits into from Dec 19, 2019

Conversation

runxel
Copy link
Member

@runxel runxel commented Dec 19, 2019

Issue: Closes #2124

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

Logo (the famous cog c) is from the press kit.
The color #8E1C04 is the calculated midpoint of the colored logo version's gradient from #BD2408 to #5F1400.

@runxel runxel added the new icon Issues or pull requests for adding a new icon label Dec 19, 2019
@foo-dogsquared
Copy link
Contributor

As per the build job log, there is an extra attribute (id) on the <path>.

Furthermore, based on quick inspection, there is a slight curving on some of the spikes of the cog with your PR compared to the source vector.

Your PR is the red while the source vector is in blue.
Both of them have half the alpha value so the combined color is basically the common area.

2019-12-20-02-54-58

2019-12-20-02-55-23

@runxel
Copy link
Member Author

runxel commented Dec 19, 2019

Hey @foo-dogsquared, thanks for your review, much appreciating!
You're right, btw! In the original source the spikes look a bit different – they have additional handles which are sometimes slightly bent. I removed those handles as this is clearly an unintended behaviour.
Good spotting, nonetheless! :)

@foo-dogsquared
Copy link
Contributor

Looks good to me!
Nice job for the quick update, @runxel .

@foo-dogsquared foo-dogsquared merged commit 427547b into simple-icons:develop Dec 19, 2019
birjj added a commit that referenced this pull request Dec 24, 2019
New icons:

- Dart (#2106)
- Audi (#2065)
- Pi-hole (#2084)
- Fijutsu (#2076)
- Uber Eats (#2078)
- RocketMQ (#2112)
- ZeroMQ (#2115)
- Cordova (#2027)
- XMPP (#2085)
- V (#2056)
- Cryengine (#2079)
- Mercedes (#2114)
- Corona Renderer (#2129)
- Fujifilm (#2131)
- Parse.ly (#2138)
- Nutanix (#2098)
- Publons (#2130)
- Webmin (#2132)
- X.Org (#2122)
- Notist (#2123)
- Directus (#2142)
- DLNA (#2080)
- Ember.js (#2143)
- Waze (#2146)
- StackPath (#2145)
- Jinja (#2144)
- Floatplane (#2140)
- Box (#2159)
- WhiteSource (#2121)
- Vodafone (#2096)
- WebAuthn (#2104)
- Xamarin (#2176)
- Dunked (#2178)
- Stencyl (#2184)
- Zend (#2171)
- Zend Framework (#2171)
- OpenSSL (#2043)
- Crehana (#2045)
- Lufthansa (#2113)
- iOS (#2147)
- Broadcom (#2163)
- Volkswagen (#2150)
- GNOME (#2177)
- cPanel (#2173)
- Coinbase (#1845)
- Sketch (#2208)
- ICON (#2165)

Updated icons

- Node.js (#2134)
- Meetup (#2062)
- GitLab (#2192)
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.

Stencyl
2 participants