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 Forestry #5454
Add Forestry #5454
Conversation
2911213
to
14acbda
Compare
Welcome to Simple Icons, @AMatisse, and thanks for the contribution 👍🏻 Could you have another look at this, though, please? It should be sized relative to its largest dimension (width in this case) rather than both dimensions being (@mondeja, could you have a look at this one when you have a minute? The height is actually |
Hey @PeterShaggyNoble 👋 |
Thanks, @AMatisse. I'm noticing some hairline thin differences (e.g., bottom right branch of the left tree, with your version in red) throughout, though. This is most likely down to the precision being set too low during optimisation rather than anything you did wrong. Could you try again with a precision of 3? |
I thought I had done the optimization with a precision of 3. To be sure I went back to SVGOMG, and the optimized file is identical to this one. (A higher precision does not change the svg either) |
You'll need to run SVGOMG on your original SVG, pre-optimisation rather than the version included in the PR. If you're still having trouble with it, you're more than welcome to use my version below: <svg role="img" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><title>Forestry icon</title><path d="M5.564.332v2.82L0 8.736l1.305 1.284 4.26-4.26v2.568L0 13.912l1.305 1.283 4.26-4.26v12.733h1.831V10.932l4.284 4.263 1.304-1.283-5.588-5.588V5.756l3.989 3.969 5.195 5.214v8.729h1.832v-8.725L24 9.355l-1.305-1.283-4.283 4.264V9.768L24 4.18l-1.305-1.284-4.283 4.264V.332H16.58v6.824l-4.26-4.26-1.304 1.284 5.564 5.584v2.568l-3.596-3.596-5.588-5.588V.332H5.564Z"/></svg> |
This time I hope it's the right one 🤞 |
Thanks for the update, @AMatisse. As you're using my path now, though, we'll need one of the other @simple-icons/maintainers to review this before merging. |
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.
LGTM! Thanks @AMatisse and @PeterShaggyNoble 💪🚀
# New Icons - Automattic (#5489) - Datto (#5493) - FamPay (#5505) - Forestry (#5454) - Immer (#5437) - KashFlow (#5465) - Kdenlive (#5193) - LIFX (#5448) - Photopea (#5518) - pnpm (#5482) # Updated Icons - Gitpod (#5464) - IBM (#5496) - IBM Watson (#5496) - ICON (#5496) - Iconfinder (#5496) - iDEAL (#5496) - IEEE (#5496) - iFixit (#5496) - iFood (#5496) - IFTTT (#5496) - iHeartRadio (#5496) - IKEA (#5496) - IMDb (#5496) - Imgur (#5496) - Imou (#5496) - Indeed (#5496) - Infiniti (#5496) - InfluxDB (#5496) - Informatica (#5496) - Infosys (#5496) - Ingress (#5496) - Inkscape (#5496) - Insomnia (#5496) - Instagram (#5496) - Instapaper (#5496) - Intel (#5496) - IntelliJ IDEA (#5496) - Intercom (#5496) - Internet Archive (#5496) - Internet Explorer (#5496) - Invoice Ninja (#5496) - ioBroker (#5496) - Ionic (#5496) - Issuu (#5496) - Itch.io (#5496) - PayPal (#5467) - Plus Codes (#5468) - R (#5285) - Zotero (#5491)
Issue: Closes #5453
Alexa rank: 94,282
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
I used the Photopea tool to put the logo in the right conditions, from this file.
The color is not clearly indicated on the website, so I used the color in which it is displayed on the website. (#343a40)