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 Blockchain.com #4818

Merged
merged 3 commits into from Feb 13, 2021
Merged

Add Blockchain.com #4818

merged 3 commits into from Feb 13, 2021

Conversation

loganmarchione
Copy link
Contributor

Issue: N/A
Alexa rank: 1,582

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

There is a SVG in their homepage. Wasn't sure on the color though, please check and let me know.
blockchain-dot-com

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 19, 2021
@service-paradis
Copy link
Member

I found a press kit containing their logos at https://www.blockchain.com/fr/press

There is also a monochrome version of their logo at https://exchange.blockchain.com/.

The icon is looking good. 🎉

I dont know about the color. They use #0c6cf2 a lot on their website (every CTA, a lot of links, ...). Maybe another @simple-icons/maintainers can help with the color?

@PeterShaggyNoble
Copy link
Member

I would say #121d33 for the colour.

@loganmarchione
Copy link
Contributor Author

I updated the color to 121d33
blockchain-dot-com

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

Thanks, @loganmarchione. Based on @service-paradis' review of the icon, this should be good to go now. But given the lack of clarity on the colour choice we'll need approval from at least one of the other @simple-icons/maintainers before merging.

@adamrusted
Copy link
Member

So from what I can see, the below three hex codes are in use in the icon:
#85B5F8 #3D89F5 #1656B9

With #121d33 and #0c6cf2 being used heavily throughout the website.

Having a quick glance at the site - I'd have gone with #0C6CF2 - it looks (by my shoddy eyes at least) to be roughly the midpoint between the two darker blues of the logo (#2A70D7 is the calculated midpoint of those two, which is another option!), and is used extensively throughout the website and the app. Unfortunately they seem to pick a vague hex that's near enough to 'their colour' for every social post, so I was unable to glean any info from there.

@adamrusted
Copy link
Member

Any of the other @simple-icons/maintainers want an input on the colour of this one? Suggestions in #4818 (comment), #4818 (comment), and #4818 (comment). Be good if we can reach a consensus before @loganmarchione makes any more changes 😅

@adamrusted adamrusted added the in discussion There is an ongoing discussion that should be finished before we can continue label Jan 25, 2021
@ericcornelissen
Copy link
Contributor

Regarding the color: judging by the website I would also have opted for lighter blue than #121d33, such as #0c6cf2 or #3D89F5. Given that the website seems to be a bit inconsistent on the website ("Unfortunately they seem to pick a vague hex that's near enough to 'their colour' for every social post, so I was unable to glean any info from there.") I would choose the color in the SVG as it is more consistently used and provided on the press page.

In any case, I agree that this can be merged as is. The color can be changed later after the discussion has concluded.

Regarding the PR: I would suggest including the press page (https://www.blockchain.com/en/press) as the "guidelines" for this brand.

@loganmarchione
Copy link
Contributor Author

@ericcornelissen - I left the color as-is but added the brand guidelines.

Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

SVG matches up perfectly, thanks @loganmarchione! 🎉

@simple-icons/maintainers, the colour may still need some further discussion, but I'm merging as-is for now to clear our backlog.

@adamrusted adamrusted merged commit 51b5c10 into simple-icons:develop Feb 13, 2021
@github-actions github-actions bot removed the in discussion There is an ongoing discussion that should be finished before we can continue label Feb 13, 2021
@loganmarchione loganmarchione deleted the blockchaindotcom branch February 13, 2021 18:00
ericcornelissen added a commit that referenced this pull request Feb 14, 2021
# New Icons

- AOL (#4954)
- Blockchain.com (#4818)
- China Eastern Airlines (#4827)
- Cloudera (#4812)
- D-Wave Systems (#4825)
- Major League Hacking (#4915)
- Oxygen (#4839)
- Pug (#4714)
- Textpattern (#4966)
- Windows Terminal (#4956)

# Updated Icons

- Android Auto (#4400)
- Google Street View (#4683)
- Google Tag Manager (#4586)
- LinkedIn (#4985, #4939)
- New Japan Pro-Wrestling (#4930)
- openSUSE (#4904)
- Roundcube (#4495)
- Svelte (#4907)
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

5 participants