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 Ripple icon #1784

Merged
merged 1 commit into from Oct 19, 2019
Merged

Add Ripple icon #1784

merged 1 commit into from Oct 19, 2019

Conversation

phatbhoy67
Copy link
Contributor

**Partially closes Issue: #1086 **

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

Isolated the Ripple icon from the ripple-logo.svg.

Hex value #0085C0 is from the brand guidelines .pdf "solid blue".

Built on GitPod, working as expected.

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

birjj commented Oct 19, 2019

👍 Clean work @phatbhoy67! Thanks for the contribution!

@birjj birjj merged commit 00bba9d into simple-icons:develop Oct 19, 2019
@phatbhoy67 phatbhoy67 deleted the ripple branch October 20, 2019 09:25
ericcornelissen added a commit that referenced this pull request Oct 23, 2019
# New icons

- LibreOffice (#1763)
- IBM (#1768)
- Apache Airflow (#1766)
- Cisco (#1769)
- Epson (#1770)
- HP (#1759)
- CentOS (#1764)
- Minetest (#1751)
- Genius (#1777)
- Lenovo (#1772)
- iDEAL (#1778)
- Plesk (#1781)
- Bancontact (#1779)
- Krita (#1775)
- XRP (#1783)
- Ripple (#1784)
- SVG (#1790)
- Nec (#1796)
- Nim (#1776)
- Habr (#1804)
- Symantec (#1797)
- Analogue (#1802)
- AdGuard (#1810)
- Strapi (#1787)
- GIMP (#1813)
- uBlock Origin (#1815)

# Updated icons

- Apple Podcasts (#1803 and #1805)
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