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

Fix iconsax font #10

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

jb3rndt
Copy link

@jb3rndt jb3rndt commented Mar 9, 2023

Hi, thank you for the icons. I noticed that all two colored icons dont work (because they are split into multiple glyphs). Additionally, some icons are rendered with an offset, so that they are not centered anymore. Both issues are demonstrated in the screenshots below:
| | | | |

This pull request is supposed to just fix the offsetting. As the other icons families (bulk and twotone) were unusable anyway, i left them out. Additionally, if im not mistaken, the "outline" family is equal to the "linear" family, so I did not recreate this one either.

To have a more verbose and thus better access to the different icon families, they have to be accessed by calling IconsaxLinear, IconsaxBroken and IconsaxBroken.

I also noticed that some of the icon names didnt match the others in the other families (e.g. each family hat ~6 icons that had names like "group" or "frame" which in reality had to be named differently) probably because of a poor figma export. So I re-arranged all the svg icons to match in the three families. If you want, I can send them over if you need them.

With that, at least the flutter library of Iconsax should be a bit more stable. Dont hesitate in case you have questions :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant