-
-
Notifications
You must be signed in to change notification settings - Fork 428
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(/lib/theme): add Tailwind CSS colors to <Alert>
s
#586
feat(/lib/theme): add Tailwind CSS colors to <Alert>
s
#586
Conversation
…ert component (issue themesberg#473) themesberg#473 re themesberg#473
…te dark add "gray" and "light" to add support to all FlowbiteColors re themesberg#473
linter
Codecov ReportBase: 99.31% // Head: 99.31% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #586 +/- ##
=======================================
Coverage 99.31% 99.31%
=======================================
Files 131 131
Lines 6387 6412 +25
Branches 477 477
=======================================
+ Hits 6343 6368 +25
Misses 44 44
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. |
…o just FlowbiteColors Since AlertColors now implement all colors from FlowbiteColors I had replaced it
<Alert>
s
This was a mistake I made while merging their PR manually.
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
I can't merge this without a re-review from you @rluders |
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. Thank you.
Description
(It's my first PR here lol. Looking forward to fix things if needed. :))
This change starts adding all colors to components in Flowbite. I'll be partially updating the components until I got every of then with all colors from FlowbiteColors avaliable.
Fixes # 473 (partially)
Type of change
How Has This Been Tested?
Since it was a visual change I used the Storybook to validate if colors are changing as expected.
Checklist: