Skip to content

Conversation

kenta-afk
Copy link
Contributor

関連チケット

変更内容

  • この変更で何をしましたか?
  • sectionのタイトルとhomeのタイトルのコンポーネントを作りました。

動作確認

  • どういった手順で動作確認を行いましたか?
  • できれば結果がわかるスクリーンショットや、動画をテンプしてください。

スクリーンショット 2025-03-27 134015
スクリーンショット 2025-03-27 134025
スクリーンショット 2025-03-27 134044
スクリーンショット 2025-03-27 134055

その他

  • レビュワーへ伝えたい内容

Copy link

@kenta-afk kenta-afk requested a review from naoido March 27, 2025 04:43
Copy link

cloudflare-workers-and-pages bot commented Mar 27, 2025

Deploying object-t-website-dev-storybook with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5aabf2d
Status: ✅  Deploy successful!
Preview URL: https://08c994c7.object-t-website.pages.dev
Branch Preview URL: https://feature-20-section-title.object-t-website.pages.dev

View logs

@kenta-afk kenta-afk requested a review from thirdlf03 March 27, 2025 05:38
Copy link
Contributor

@thirdlf03 thirdlf03 left a comment

Choose a reason for hiding this comment

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

スクリーンショット 2025-03-27 15 15 22

多分意図としては、こんな感じに一つのコンポーネントにまとめて欲しいってことだと思う
<Home title="Object<T>'" subtitle="学生の未来をもっと「明るく」" /> って感じ
今のままだと、homeとsectionそれぞれのtitleコンポーネントを後から組み合わせる必要があって使い回す時に少し不便かも

もうちょっと詳しくdiscordに書いてます

@kenta-afk kenta-afk requested a review from thirdlf03 March 27, 2025 07:46
@kenta-afk
Copy link
Contributor Author

スクリーンショット 2025-03-27 164823
スクリーンショット 2025-03-27 164830
スクリーンショット 2025-03-27 164835

Copy link
Contributor

@thirdlf03 thirdlf03 left a comment

Choose a reason for hiding this comment

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

LGTM!!

@kenta-afk kenta-afk merged commit fdf5f6c into develop Mar 27, 2025
1 check passed
@kenta-afk kenta-afk deleted the feature/20-section-title branch March 27, 2025 07:58
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.

2 participants