-
Notifications
You must be signed in to change notification settings - Fork 55
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
use fa6 icon names #121
use fa6 icon names #121
Conversation
You are using the role name to define the style of the icon, it should be alined with the version of font awesome advertized i.e. fa6
Thanks for submitting your first pull request! You are awesome! 🤗 |
aline icon styles on fa6
Codecov ReportBase: 90.02% // Head: 90.02% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #121 +/- ##
=======================================
Coverage 90.02% 90.02%
=======================================
Files 11 11
Lines 942 942
=======================================
Hits 848 848
Misses 94 94
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
for more information, see https://pre-commit.ci
Heya, thanks but... |
You are using the role name to define the style of the icon, it should be aligned with the version of font awesome advertised i.e. fa6.
To avoid older documentations with hundreds of icons stopping working overnight I added a correspondence table between the fa keys as they are changing between each version. o you think it would be useful to raise a warning when using an old one?
note: there is a bug in the pre-commit checks that is not related to my modifications.