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 Xilinx #3424

Merged
merged 7 commits into from
Jan 6, 2021
Merged

Add Xilinx #3424

merged 7 commits into from
Jan 6, 2021

Conversation

sharmavins23
Copy link
Contributor

@sharmavins23 sharmavins23 commented Aug 5, 2020

This is a part of my original request for HDL oriented icons. This adds the branded Xilinx icon, creators of Xilinx Vivado (one of the more popular SystemVerilog and Verilog IDEs.)

xilinx

Issue: Closes #3422
Alexa rank: 38,837

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

The color was picked based on the two major colors that the logo originally had. This was the main color (not the shadow color.)
I vectorized the logo with Inkscape.

@NovaGL
Copy link
Contributor

NovaGL commented Aug 6, 2020

Thanks for this. Where did you get the logo from or did you convert it from a PNG yourself?

Why did you choose that color? I would have gone with #ec1c24 myself

@sharmavins23
Copy link
Contributor Author

I went ahead and converted from a PNG. The color came from a screen select of that PNG's main color as well.

@NovaGL NovaGL linked an issue Aug 7, 2020 that may be closed by this pull request
@NovaGL
Copy link
Contributor

NovaGL commented Aug 7, 2020

@sharmavins23

I managed to get a vector version of this logo and the color choices available are
#ed1c29 or #b30838

Here is the path I used
M23.7,0l-5.2,6l-5.2-6H23.7z M18.5,18l-5.2,6h10.4L18.5,18z M13.3,12L8,6l5.2-6H5.6L0.3,6l5.2,6l-5.2,6 l5.2,6h7.7L8,18L13.3,12z

It can be found in this file
https://www.xilinx.com/support/documentation/user_guides/ug116.pdf

As you can see your treatment is different than the vector. Yours in red, original in black.
image

@sharmavins23
Copy link
Contributor Author

@NovaGL added your changes, awaiting CI and approval

@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Aug 24, 2020
icons/xilinx.svg Outdated Show resolved Hide resolved
@PeterShaggyNoble
Copy link
Member

Welcome to Simple Icons, @sharmavins23 and thanks for the contribution 🙂

Further to the comments already made by the rest of the team above, I think we should also consider #171c2d for the brand colour as that was my first instinct on visiting their website.

@adamrusted, I think, in the absence of any statement to the contrary from either company, that we should work under the assumption that Xilink will continue to exist as a separate brand.

@adamrusted
Copy link
Member

@PeterShaggyNoble looking through the publications here and the favicon of the website, I'd suggest the red is more synonymous with the brand, especially when all their hardware is bright red too. Happy for another @simple-icons/maintainers to say otherwise though 😅

@adamrusted
Copy link
Member

xilinx

@simple-icons/maintainers due to lack of response by @sharmavins23, I've merged the newer path in that I'd suggested above, and pulled the hex value from the SVG on the homepage directly.

@fbernhart
Copy link
Contributor

@adamrusted Perfect! @sharmavins23 thanks as well for the initial work on this icon! 🏆

@fbernhart fbernhart merged commit 27f1184 into simple-icons:develop Jan 6, 2021
ericcornelissen added a commit that referenced this pull request Jan 10, 2021
# New Icons

- 3M (#3631)
- Accenture (#4396)
- AIOHTTP (#4536)
- AniList (#4609)
- AnyDesk (#4663)
- Apache JMeter (#4594)
- Apache Kylin (#4545)
- ARK Ecosystem (#4699)
- B&R Automation (#4552)
- bigbasket (#4572, #4616)
- Bugcrowd (#4615)
- CheckiO (#4512)
- Chrysler (#4357)
- Cloudways (#3352)
- Cockroach Labs (#4665)
- CodeProject (#4617)
- CodinGame (#4575)
- Cognizant (#4392)
- Contentful (#4537)
- Cookiecutter (#3649)
- DataStax (#4393)
- De'Longhi (#4426)
- Deepnote (#4546)
- Digi-Key Electronics (#4398)
- Eclipse Vert.x (#4631)
- edX (#4452)
- egghead (#4644)
- Eight Sleep (#3638)
- Facepunch (#3671)
- FedEx (#4564)
- Fortran (#4397)
- FreeNAS (#4573)
- Gitee (#4625)
- Google Domains (#4622)
- Google Photos (#4341)
- Grubhub (#4544)
- Homebridge (#4607)
- Hyperledger (#4673)
- IKEA (#4555)
- Ingress (#3389)
- Jitsi (#4540)
- Kitsu (#4520)
- Klook (#4548)
- Lada (#4361)
- Lichess (#3535)
- Linktree (#4662)
- Looker (#4442)
- McLaren (#4538)
- MetaFilter (#4547)
- Motorola (#3425)
- Noun Project (#4608)
- Observable (#4640)
- Open Bug Bounty (#4627)
- Open Nebula (#4551)
- OpenCV (#4580)
- PageSpeed Insights (#4553)
- Paytm (#4643)
- pfSense (#4671)
- Pop!_OS (#3769)
- Progate (#3680)
- Progress (#4611)
- PyPy (#4281)
- Racket (#3483)
- Radar (#4319)
- Rainmeter (#3554)
- Reliance Industries Limited (#4702)
- Revue (#4391)
- ROS (#4658)
- SciPy (#3772)
- Scrimba (#4556)
- SEPA (#3154)
- SoloLearn (#4624)
- Spark AR (#4468)
- Speedtest (#4626)
- STARZ (#4612)
- Streamlit (#4412)
- Swiper (#4562)
- São Paulo Metro (#4570)
- tado° (#4637)
- TaxBuzz (#4619)
- Testin (#4571)
- Testing Library (#4542)
- ThinkPad (#4550)
- Ticketmaster (#4526)
- Tile (#4614)
- tmux (#3512)
- Treyarch (#4690)
- TrueNAS (#4574)
- Unraid (#4613)
- UPS (#4565)
- USPS (#4549)
- Vault (#4606)
- Vimeo Livestream (#4703)
- Webflow (#4568)
- Weights & Biases (#4630)
- Wikiquote (#4687)
- Winmate (#3316)
- Wipro (#4359)
- Xilinx (#3424)
- Zelle (#4605)
- Zoho (#3661)
- Zomato (#4664)
- Zotero (#4561)

# Updated Icons

- Adobe XD (#3386)
- Amazon Lumberyard (#4591)
- Andela (#4592)
- Apache Maven (#4593)
- Daimler (#4618)
- DigitalOcean (#4648)
- Fossa (#4473)
- Gatsby (#4620)
- GIPHY (#4621)
- Google (#4508)
- Google Analytics (#4504)
- Google Cardboard (#4058)
- Google Fit (#4506)
- Harbor (#4598)
- Jira Software (#4657)
- Kahoot! (#4635)
- Kotlin (#4485)
- Kyocera (#4475)
- Lenovo (#4477)
- Material Design Icons (#4642)
- Mozilla (#4656)
- pr.co (#4655)
- SonicWall (#4654)
- StackPath (#4652)
- Stadia (#4503)
- Synology (#4220)
- TED (#4653)
- Trustpilot (#4481)
- Uber (#4650)
- Uber Eats (#4651)
- Udacity (#4596)
- Viber (#4597)
- Waze (#4583)
- Wish (#4482)
- WP Engine (#4639)
- XMPP (#4483)
- YouTube (#4582)
- YouTube Studio (#4581)
- YouTube TV (#4405)
- Zendesk (#4595)
- Zillow (#4636)
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.

Request: Verilog/Xilinx Vivado
6 participants