Skip to content
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

Closed
cchaos opened this issue Jun 15, 2021 · 4 comments · Fixed by #4888
Closed

Deprecate all usages of secondary as a color type #4882

cchaos opened this issue Jun 15, 2021 · 4 comments · Fixed by #4888

Comments

@cchaos
Copy link
Contributor

cchaos commented Jun 15, 2021

#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 deprecating secondary.

For someone who takes this on, you can see the referenced issue as an example.

@babs20
Copy link
Contributor

babs20 commented Jun 16, 2021

Hello! Can I be assigned to this issue?

@cchaos
Copy link
Contributor Author

cchaos commented Jun 16, 2021

👋 Welcome @babs20! Feel free to tackle any of the github issues (good first issues are especially great to grab). For more contribution guidance, you can read through the CONTRIBUTING doc. I'll also specifically point out that we don't "assign" issues to community members, but you're always welcome to "claim" it 😉 .

@cchaos
Copy link
Contributor Author

cchaos commented Jun 16, 2021

I also want to point out that specifically for this issue, we don't want to remove the secondary options just yet. We just need to be sure we add success where we currently support secondary, and add a deprecation notice in the props comment like this:

* Any of our named colors.
* **`secondary` color is DEPRECATED, use `success` instead**
*/
color?: ButtonColor;

@babs20
Copy link
Contributor

babs20 commented Jun 16, 2021

I'll also specifically point out that we don't "assign" issues to community members, but you're always welcome to "claim" it 😉 .

Sounds good, and thanks for letting me know!

I also want to point out that specifically for this issue, we don't want to remove the secondary options just yet. We just need to be sure we add success where we currently support secondary, and add a deprecation notice in the props comment like this:

* Any of our named colors.
* **`secondary` color is DEPRECATED, use `success` instead**
*/
color?: ButtonColor;

Thanks for the heads up! I will keep this in mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants