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 Trustpilot icon (#2004) #2008

Merged
merged 3 commits into from Nov 30, 2019
Merged

Conversation

runxel
Copy link
Member

@runxel runxel commented Nov 28, 2019

Issue:
Closes #2004

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

I stenciled the dark part of star out. This gives imho a good monochrome representation of the brand. The color #00b67a is eyedropped from the lighter green part of the star.

@runxel runxel changed the title Trustpilot icon Add Trustpilot icon (#2004) Nov 28, 2019
@phatbhoy67
Copy link
Contributor

Hey @runxel thanks for taking on this issue 👍

Hex value chosen is confirmed as the light green in the logo. 🎨

Your manual stencil version is a very good effort but just a little bit off.

Can you please update the PR to use the mono version of the "star" available on (https://www.trustpilot.com/) - you can isolate the "star" from the 5 star bar which you can either grab with svg grabber or download from (https://cdn.trustpilot.net/brand-assets/4.1.0/stars/stars-5.svg).

@phatbhoy67 phatbhoy67 added changes requested new icon Issues or pull requests for adding a new icon labels Nov 29, 2019
@runxel
Copy link
Member Author

runxel commented Nov 29, 2019

Aw maaaan, @phatbhoy67 😄
That's so close. You're a nice detective, I can say that 👍

@PeterShaggyNoble
Copy link
Member

The source URL should probably be either the brand guidelines or the homepage.

@runxel
Copy link
Member Author

runxel commented Nov 29, 2019

Thanks for the link, @PeterShaggyNoble! :)
I've incorporated it.

@phatbhoy67
Copy link
Contributor

Thanks for updating the the .svg @runxel and thanks @PeterShaggyNoble for review, merging this one in 👍

@phatbhoy67 phatbhoy67 merged commit da8b0e3 into simple-icons:develop Nov 30, 2019
@runxel runxel deleted the trustpilot branch December 1, 2019 15:32
birjj added a commit that referenced this pull request Dec 10, 2019
New version number `v2.0.0`, closes [the related Milestone](https://github.com/simple-icons/simple-icons/milestone/1).

New icons:

- Fiverr (#1978)
- ResearchGate (#1980)
- Formstack (#1983)
- EPEL (#1942)
- SonarQube (#1943)
- SonarCloud (#1944)
- SonarSource (#1945)
- LLVM (#1958)
- Linode (#1981)
- W3C (#1990)
- OpenAPI Initiative (#1994)
- React Router (#1995)
- repl.it (#1996)
- Internet Archive (#1977)
- styled-components (#1970)
- Invoice Ninja (#1948)
- PrestaShop (#1969)
- BMC Software (#1900)
- Square (#2006)
- Loop (#2011)
- PJSIP (#2018)
- Trustpilot (#2008)
- Smugmug (#1918)
- Blackberry (#1927)
- Imgur (#1988)
- OBS Studio (#2009)
- Datacamp (#2016)
- Steinberg (#2015)
- SVGO (#2001)
- Gutenberg (#2014)
- GNU IceCat (#2017)
- SonarLint (#1946)
- BandLab (#2013)
- Tailwind (#2010)
- Pocket Casts (#2029)
- Workplace (#2021)
- Academia (#2026)
- Fossil SCM (#2036)
- Breaker (#2028)
- Brandfolder (#2035)
- Accusoft (#2031)
- F-Droid (#2039)
- XDA-Developers (#2020)
- PeerTube (#2033)
- Curl (#2012)
- write.as (#2007)
- RabbitMQ (#2052)
- iTunes (#2019)
- QuickTime (#2066)
- 1password (#2030)
- Wish (#2044)
- Reason Studios (#2054)
- Apache Kafka (#2060)
- Fossa (#2041)
- Dolby (#2081)
- Amazon Alexa (#2072)
- IcoMoon (#2074)
- Deliveroo (#2077)
- Home Assistant (#2088)
- Hypothesis (#2089)
- Trend Micro (#2090)
- OpenID (#2091)
- Teradata (#2092)
- Microsoft Office (#2093)
- LastPass (#2083)
- Quip (#2046)

Updated icons:

- Sublime Text (#1975)
- Deezer (#2005)
- Facebook (#2047)
- Uber (#2048)
- Yahoo! (#2097)

Removed icons:

- Google Allo (6ac966f)
- Lanyrd (6ac966f)
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.

TrustPilot icon request
3 participants