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

Added B&R Automation #4552

Merged
merged 2 commits into from
Jan 6, 2021
Merged

Added B&R Automation #4552

merged 2 commits into from
Jan 6, 2021

Conversation

adamrusted
Copy link
Member

b-and-rautomation

Issue: Closes #3321
Alexa rank: 179,013

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

SVG in header of site. Orange pulled from SVG.

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 1, 2021
@fbernhart
Copy link
Contributor

@adamrusted Did you use this SVG from the header as source? Because I can see quite some differences (your version in green). The frame matches perfectly, but not the "B&R". Can you have another look at it?

image

@adamrusted
Copy link
Member Author

@fbernhart - I've just rebuilt from source, and compared against the source SVG. Should be good to go now!

@fbernhart
Copy link
Contributor

@adamrusted Strange ... I'm still seeing the same differences. 🤔 Are you sure you've pushed the correct file? 😅 The SVG in your new commit almost looks the same as the one in your old commit.

@adamrusted
Copy link
Member Author

@fbernhart yep - this is what I'm seeing on my end 😅 original icon below my path in pinkish-purple.

image

@fbernhart
Copy link
Contributor

Are we using the same source? 😄 I'm using the SVG found in the header, which is:

https://www.br-automation.com/typo3conf/ext/br_sitesetup/Resources/Public/Images/logo.svg

It isn't actually a full SVG. The letters B&R are image data, therefore it looks pixelated. While yours doesn't. 🤔 Which source are you using?

@adamrusted
Copy link
Member Author

adamrusted commented Jan 6, 2021

@fbernhart I'm using exactly that. The letters are vectors, they just have a weird texture mesh on it (see below) - which may just be an Illustrator thing. That may be causing the issue?

image

@fbernhart
Copy link
Contributor

@adamrusted Oh, now I saw it. You're right! I only saw the image texture first. But there is as well a clipping path laying underneath. 👍🏼

In Inkscape it's only displaying the texture by default, that's why it looked different on my side (see image) 😅 Thanks for the help! Let's merge it.

image

@fbernhart fbernhart merged commit 7bd502a into simple-icons:develop Jan 6, 2021
@adamrusted adamrusted deleted the BRAutomation branch January 6, 2021 16:24
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.

Icon request for B&R Automation
2 participants