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(Storybook): update storybook to use carbon styles #7866

Merged
merged 6 commits into from
Feb 24, 2021

Conversation

tw15egan
Copy link
Member

@tw15egan tw15egan commented Feb 22, 2021

Updates our Storybook environment to use Carbon styles to better match some of the other Carbon storybook environments out there. Wonder if we also want to add some sort of intro page like Carbon Charts?

Changelog

New

  • Added Carbon favicon
  • Added Carbon theme file

Testing / Reviewing

Everything should work the same, let me know if we want to tweak any of the colors/styles, more info can be found in the storybook documentation

@tw15egan tw15egan requested a review from a team as a code owner February 22, 2021 22:35
@netlify
Copy link

netlify bot commented Feb 22, 2021

Deploy preview for carbon-elements ready!

Built with commit 9277e86

https://deploy-preview-7866--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Feb 22, 2021

Deploy preview for carbon-components-react ready!

Built with commit 9277e86

https://deploy-preview-7866--carbon-components-react.netlify.app

Copy link
Contributor

@joshblack joshblack left a comment

Choose a reason for hiding this comment

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

Looks awesome!

@netlify
Copy link

netlify bot commented Feb 22, 2021

Deploy preview for carbon-elements ready!

Built with commit c9152d3

https://deploy-preview-7866--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Feb 22, 2021

Deploy preview for carbon-components-react ready!

Built with commit c9152d3

https://deploy-preview-7866--carbon-components-react.netlify.app

Copy link
Contributor

@jnm2377 jnm2377 left a comment

Choose a reason for hiding this comment

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

So cool! I think an intro page would be really dope, but not a blocker!

@aagonzales
Copy link
Member

This looks nice! I like it reflecting the other storybooks and the carbon brand more. A intro page would be nice too so we don't have to link to accordion or another component when sharing a link to it as well. But I agree that's not a stop ship.

Some small visual updates if you're looking for recommendations:

  • Change the tab label's and knob label's font-weight to semiBold / 600.
  • Change the selected tab text color to Gray100 #161616 to closer match the style of our component tabs
  • Change the border dividers to use Gray20 #e0e0e0
  • Change the selected page background in the left panel to be Gray20 #e0e0e0 and use Gray100 #161616 text.

@tw15egan
Copy link
Member Author

@aagonzales Thanks! I'll make those updates. If you have any quick mocks for an intro page I'd be happy to add that in, but we can add that in later if you're busy 🙂

@tw15egan
Copy link
Member Author

tw15egan commented Feb 24, 2021

@joshblack @jnm2377 I grouped the components into categories, so you may want to take another look and see if this is something we would want to do. We could probably group a few more components together (Tooltip variants stand out), but figured this was a quick easy start to see if we wanted to stick with this

@aagonzales style changes made and welcome page added 🎉

@jnm2377
Copy link
Contributor

jnm2377 commented Feb 24, 2021

Oooh, I really like the groups. It feels a lot more organized. Esp having deprecated and experimental in their own sections. This is great!!! Thanks for doing this! 🔥 🎉 💯

Copy link
Member

@aagonzales aagonzales left a comment

Choose a reason for hiding this comment

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

Yes! That looks great

@joshblack
Copy link
Contributor

@tw15egan love it 🔥

@kodiakhq kodiakhq bot merged commit 1803315 into carbon-design-system:master Feb 24, 2021
@dakahn dakahn mentioned this pull request Mar 4, 2021
emyarod added a commit to emyarod/carbon that referenced this pull request Jun 30, 2021
emyarod added a commit to emyarod/carbon that referenced this pull request Jun 30, 2021
kodiakhq bot added a commit that referenced this pull request Jul 1, 2021
* chore(react): begin storybook v6 migration

* fix(storybook): use carbon-react babel-loader exclusions

* docs(Tag): update storybook action import

* chore: regenerate lockfile

* chore: dedupe packages

* docs(storybook): restore favicon

originally added in #7866, relocated (#8630), but never reverted (#8684)

* docs(storybook): add style overrides

* chore: update lockfile

* style(Welcome): fix background

Co-authored-by: TJ Egan <tw15egan@gmail.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
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.

4 participants