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

Null safety support #45

Open
pasblin opened this issue Jun 19, 2021 · 8 comments · May be fixed by #47
Open

Null safety support #45

pasblin opened this issue Jun 19, 2021 · 8 comments · May be fixed by #47

Comments

@pasblin
Copy link

pasblin commented Jun 19, 2021

Env info

Bug info

@rodyfernandez
Copy link

#41

@jibiel jibiel linked a pull request Jul 10, 2021 that will close this issue
@jibiel
Copy link

jibiel commented Jul 10, 2021

Made the PR that addresses the problem: #47.

@josephcrowell
Copy link

I pushed out a fork which includes all missing icons including Fontisto fonts and jibiel's fixes for null safety support: https://pub.dev/packages/flutter_font_icons

@deadsoul44
Copy link

deadsoul44 commented Sep 9, 2021

I pushed out a fork which includes all missing icons including Fontisto fonts and jibiel's fixes for null safety support: https://pub.dev/packages/flutter_font_icons

Why did you remove common api FlutterIcons? @josephcrowell

@josephcrowell
Copy link

It's duplication of functionality from the other methods. I also don't have the split function that the other library had because I plan to implement that differently.

@yasinarik
Copy link

I pushed out a fork which includes all missing icons including Fontisto fonts and jibiel's fixes for null safety support: https://pub.dev/packages/flutter_font_icons

Thanks a lot @josephcrowell. This one is working: flutter_font_icons 2.2.4

@deadsoul44
Copy link

@josephcrowell do you have any plan to implement a common api? This way, it is hard to search and find icons.

@josephcrowell
Copy link

No

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 a pull request may close this issue.

6 participants