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
Adding Seagate Icon #1740
Adding Seagate Icon #1740
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @ngrosso!
Unfortunately it looks like something went wrong with the PR; nothing is showing up. Usually this means that the actual path is somewhere outside of the 24x24 viewbox.
Could you look into fixing this? If you need any help you can always ask here and we'll see if we can help you out.
@ngrosso Thanks for fixing this! The SVG looks good to me now, except for one detail: the logo seems to be just a bit less than 24pt wide. We want the logos to fill as much as they can, so the biggest side (either height or width) should take up the entire 24pt viewbox. Could you resize the logo slightly? Your editor should have a way to set the size of a path directly, instead of having to fiddle with resizing it using the mouse (e.g. see this comment if you're using Illustrator) - that should make it easier to make the fit perfect. |
👍 Looks good! Thanks for your work on this @ngrosso, it is much appreciated! |
# New icons - JetBrains (#1714) - Mattermost (#1712) - Gradle (#1724) - Anaconda (#1728) - Swagger (#1715) - HTML Academy (#1716) - Issuu (#1733) - Palantir (#1732) - Sonos (#1739) - LaTeX (#1722) - Stackbit (#1699) - Node-RED (#1727) - HackerOne (#1723) - Octave (#1743) - Lumen (#1741) - Gitea (#1742) - Groovy (#1735) - Kaspersky (#1738) - Seagate (#1740) - Quarkus (#1749) - RadioPublic (#1752) - Visual Studio (#1757) - Nokia (#1754) - Glitch (#1756) # Updated icons - Android (#1717)
# New icons - JetBrains (simple-icons#1714) - Mattermost (simple-icons#1712) - Gradle (simple-icons#1724) - Anaconda (simple-icons#1728) - Swagger (simple-icons#1715) - HTML Academy (simple-icons#1716) - Issuu (simple-icons#1733) - Palantir (simple-icons#1732) - Sonos (simple-icons#1739) - LaTeX (simple-icons#1722) - Stackbit (simple-icons#1699) - Node-RED (simple-icons#1727) - HackerOne (simple-icons#1723) - Octave (simple-icons#1743) - Lumen (simple-icons#1741) - Gitea (simple-icons#1742) - Groovy (simple-icons#1735) - Kaspersky (simple-icons#1738) - Seagate (simple-icons#1740) - Quarkus (simple-icons#1749) - RadioPublic (simple-icons#1752) - Visual Studio (simple-icons#1757) - Nokia (simple-icons#1754) - Glitch (simple-icons#1756) # Updated icons - Android (simple-icons#1717)
# New icons - JetBrains (simple-icons#1714) - Mattermost (simple-icons#1712) - Gradle (simple-icons#1724) - Anaconda (simple-icons#1728) - Swagger (simple-icons#1715) - HTML Academy (simple-icons#1716) - Issuu (simple-icons#1733) - Palantir (simple-icons#1732) - Sonos (simple-icons#1739) - LaTeX (simple-icons#1722) - Stackbit (simple-icons#1699) - Node-RED (simple-icons#1727) - HackerOne (simple-icons#1723) - Octave (simple-icons#1743) - Lumen (simple-icons#1741) - Gitea (simple-icons#1742) - Groovy (simple-icons#1735) - Kaspersky (simple-icons#1738) - Seagate (simple-icons#1740) - Quarkus (simple-icons#1749) - RadioPublic (simple-icons#1752) - Visual Studio (simple-icons#1757) - Nokia (simple-icons#1754) - Glitch (simple-icons#1756) # Updated icons - Android (simple-icons#1717)
# New icons - JetBrains (simple-icons#1714) - Mattermost (simple-icons#1712) - Gradle (simple-icons#1724) - Anaconda (simple-icons#1728) - Swagger (simple-icons#1715) - HTML Academy (simple-icons#1716) - Issuu (simple-icons#1733) - Palantir (simple-icons#1732) - Sonos (simple-icons#1739) - LaTeX (simple-icons#1722) - Stackbit (simple-icons#1699) - Node-RED (simple-icons#1727) - HackerOne (simple-icons#1723) - Octave (simple-icons#1743) - Lumen (simple-icons#1741) - Gitea (simple-icons#1742) - Groovy (simple-icons#1735) - Kaspersky (simple-icons#1738) - Seagate (simple-icons#1740) - Quarkus (simple-icons#1749) - RadioPublic (simple-icons#1752) - Visual Studio (simple-icons#1757) - Nokia (simple-icons#1754) - Glitch (simple-icons#1756) # Updated icons - Android (simple-icons#1717)
Issue:#1660
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
Credits for freepik
Got the simplified hex value from SVGOMG optimization