Skip to content

feat(progress): add component tokens#8661

Merged
Elijbet merged 6 commits intoepic/7180-component-tokensfrom
elijbet/7180-add-component-tokens-for-progress
Feb 21, 2024
Merged

feat(progress): add component tokens#8661
Elijbet merged 6 commits intoepic/7180-component-tokensfrom
elijbet/7180-add-component-tokens-for-progress

Conversation

@Elijbet
Copy link
Copy Markdown
Contributor

@Elijbet Elijbet commented Jan 28, 2024

Related Issue: #7180

Summary

Adds the following component tokens : 🥇

--calcite-progress-background-color: defines the background color of the component.
--calcite-progress-bar-fill-color: defines the background color of the progress bar.
--calcite-progress-text-color: defines the text color of the component.

@Elijbet Elijbet changed the title Elijbet/7180 add component tokens for progress feat(progress): add component tokens Jan 28, 2024
@Elijbet Elijbet changed the base branch from main to epic/7180-component-tokens January 28, 2024 23:56
@Elijbet Elijbet marked this pull request as ready for review January 28, 2024 23:57
@Elijbet Elijbet requested a review from a team as a code owner January 28, 2024 23:57
Copy link
Copy Markdown
Contributor

@alisonailea alisonailea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good! Component tokens should be set at :host in most cases.

Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 6, 2024

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, 2024
@alisonailea alisonailea added design-tokens Issues requiring design tokens. and removed Stale Issues or pull requests that have not had recent activity. labels Feb 14, 2024
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
@Elijbet Elijbet force-pushed the elijbet/7180-add-component-tokens-for-progress branch from 1249127 to 6d54811 Compare February 17, 2024 23:40
Comment thread packages/calcite-components/src/components/progress/progress.scss
Copy link
Copy Markdown
Contributor

@alisonailea alisonailea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor thing on text-color. Looks good!

Comment thread packages/calcite-components/src/components/progress/progress.scss Outdated
@Elijbet Elijbet added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Feb 21, 2024
@Elijbet Elijbet merged commit a9161fb into epic/7180-component-tokens Feb 21, 2024
@Elijbet Elijbet deleted the elijbet/7180-add-component-tokens-for-progress branch February 21, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design-tokens Issues requiring design tokens. pr ready for visual snapshots Adding this label will run visual snapshot testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants