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

feat(accordion): add new component tokens and deprecate old tokens #11390

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

alisonailea
Copy link
Contributor

@alisonailea alisonailea commented Jan 27, 2025

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.

@alisonailea alisonailea changed the title chore(accordion): swap deprecate accordiuon/accordion-item tokens chore(accordion): swap deprecate accordion/accordion-item tokens Jan 27, 2025
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Jan 27, 2025
Copy link
Contributor

@macandcheese macandcheese left a 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.

Copy link
Member

@driskull driskull left a 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

Copy link
Member

@driskull driskull left a 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

@jcfranco jcfranco changed the title chore(accordion): swap deprecate accordion/accordion-item tokens chore(accordion): swap deprecated accordion/accordion-item tokens Jan 29, 2025
Copy link
Contributor

github-actions bot commented Feb 6, 2025

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.

@github-actions github-actions bot added the Stale Issues or pull requests that have not had recent activity. label Feb 6, 2025
@alisonailea alisonailea changed the title chore(accordion): swap deprecated accordion/accordion-item tokens feat(accordion): add new component tokens and deprecate old tokens Feb 10, 2025
@github-actions github-actions bot added the enhancement Issues tied to a new feature or request. label Feb 12, 2025
@alisonailea alisonailea requested a review from driskull February 12, 2025 00:56
@alisonailea alisonailea removed the Stale Issues or pull requests that have not had recent activity. label Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. enhancement Issues tied to a new feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants