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 Zhihu icon (#1889) #1959

Merged
merged 4 commits into from Nov 23, 2019
Merged

Conversation

phatbhoy67
Copy link
Contributor

@phatbhoy67 phatbhoy67 commented Nov 20, 2019

Issue: #1889

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

Source is .svg from the website using svg grabber.

Hex value #0084ff is from the logo and also used on the website.

Built on GitPod, working as expected.

@foo-dogsquared
Copy link
Contributor

This looks good to me.
It does agree with the conversation from the related issue.

@foo-dogsquared foo-dogsquared merged commit 351236a into simple-icons:develop Nov 23, 2019
@phatbhoy67 phatbhoy67 deleted the zhihu branch November 23, 2019 19:16
@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Nov 25, 2019
birjj added a commit that referenced this pull request Nov 25, 2019
New icons:

- AlliedModders (#1893)
- Bitrise (#1829)
- Khan Academy (#1884)
- C# (#1807)
- Mapbox (#1897)
- Monster (#1892)
- MicroStrategy (#1899)
- Pastebin (#1887)
- strongSwan (#1906)
- ABB RobotStudio (#1886)
- Asciidoctor (#1886)
- Source Engine (#1898)
- Valve (#1907)
- Steamworks (#1908)
- Apache NetBeans IDE (#1885)
- Beats by Dre (#1924)
- MDN Web Docs (#1925)
- Algolia (#1929)
- CodeChef (#1921)
- Minds (#1917)
- Duolingo ( #1922, 5388f8d)
- ManageIQ (#1926)
- ReactOS (#1941)
- IPFS (#1949)
- PureScript (#1910)
- PyUp (#1954)
- QEMU (#1957)
- Prometheus (#1964)
- Timescale (#1962)
- awesomeWM (#1966)
- Anchor (#1960)
- Zhihu (#1959)
- Google Search Console (#1947)
- InfluxDB (#1965)
- Apache OpenOffice (#1976)

Updated icons:

- Houzz (#1928)
- Vue.js (#1979)
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