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

Add React storybook #11

Merged
merged 11 commits into from Jan 11, 2017
Merged

Add React storybook #11

merged 11 commits into from Jan 11, 2017

Conversation

kenju
Copy link
Contributor

@kenju kenju commented Jan 11, 2017

React storybookを導入します。

メリット

  • 使用イメージがつかみやすい(開発側も、ライブラリ使用側も)
  • Kibela本体導入の実験も兼ねて

将来的には、build/配下にstaticなHTMLファイルとして生成したものを、Github Pagesのデモサイトとしても利用できます。

Image

yarn story:start で下記のデベロッパーツールが開きます。

screen shot 2017-01-11 at 10 54 39

@bitjourney/developers

レビューお願いします。

@@ -7,10 +7,11 @@
"import"
],
"rules": {
"arrow-body-style": 0
"arrow-body-style": 0,
"react/jsx-closing-bracket-location": 0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

なぜかVSCodeのESLint autosaveと相性が悪く、かつ体裁のルールなので、オフで問題ないと思い一旦オフにしてます 😓

Copy link
Contributor

Choose a reason for hiding this comment

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

おや。これがあるとどうなります?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

↓これはESLint pluginのautosaveをONにした状態で「保存」を試している例なのですが、なぜかインデント戻ってしまうのです。autosaveをdisableにするか、ここのルールだけignoreするかで考えて、後者を選びました。

screencast 2017-01-11 11-34-03-dq051

Copy link
Contributor

Choose a reason for hiding this comment

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

なるほど。一旦disableしてしかるべきところに報告できるといいすな。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

はい。たぶんPlugin側かな... issues ないか見てみます。

@gfx
Copy link
Contributor

gfx commented Jan 11, 2017

LGTM

@kenju
Copy link
Contributor Author

kenju commented Jan 11, 2017

レビューありがとうございます. マージします。

@kenju kenju merged commit 1867b41 into master Jan 11, 2017
@kenju kenju deleted the react-storybook branch January 11, 2017 03:01
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