-
Notifications
You must be signed in to change notification settings - Fork 79
feat(filter): add component tokens #11885
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
Conversation
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. |
Ping. still need a review on this one. |
/** | ||
* CSS Custom Properties | ||
* | ||
* @prop --calcite-filter-content-space: Specifies the padding of the component's content. |
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.
Is this in practice the space around the input? Might be a little confusing here as it kind of insinuates there is a default slot for Filter. Nothing actionable here I don't think - are we still planning on deprecating the Filter component?
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 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.
I commented on the issue as well.
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 good, just some nitpick / tangential thoughts.
**Related Issue:** #7180 ## Summary Adds the following tokens for `filter` component. - --calcite-filter-content-space: Specifies the padding of the component's content. - --calcite-filter-input-background-color: Specifies the input's background color. - --calcite-filter-input-border-color: Specifies the input's border color. - --calcite-filter-input-corner-radius: Specifies the input's corner radius. - --calcite-filter-input-shadow: Specifies the shadow around the input. - --calcite-filter-input-icon-color: Specifies the input's icon color. - --calcite-filter-input-text-color: Specifies the input's text color. - --calcite-filter-input-placeholder-text-color: Specifies the input's placeholder text color. - --calcite-filter-input-actions-background-color: Specifies the background color of the input's `clearable` and `number-button-type` elements. - --calcite-filter-input-actions-background-color-hover: Specifies the background color of the input's `clearable` and `number-button-type` elements when hovered. - --calcite-filter-input-actions-background-color-press: Specifies the background color of the input's `clearable` and `number-button-type` elements when pressed. - --calcite-filter-input-actions-icon-color: Specifies the icon color of the input's `clearable` and `number-button-type` elements. - --calcite-filter-input-actions-icon-color-hover: Specifies the icon color of the input's `clearable` and `number-button-type` elements when hovered. - --calcite-filter-input-actions-icon-color-press: Specifies the icon color of the input's `clearable` and `number-button-type` elements when pressed. /nts when pressed.
Related Issue: #7180
Summary
Adds the following tokens for
filter
component.clearable
andnumber-button-type
elements.clearable
andnumber-button-type
elements when hovered.clearable
andnumber-button-type
elements when pressed.clearable
andnumber-button-type
elements.clearable
andnumber-button-type
elements when hovered.clearable
andnumber-button-type
elements when pressed./nts when pressed.