fluttercommunity/font_awesome_flutter

FaIcon should not accept a nullable IconData

hamishjohnson opened this issue · 1 comments

final IconData? icon;

I ran into an issue that caused a bug, where I was dynamically passing a value here, and one of the states was null. If this line changes to

final IconData icon;

Then we'll get compiler warnings :)

I thought really hard about this one. For me, it'd make total sense to do this change.
However, it would also further dissipate the feature gap between flutter's Icon and our FaIcon.

As there is already work underway to bring them both together again (flutter/flutter#99830), I won't add any more features to FaIcon.

You may however recreate this issue over at the flutter repository and let them have a look at it!