-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: migrate storybook to use args #443
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/ties-makimura/chibi-ch-i-lo/9MN5s8MwhcPF3W4ewe1ujTUNnwMF |
優先度設定を経ずに着手したこのプルリクエストですが、移行の対象である箇所をそのままにしても問題なく動作する(新規と既存の共存は可能)ので、この時点でレビューとマージにすすめるのでいかがでしょう? @horimasumi 現状機能追加にあわせてリファクタリングしていることがあるので、このプルリクエストの移行作業に当たるものも同じようにすすめればよいかと思いまして |
コンフリクト解消直後の状態だとなぜかstorybookが正常に機能しなかった(真っ白な画面が表示される)ので、 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
より便利になって良さそうに感じます 👍
export const Linked = Template.bind({}); | ||
Linked.args = { | ||
...Default.args, | ||
onBookLinkClick: undefined, | ||
linked: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Controlsによって引数が表示され操作できるようになっているためLinkedのようなストーリーは省いても良いようになったかもしれませんね。もちろんこのままでも良いです。ご判断にお任せします。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#255 (comment) への対応です