-
Notifications
You must be signed in to change notification settings - Fork 3
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
40. よくあるボタンコンポーネントを作成する #212
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
# 課題1 課題2 | ||
## 以下の使用を満たすボタンのコンポーネントを作成してください | ||
## 上記ボタンコンポーネントのStoryをStorybookで作成してください | ||
https://github.com/YamazakiYusuke/praha-challenge-front-end/pull/5 | ||
|
||
# 課題3 | ||
## 今回のように「children」として要素を受け取ることでボタンの中身を変えることも、「text」といったpropsを定義してボタンのテキスト文言だけを変えることもできますどちらの設計でも似たようなことが実現できそうですが、果たしてどちらを採用するべきか、考えてみてください。 | ||
|
||
### 「children」として要素を受け取る | ||
メリット | ||
- childrenを自由に変更できるので、自由度が高く柔軟に引数から表示内容を変更できる。 | ||
|
||
デメリット | ||
- 自由度が高いので、引数による予期しないエラーやレイアウト崩れが発生する可能性がある。 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
これを言ってしまったら、他のコンポーネントもぜんぶそうなのでデメリットとしてあげるべきものではないと思います。 |
||
|
||
### 「text」といったpropsを定義してボタンのテキスト文言だけを変える | ||
メリット | ||
- 変更できる箇所が限られているので、コードが明瞭である。 | ||
|
||
デメリット | ||
- 自由度が低い。 | ||
Comment on lines
+16
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
つまり、複数の children を設け、children を挿入できるHTML構造を特定できるのが旨みです。 |
||
|
||
不用意に自由度の高いパラメータをセットすべきではない。 | ||
コンポーネントが求められる要件に応じて、適切な設計を選択することが重要である。 |
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.
おつかれさまでした。
YamazakiYusuke/praha-challenge-front-end#5 (review)