-
Notifications
You must be signed in to change notification settings - Fork 77
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(accordion): add new component tokens and deprecate old tokens #11390
base: dev
Are you sure you want to change the base?
Conversation
packages/calcite-components/src/components/accordion-item/accordion-item.scss
Outdated
Show resolved
Hide resolved
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.
Just some nitpicks on the doc text.
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.
@alisonailea can we also change this to a feat()
instead of chore
since new variables are being added?
Something like feat(accordion): add new component tokens and deprecate old tokens
packages/calcite-components/src/components/accordion-item/accordion-item.scss
Show resolved
Hide resolved
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.
Marking as changes requested
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. |
…mp/11389-accordion
Related Issue: #10830
Summary
Deprecate
--calcite-accordion-item-background-color
Deprecate
--calcite-accordion-item-border-color
Deprecate
--calcite-accordion-item-icon-color
Deprecate
--calcite-accordion-item-text-color
Deprecate
--calcite-accordion-item-text-color-hover
Deprecate
--calcite-accordion-item-text-color-press
--calcite-accordion-border-color: Specifies the component's border color.
--calcite-accordion-background-color: Specifies the component's background color.
--calcite-accordion-text-color: Specifies the component's text color.
--calcite-accordion-text-color-hover: Specifies the component's main text color on hover.
--calcite-accordion-text-color-press: Specifies the component's main text color when pressed.
--calcite-accordion-item-background-color: [Deprecated] Use --calcite-accordion-background-color. Specifies the component's background color.
--calcite-accordion-item-border-color: [Deprecated] Use --calcite-accordion-border-color. Specifies the component's border color.
--calcite-accordion-item-content-space: Specifies the component's padding.
--calcite-accordion-item-end-icon-color: Specifies the component's iconEnd color. Fallback to --calcite-accordion-item-icon-color or current color.
--calcite-accordion-item-expand-icon-color: Specifies the component's expand icon color.
--calcite-accordion-item-header-background-color: Specifies the component's heading background color.
--calcite-accordion-item-heading-text-color: Specifies the component's heading text color.
--calcite-accordion-item-icon-color: [Deprecated] Use --calcite-icon-color. Specifies the component's default icon color.
--calcite-accordion-item-start-icon-color: Specifies the component's iconStart color. Fallback to --calcite-accordion-item-icon-color or current color.
--calcite-accordion-item-text-color: [Deprecated] Use --calcite-accordion-text-color. Specifies the component's text color.
--calcite-accordion-item-text-color-hover: [Deprecated] Use --calcite-accordion-text-color-hover. Specifies the component's text color on hover.
--calcite-accordion-item-text-color-press: [Deprecated] Use --calcite-accordion-text-color-press. Specifies the component's text color on press.