-
Notifications
You must be signed in to change notification settings - Fork 304
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
BaseコンポーネントのStorybookファイルを作成 #2227
BaseコンポーネントのStorybookファイルを作成 #2227
Conversation
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.
おー!!!良いですね!!!!!!
まだ見てないのですがテストが落ちてそうだったのでそれだけ報告まで・・・!
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.
ほぼLGTMです!!!
非常に見やすかったです!!!
|
||
render: (args) => ({ | ||
components: { BaseButton }, | ||
|
||
setup() { | ||
return { args }; | ||
}, | ||
|
||
template: '<BaseButton v-bind="args" />', | ||
}), |
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.
Storybook、書式がコロコロ変わってよくわからないんですけど、多分この形(Meta
で指定したComponentにargsを直接渡し、特に表示物も変更しない場合)はこの関数がまるまるいらないはずです!
https://storybook.js.org/docs/api/csf#args-story-inputs
めちゃめちゃスリムになると思います 🙏
render: (args) => ({ | |
components: { BaseButton }, | |
setup() { | |
return { args }; | |
}, | |
template: '<BaseButton v-bind="args" />', | |
}), |
指摘いただいた冗長だった各箇所の記述を修正しました! |
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.
LGTM!!!
作ってくださってありがとうございます!!!
Storybook、展覧会みたいでやっぱ楽しいですね!!
ちなみにchromaticも一応アカウント作ってたりします。
とりあえず動かしてみてるだけですが、多分ここにコンポーネントのリストが追加されるはず。
https://www.chromatic.com/builds?appId=667d9c007418420dbb5b0f75
確かmainブランチにpushされたら更新されるようにはなっていて、storyごとにVRT(Visual Regression Testi)もしてくれるので何か変更があったら教えてくれるかもです。
本当はレビュープロセスにもVTR含めたいのですが、確か無料枠だと月5000スクリーンショットまでで、pushのたびにテストが走ってたら足りなくなりそうだなと思って省いています。
でも多分更新されたところだけVRTするって設定があったはずなので、試してみたいなと思ってたりします。
この辺りもし興味あれば・・・!
内容
今までに作成したBaseコンポーネントのStorybookファイルを作成します。また、それに付随して以下を行います。