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 Semaphore CI icon (#1511) #1842

Merged
merged 3 commits into from Oct 29, 2019
Merged

Conversation

phatbhoy67
Copy link
Contributor

**Issue: #1511 **

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 logo from full official mono logo .svg.

Brand colour appears to be green, #19a974 is used in the logo and a similar colour #0DCC6B is used on the website.

Hex value #19A974 is chosen from the Semaphore CI colour logo .svg.

Built on GitPod, working as expected.

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

I agree with the chosen brand color.
It looks ugly when the brand color is applied on the website anyways so I guess they have to adjust it.

Anyways, I approve of this PR.
Though, I'll let the other @simple-icons/maintainers to see their thoughts and merge this in.

@birjj
Copy link
Contributor

birjj commented Oct 29, 2019

👍 I definitely agree with taking the green from the logo too. Good work @phatbhoy67!

@birjj birjj merged commit 5512877 into simple-icons:develop Oct 29, 2019
@phatbhoy67 phatbhoy67 deleted the semaphoreci branch October 29, 2019 17:19
@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