FaIcon should not accept a nullable IconData
hamishjohnson opened this issue · 1 comments
font_awesome_flutter/lib/src/fa_icon.dart
Line 31 in 0fc7f87
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!