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 Zoom icon #1859

Merged
merged 2 commits into from Oct 30, 2019
Merged

Add Zoom icon #1859

merged 2 commits into from Oct 30, 2019

Conversation

bonez0607
Copy link
Contributor

@bonez0607 bonez0607 commented Oct 29, 2019

Issue: #1625

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

  1. Hex value: #5090CD - based on brand guidelines
  2. Downloaded wordmark straight from site - simplified down to a single path.

optimized

fixed zoom logo
@foo-dogsquared
Copy link
Contributor

This PR is good for merging.
Thank you @bonez0607 for taking the time to contribute and following the contributing guidelines!

Though, I don't see the chosen brand color (#5090CD) mentioned in the guidelines.
It's already provided from the official palette (at page 11 in the brand guide PDF) that #2D8CFF is the main brand color.

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

Happy to help! And apologies - I got the hex value straight from the logo via Illustrator. But I clearly see the specified color in the PDF. I went ahead and updated the hex color.

@foo-dogsquared foo-dogsquared merged commit c8eb390 into simple-icons:develop Oct 30, 2019
@bonez0607 bonez0607 deleted the zoom branch October 31, 2019 16:17
@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

2 participants