Navigation Menu

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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Apache Spark icon (#1329) #1820

Merged
merged 3 commits into from Oct 24, 2019
Merged

Conversation

phatbhoy67
Copy link
Contributor

**Issue: #1329 **

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

Extracted broken "star" from from official wordmark logo. The wikipedia version is converted from the official .eps version.

Manually edited to complete "star" by following the "V" made from the "k" in "Spark" to match the favicon.

Predominant brand colour used on Apache Spark website is blue, chosen hex value #2FA4E7 as it appears to be the colour most used (changed my mind 馃槅).

Built on GitPod, working as expected.

@birjj
Copy link
Contributor

birjj commented Oct 24, 2019

I think that we would prefer using the orange from the logo as their color. Their website is predominantly blue, but I think the brand itself is mostly connected to the orange of the star in their wordmark. Thoughts @ericcornelissen?

@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Oct 24, 2019
@ericcornelissen
Copy link
Contributor

Its a tough choice but I personally associate orange with Spark more than blue. It's an easy change later on so I think it is fine if we go with orange for now 馃憤

@phatbhoy67
Copy link
Contributor Author

Hey @ericcornelissen @birjolaxew I've updated the hex value to #E25A1C which is extracted from the official .eps version of the logo.

@birjj
Copy link
Contributor

birjj commented Oct 24, 2019

馃憤 Looks good! Thanks a lot for your work on the vector and this PR!

@birjj birjj merged commit c384383 into simple-icons:develop Oct 24, 2019
@phatbhoy67 phatbhoy67 deleted the apache-spark branch October 24, 2019 22:41
@birjj birjj mentioned this pull request Nov 3, 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

3 participants