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 XDA-Developers icon #2020

Merged
merged 5 commits into from Dec 2, 2019
Merged

Add XDA-Developers icon #2020

merged 5 commits into from Dec 2, 2019

Conversation

PeterShaggyNoble
Copy link
Member

XDA-Developers

Issue: #1931

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

They use the xda for their favicon as well as their app icons. Colour was taken from website logo.

@runxel runxel added the new icon Issues or pull requests for adding a new icon label Nov 30, 2019
@phatbhoy67
Copy link
Contributor

Hey @PeterShaggyNoble thanks for taking this one on, the .svg is great. 👍

I can't see anywhere the dash is used i.e. "XDA-Developers" other than in the site url, so I think it should just be "XDA Developers".

Can you update the title in the .svg file, the filename and the .json entry please.

@PeterShaggyNoble
Copy link
Member Author

I was going by the <title> tag on the website but I see now that the hyphen isn't used anywhere else. Will update shortly.

@phatbhoy67
Copy link
Contributor

Hey @PeterShaggyNoble thanks for the quick reply.

Can you also update .svg file with <title>XDA Developers</title> (removing the "dash").

This will remove the linting error causing the check to fail. 🎉

@PeterShaggyNoble
Copy link
Member Author

Ah-ha! That's what I was missing!

Updated. Thanks, @phatbhoy67.

@phatbhoy67
Copy link
Contributor

Thanks for your work on this one @PeterShaggyNoble appreciated 👍

@phatbhoy67 phatbhoy67 merged commit c44f9f1 into simple-icons:develop Dec 2, 2019
phatbhoy67 added a commit that referenced this pull request Dec 2, 2019
@PeterShaggyNoble PeterShaggyNoble deleted the xdadevelopers branch December 2, 2019 15:56
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.

None yet

3 participants