-
Notifications
You must be signed in to change notification settings - Fork 837
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
Deprecate all usages of secondary
as a color
type
#4882
Comments
Hello! Can I be assigned to this issue? |
👋 Welcome @babs20! Feel free to tackle any of the github issues ( |
I also want to point out that specifically for this issue, we don't want to remove the eui/src/components/button/button.tsx Lines 91 to 94 in 845cacb
|
Sounds good, and thanks for letting me know!
Thanks for the heads up! I will keep this in mind. |
#4874 Does this for all of our buttons, but there are still other usages like in EuiExpression and EuiBadge. We should get to a better consistency across the board and add
success
while deprecatingsecondary
.For someone who takes this on, you can see the referenced issue as an example.
The text was updated successfully, but these errors were encountered: