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

Paint storybook to dark #59

Merged
merged 3 commits into from
Jan 5, 2025
Merged

Conversation

AyIong
Copy link
Contributor

@AyIong AyIong commented Jan 5, 2025

About the PR

Changed the Storybook theme to a dark theme, since the TGUI components were originally made for a dark theme.

Why's this needed?

Debug and view components in a slightly more truthful environment, rather than on a white background

Video Idk how to fix flickering
chrome_XTZBJWwjxo.mp4

@tgstation-ci
Copy link

tgstation-ci bot commented Jan 5, 2025

Thank you for contributing to tgui-core! The workflow 'Chromatic Security' requires repository secrets and will not run without approval. Maintainers can add the CI Cleared label to allow it to run. Note that any changes to chromaitc-security.yml and chromatic.yml will not be reflected.

@AyIong
Copy link
Contributor Author

AyIong commented Jan 5, 2025

Thank you for contributing to tgui-core! The workflow 'Chromatic Security' requires repository secrets and will not run without approval. Maintainers can add the CI Cleared label to allow it to run. Note that any changes to chromaitc-security.yml and chromatic.yml will not be reflected.

You're getting annoying

Copy link
Member

@jlsnow301 jlsnow301 left a comment

Choose a reason for hiding this comment

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

excellent

@jlsnow301 jlsnow301 merged commit 7e7e5e0 into tgstation:main Jan 5, 2025
8 of 11 checks passed
@AyIong AyIong deleted the storybook-styling branch January 6, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants