-
Notifications
You must be signed in to change notification settings - Fork 349
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
feat(Icons): Add fa regular icons and update solid icons #1385
Conversation
Btw we are still missing FA-brands-free icons. Which is another 414 icons. |
PatternFly-React preview: https://1385-pr-patternfly-react-patternfly.surge.sh |
Pull Request Test Coverage Report for Build 4516
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM .Can you please resolve conflicts so we can merge this one. Thanks!
615a1a8
to
4d9aa6c
Compare
4d9aa6c
to
0b0b408
Compare
0b0b408
to
7a75d1b
Compare
7a75d1b
to
69f76cf
Compare
Codecov Report
@@ Coverage Diff @@
## master #1385 +/- ##
=======================================
Coverage 83.41% 83.41%
=======================================
Files 543 543
Lines 5619 5619
Branches 12 12
=======================================
Hits 4687 4687
Misses 930 930
Partials 2 2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What:
Because new icons were added to solid icons we should pull them and expose them for others as well.
Also add new package with regular icons (outlined solid) with prefix
outlined${IconName}
so we don't have any name clashes.Overall adds 277 of new icons!