Storybook: Fix welcome component to work with dark theme

Created on 27 Feb 2019  路  10Comments  路  Source: storybookjs/storybook

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

bug good first issue ui

Most helpful comment

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

All 10 comments

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

Need any help @francopetra ?

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!

@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.

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.

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.

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.)

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!

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.

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!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tomitrescak picture tomitrescak  路  3Comments

zvictor picture zvictor  路  3Comments

xogeny picture xogeny  路  3Comments

miljan-aleksic picture miljan-aleksic  路  3Comments

tirli picture tirli  路  3Comments