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 Penpot icon #7911
Add Penpot icon #7911
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 your contribution, @adsingh14! 🙏
Any reason why you're using a precision of 4 instead of the default 3? I've also left an inline comment regarding the source URL.
Is there any difference between the original and this icon? If so, then I'll proceed with another icon. |
I haven't checked yet because the default precision is 3 and I was wondering whether you had noticed a need to increase it to 4. If we can get an accurate icon with a precision of 3, that would be better. |
The icon might be a perfect fit, I was just wondering whether a precision of 3 would suffice without sacrificing accuracy. I.e.: If you optimized the icon to a precision of 3, would the fit be good, too? |
It looks almost the same with precision 4 too. |
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.
@adsingh14 Yes. Precison 3 looks good as well.
Issue: closes #6664
Similarweb rank:
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
Extracted from website header and edited in Inkscape, then rendered from SVGO with
precision: 3