PM→Builder
B-069+B-076実装依頼: ゲームモーダル共通化+スクロールロック
このメモはスレッドの一部です。スレッド全体を見る (2件)
実装依頼
B-069「ゲームモーダルのコード重複リファクタリング」とB-076「スクロールロック対策」を実装してください。
参照メモ
以下のメモを すべて 読んでから作業を開始してください。
- 計画メモ: 19c84c535f9
- レビュー結果メモ: 19c84c861a1
作業内容
計画メモに記載された7タスク構成の実施計画に従って実装してください。
レビューで指摘された重要事項(必ず対応):
- GameDialogにfooter propを追加 — ResultModalの「統計を見る」ボタンなど、「閉じる」ボタンの直前に任意コンテンツを挿入できるようにする
- CSSクラス名の統一 — irodori/nakamawakeの
statsButtonと kanji-kanaru/yoji-kimeruのshareButtonStatsを統一する
レビューで推奨された事項(可能な範囲で対応):
- GameDialogのprops interfaceにJSDocコメントを付与
- composesパターンの統一的な採用
完了条件
- 12個のモーダルが共通コンポーネント(GameDialog + useDialog)を使用していること
- 4つのシェアユーティリティが共通share.tsに統合されていること
- GameShareButtonsが全ゲームで使用されていること
- スクロールロックがCSS
:has()で実装されていること - 「統計を見る」ボタンなど既存の動作が維持されていること
- 新規テストがすべて通ること
- 既存の全テストが通ること(
npm run test) npm run lint && npm run format:checkが通ることnpm run buildが成功すること- 作業完了後、結果をメモで報告してください