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

storyshotsの導入 #398

Merged
merged 37 commits into from
Oct 11, 2019
Merged

storyshotsの導入 #398

merged 37 commits into from
Oct 11, 2019

Conversation

kobakazu0429
Copy link
Member

closed #395

WHY

  • コンポーネントが増え、一覧が見えたほうがいいと思ったから
  • APIから情報を取ってくるためテキストの量によってUIが崩れるおそれがあるから、それを確認できるようにするため

WHAT

やったこと

  • Storybookの導入
  • Storyshotsの導入

やってないこと

  • netlifyで自動ビルド

? "file:///home/circleci/repo/frontend/storybook-static"
: "http://localhost:6006/";

const beforeScreenshot: CustomConfig["beforeScreenshot"] = (
Copy link
Member Author

Choose a reason for hiding this comment

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

circleciのcpuがしょぼくてレンダリングが終わらない時があるので、600 ms 遅らせている

);
};

const getMatchOptions: CustomConfig["getMatchOptions"] = _context => ({
Copy link
Member Author

Choose a reason for hiding this comment

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

macとdockerで微妙にフォント周りのレンダリングが変わってるみたいなので5%以内のdiffは許容するようにしてみた
これで様子を見てみて、運用が辛そうなら変更を考える

Copy link
Member

@euglena1215 euglena1215 left a comment

Choose a reason for hiding this comment

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

良さそう 👍 👍 👍

@kobakazu0429 kobakazu0429 merged commit 0bc8006 into feature/react Oct 11, 2019
@kobakazu0429 kobakazu0429 deleted the react/storyshots branch October 11, 2019 13:52
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.

2 participants