Closed kuniyuki-f closed 11 months ago
https://63d52217f1430a5ad69846cd-irtqdeawyc.chromatic.com/?path=/story/templates-timertemplate--default
ページ内の見出しに適切な文言が入るように調整を行いました。 計測ページの見出しはFigmaに合わせました。 FIgma: https://www.figma.com/file/Mowdr0KHcHfm2bmi7pfg75/UI%E3%83%87%E3%82%B6%E3%82%A4%E3%83%B3%E8%B3%87%E6%96%99?type=design&node-id=258-90026&mode=design&t=42bVz7ZyNQcrSiop-0
計測ページについては、 pages/Timer.tsx から TimerTemplate -> DefaultLayout とtitleを受け渡しております。 (Pageコンポーネントではなく、TimerTemplateから直接titleで渡しても良いかと思いましたが、実装者が何かしら改修する際はおそらくpages起点でコードを読むと思ったのでpagesから受け渡すようにしました。)
とくになし
The latest updates on your projects. Learn more about Vercel for Git ↗︎
対応完了しました。レビューよろしくお願いします!
issueURL
122
この PR で対応する範囲 / この PR で対応しない範囲
Storybook の URL、 スクリーンショット
https://63d52217f1430a5ad69846cd-irtqdeawyc.chromatic.com/?path=/story/templates-timertemplate--default
変更点概要
ページ内の見出しに適切な文言が入るように調整を行いました。 計測ページの見出しはFigmaに合わせました。 FIgma: https://www.figma.com/file/Mowdr0KHcHfm2bmi7pfg75/UI%E3%83%87%E3%82%B6%E3%82%A4%E3%83%B3%E8%B3%87%E6%96%99?type=design&node-id=258-90026&mode=design&t=42bVz7ZyNQcrSiop-0
122 の補足事項にも記載いただいていましたように、今後の拡張性を考慮し、DefaultLayoutでPropsとしてtitleを受け取れるようにしています。
レビュアーに重点的にチェックして欲しい点
計測ページについては、 pages/Timer.tsx から TimerTemplate -> DefaultLayout とtitleを受け渡しております。 (Pageコンポーネントではなく、TimerTemplateから直接titleで渡しても良いかと思いましたが、実装者が何かしら改修する際はおそらくpages起点でコードを読むと思ったのでpagesから受け渡すようにしました。)
補足情報
とくになし