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

Fix welcome component to work with dark theme #5785

Closed
shilman opened this issue Feb 27, 2019 · 10 comments
Closed

Fix welcome component to work with dark theme #5785

shilman opened this issue Feb 27, 2019 · 10 comments

Comments

@shilman
Copy link
Member

shilman commented Feb 27, 2019

Currently the welcome component that sb init installs has no background and is unreadable with the dark theme. It's a detail, and presumably most people delete the story anyway, but it's an ugly detail. 😄

storybook

@0xPetra
Copy link

0xPetra commented Mar 4, 2019

Oh yes, I noticed. I can take a look at this.

@ndelangen
Copy link
Member

Need any help @francopetra ?

@0xPetra
Copy link

0xPetra commented Mar 5, 2019

Hi @ndelangen! I don't know how to try sb init from my local repo (without a package manager like npm). And also how to test it for different frameworks (I only use React)
Thanks!

@ndelangen
Copy link
Member

@francopetra Want to schedule a call? I can give you a tour around the codebase and show you how I work. And talk a bit about the roadmap and such.

The gist is that we have examples in the monorepo, so after bootstrapping you can just run one of those and see the changes.

@0xPetra
Copy link

0xPetra commented Mar 9, 2019

Sure! I would love to take a look at it. Will send you an email right away!

ps. Sorry for the delay, just moved to a different city.

@malykhinvi
Copy link
Contributor

malykhinvi commented Mar 9, 2019

Hello everyone! @shilman what is the expected UI? White background or light text?
5785
If I understand correctly, component look and feel should not depend on theme in this case, so white background is expected, right?

@francopetra if you don't have enough bandwidth to work on this issue, I would love to contribute as well.

@shilman
Copy link
Member Author

shilman commented Mar 9, 2019

Yeah I think white background is the simplest thing. It's a simple fix, but it should be applied to all the different frameworks (React, Vue, Angular, etc. etc.)

@0xPetra
Copy link

0xPetra commented Mar 9, 2019

Hello everyone! @shilman what is the expected UI? White background or light text?
5785
If I understand correctly, component look and feel should not depend on theme in this case, so white background is expected, right?

@francopetra if you don't have enough bandwidth to work on this issue, I would love to contribute as well.

@maacky I see you already moved forward, so go ahead!

shilman added a commit that referenced this issue Mar 11, 2019
Fix #5785 - welcome component in dark theme
@shilman
Copy link
Member Author

shilman commented Mar 11, 2019

Yay!! I just released https://github.com/storybooks/storybook/releases/tag/v5.1.0-alpha.4 containing PR #5998 that references this issue. Upgrade today to try it out!

Because it's a pre-release you can find it on the @next NPM tag.

shilman added a commit that referenced this issue Mar 15, 2019
Fix #5785 - welcome component in dark theme
@shilman
Copy link
Member Author

shilman commented Mar 17, 2019

Huzzah!! I just released https://github.com/storybooks/storybook/releases/tag/v5.0.2 containing PR #5998 that references this issue. Upgrade today to try it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants