Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Add Themes to Storybook Helpers #3586

Merged
merged 3 commits into from
Jul 9, 2020
Merged

Conversation

ryanmccombe
Copy link
Contributor

Requirement for bbc/simorgh#7178

Overall change:
Move storybook themes into psammead-helpers

Preparation work for allowing individual stories to change the storybook theme, when using dark mode

Code changes:

  • Move storybook themes into psammead-helpers

  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@ryanmccombe ryanmccombe self-assigned this Jul 8, 2020
});

export default {
light: createTheme({ base: 'light' }),
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we call this mode instead of base? Just thinking

Copy link
Contributor Author

Choose a reason for hiding this comment

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

base is from the storybook-theming api - it's not something we can rename unfortunately

https://storybook.js.org/docs/configurations/theming/#create-a-theme-quickstart

@ryanmccombe ryanmccombe merged commit d90bbe7 into latest Jul 9, 2020
@amywalkerdev amywalkerdev deleted the add-themes-to-storybook-helpers branch September 28, 2020 12:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants