-
Notifications
You must be signed in to change notification settings - Fork 75
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(select): add component tokens #8599
feat(select): add component tokens #8599
Conversation
…onent-tokens-for-select
This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions. |
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.
Looks pretty good. I think the tokens can be simplified and font-size
and space
made internal
…onent-tokens-for-select
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.
There seems to be something off with the VRT. Looks like the font size changed?
Do e2e tests pass for you? I'm noticing an issue with the select > form associated tests. |
…onent-tokens-for-select
FYI, looking into this one. 👀 |
@alisonailea E2E tests are ✅ now. |
Related Issue: #7180
Summary
Adds the following component tokens (CSS props):
--calcite-select-background-color
--calcite-select-background-color-hover
--calcite-select-border-color
--calcite-select-text-color
and deprecates the following:
--calcite-select-font-size
--calcite-select-spacing
Note:
--calcite-select-space-inline
was removed from the doc as it is not defined nor used