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

React storybook #91

Merged
merged 2 commits into from Oct 12, 2016
Merged

React storybook #91

merged 2 commits into from Oct 12, 2016

Conversation

kschaper
Copy link
Contributor

I integrated a very useful tool called react-storybook. It basically allows you to create a catalogue of the app's React components. Have a look at the docs to learn more. Please give it a try:

$ nvm install
$ npm install
$ npm run storybook

Then open http://localhost:9001/ in a browser.

In my opinion this is a very nice companion to testing. What do you think?

import { storiesOf, action } from '@kadira/storybook';

import './storybook.css'
// for now just simply $ rake asset:precompile && mv public/assets/application-ebf... storybook/application.css
Copy link
Member

Choose a reason for hiding this comment

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

Is this because some of my components use CSS? What's the way forward here? Move the CSS into something like Radium, or the other way around, use CSS modules or whatever they're called..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's just a quick and dirty way to give the components the same style as in the app. Unfortunately I have no clue what's the best way to handle component styling and I hadn't the chance to try radium etc yet. But it would be definitely nice if components would bring their own styles.

@Gargron Gargron merged commit 4eeb794 into mastodon:master Oct 12, 2016
@kschaper kschaper deleted the react-storybook branch October 12, 2016 15:28
alpaca-tc pushed a commit to pixiv/mastodon that referenced this pull request Jun 15, 2017
…ns-max-tracks

DECKに入れられる最大曲数を表示
@Gargron Gargron mentioned this pull request Jul 27, 2017
hcmiya pushed a commit to hcmiya/v6don that referenced this pull request Jul 29, 2017
Kirishima21 pushed a commit to Kirishima21/mastodon that referenced this pull request Jun 23, 2018
ClearlyClaire pushed a commit to ClearlyClaire/mastodon that referenced this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants