Skip to content

chore(custom-theme): refactor storybook theme for component tokens#10111

Merged
alisonailea merged 9 commits intodevfrom
ali-active/7180-custom-theme
Aug 20, 2024
Merged

chore(custom-theme): refactor storybook theme for component tokens#10111
alisonailea merged 9 commits intodevfrom
ali-active/7180-custom-theme

Conversation

@alisonailea
Copy link
Copy Markdown
Contributor

Related Issue: #7180

Summary

Breaks the custom-theme storybook page into component files for easier maintenance. This will allow us to move all component token chromatic tests to a single file.

@alisonailea alisonailea self-assigned this Aug 19, 2024
@github-actions github-actions Bot added the chore Issues with changes that don't modify src or test files. label Aug 19, 2024
Copy link
Copy Markdown
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.

Nice! 👍

Just a few minor tweaks

Comment thread packages/calcite-components/src/custom-theme/global.css Outdated
Comment thread packages/calcite-components/src/custom-theme/accordion.tsx
Comment thread packages/calcite-components/src/custom-theme/card.tsx Outdated
Copy link
Copy Markdown
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.

👍 ⭐

Copy link
Copy Markdown
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

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

Awesome! 🚀

Once comments are addressed, this should be good to merge!

Comment thread packages/calcite-components/src/custom-theme.stories.ts Outdated
Comment thread packages/calcite-components/src/custom-theme/button.ts Outdated
Comment thread packages/calcite-components/src/custom-theme/button.ts Outdated
Comment thread packages/calcite-components/src/custom-theme/accordionItem.ts Outdated
@alisonailea alisonailea added the skip visual snapshots Pull requests that do not need visual regression testing. label Aug 20, 2024
@alisonailea alisonailea merged commit 971e309 into dev Aug 20, 2024
@alisonailea alisonailea deleted the ali-active/7180-custom-theme branch August 20, 2024 20:58
@github-actions github-actions Bot added this to the 2024-08-27 - Aug Release milestone Aug 20, 2024
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. skip visual snapshots Pull requests that do not need visual regression testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants