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

docs: Dark Mode applied in Storybook controls #2034

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

taylorvnoj
Copy link
Contributor

@taylorvnoj taylorvnoj commented Sep 18, 2024

Motivations

The controls section in stories didn't have Dark Mode applied and it made it difficult to read the props, etc.

Changes

Changed

  • adjusted tokens in theme.ts so the controls section can be viewed in Dark Mode

Before

Screenshot 2024-09-17 at 10 44 09 PM

After

Screenshot 2024-09-17 at 10 42 31 PM

Testing

Look at any story, and make sure the controls section (with the props) responds to the sun/moon theme toggle in the top nav

Screenshot 2024-09-17 at 10 41 11 PM

Changes can be
tested via Pre-release


In Atlantis we use Github's built in pull request reviews.

Random photo of Atlantis

@taylorvnoj taylorvnoj self-assigned this Sep 18, 2024
@taylorvnoj taylorvnoj marked this pull request as ready for review September 18, 2024 02:45
@taylorvnoj taylorvnoj requested a review from a team as a code owner September 18, 2024 02:45
appPreviewBg: 'var(--color-surface)',
appBorderColor: '#585C6D',
appBorderRadius: 4,
appBg: tokens["color-base-blue--1000"],
Copy link
Contributor

Choose a reason for hiding this comment

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

As a follow up it would be good to use semantic tokens here!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sounds good - I can probably do that as side desk work within this sprint

Copy link
Contributor

@chris-at-jobber chris-at-jobber left a comment

Choose a reason for hiding this comment

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

Looks good! Left a comment about a potential improvement but not a blocker

@taylorvnoj taylorvnoj merged commit e95e2f4 into master Sep 18, 2024
12 checks passed
@taylorvnoj taylorvnoj deleted the TAYLOR/style-storybook-controls-dark branch September 18, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants