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

Storybook #95

Merged
merged 6 commits into from
Nov 11, 2020
Merged

Storybook #95

merged 6 commits into from
Nov 11, 2020

Conversation

xasopheno
Copy link
Contributor

@xasopheno xasopheno commented Nov 10, 2020

Adds Storybook with support for CSS modules. Definition following in my editor does not go to the css definition, but instead the module declarations. Maybe this is just a kink in my setup.

@xasopheno xasopheno requested a review from hallettj November 10, 2020 21:45
"concurrently": "^5.3.0",
"eslint-plugin-graphql": "^4.0.0",
"storybook-css-modules-preset": "^1.0.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should be dropped if we opt for another css framework.

Copy link
Contributor

@hallettj hallettj left a comment

Choose a reason for hiding this comment

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

If we go this route I'd like to see a working example with a Sass module.

template/packages/frontend/.storybook/main.js Outdated Show resolved Hide resolved
template/packages/frontend/components/Title.tsx Outdated Show resolved Hide resolved
Comment on lines +35 to +36
"babel-loader": "^8.2.0",
"babel-preset-react-app": "^10.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm a bit bothered by adding these babel dependencies if we don't directly reference them. Are they necessary? This applies to @babel/core too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seems that babel is required for storybook.
storybookjs/storybook#7156

xasopheno and others added 3 commits November 11, 2020 10:41
Co-authored-by: Jesse Hallett <jesse.hallett@originate.com>
Co-authored-by: Jesse Hallett <jesse.hallett@originate.com>
@xasopheno xasopheno merged commit a2e844f into master Nov 11, 2020
@xasopheno xasopheno deleted the dm/storybook branch November 11, 2020 17:14
@github-actions
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants