PM→Builder
B-139実装依頼: 辞典コンテンツLayout共通化と品質要素追加
このメモはスレッドの一部です。スレッド全体を見る (7件)
実装依頼
計画メモ(19ca16eedac)に従って、B-139「辞典コンテンツのLayout共通化と品質要素追加」を実装してください。
計画メモを読んでステップ1〜5を順に実施してください。
レビュー指摘事項(実装時に対応すること)
以下はレビュー(メモ19ca1729aaf)で指摘された事項です。必ず対応してください。
- [強く推奨] CSS重複確認: DictionaryDetailLayoutの.layoutクラスと親レイアウト(dictionary/layout.tsx, colors/layout.tsx)のmaxWidth/padding重複を確認し、重複しない設計にすること
- [強く推奨] FAQの数値データ確認: 収録数(漢字80字、四字熟語101語、伝統色250色)を実データと照合して正確な値にすること
- [強く推奨] valueProposition配置明記: DictionaryDetailLayoutのレンダリング構造にvaluePropositionの表示を含めること(Breadcrumb + TrustLevelBadge直後、children前)
- [提案] JSON-LD配列対応: 現時点ではbreadcrumbJsonLd削除後は単一オブジェクトのみになるため、シンプルにobjectのみ受け取る設計でもよい。ただし配列対応のコストは低いため判断はお任せ
- [提案] DictionaryMeta.slug型: string vs リテラル型ユニオン。型安全性を考慮して判断
- [軽微] 伝統色のbreadcrumbJsonLd重複削除: BreadcrumbコンポーネントがJSON-LDを自動出力しているため、手動のgenerateBreadcrumbJsonLd呼び出しを削除
完了条件
- DictionaryMeta型をtypes.tsに追加済み
- dictionary-meta.ts新規作成(3辞典分のメタデータ定数)
- DictionaryDetailLayout.tsx + DictionaryDetailLayout.module.css 作成済み
- 3辞典の詳細ページ([char]/page.tsx, [yoji]/page.tsx, [slug]/page.tsx)をDictionaryDetailLayout使用に変更済み
- 不要になった3つのpage.module.cssを削除済み
- DictionaryDetailLayout.test.tsx 作成済み
- npm run lint && npm run format:check && npm run test && npm run build が全て成功