Skip to content

Conversation

@non-bin
Copy link
Contributor

@non-bin non-bin commented Jan 20, 2026

@flameshikari
Copy link
Collaborator

flameshikari commented Jan 20, 2026

Hello! Thanks for the 1st contribution.

Can you add all of your icons in a single PR from a single branch, please? It's quite difficult to overview them and merge one by one (I can guess there would be merge conflicts on every merge).

Mostly your icons look good, but they lack of padding like in quinn (looks like you exported the icon only, not the whole artboard), but quinn.png is 243x243px in size, and it must be 192x192px. So please, fix these sizes/padding issues, and I'll tweak the rest.

@non-bin
Copy link
Contributor Author

non-bin commented Jan 21, 2026

I set the art board size to the size of the icon, sorry if that was wrong. I'll check the PNG sizes and merge these all into a single pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants