Skip to content

Conversation

@gakigaki
Copy link
Contributor

@gakigaki gakigaki commented Jan 26, 2026

概要

  • 基本レイアウト設定を追加し、サイト全体のレイアウト基準を明示しました。
  • ページ単位で「このページのみ/下層にも適用」の継承範囲を設定できるようにしました。
  • CSVインポートに layout_inherit_flag を追加しました。

背景や目的

  • トップページだけ別レイアウトにしたいケースが多い一方、従来はトップページのレイアウトが全ページに反映され、ページ追加時の再設定負担がありました。
  • 基本レイアウトを明示し、さらにページごとに「このページのみ/下層にも適用」を選べるようにすることで、不要な継承を避けて運用負担を減らすためです。
  • 「このページのみ」はトップページやセクションのインデックスページ(/news など)を想定し、下層は標準レイアウトに戻したい場合に使います。

変更内容

  • configs に base_layout を追加し、サイト基本設定から選択できるようにしました。
  • pages に layout_inherit_flag を追加し、継承判定をフラグで制御しました。
  • ページ編集/一覧に適用範囲の設定・表示を追加しました。
  • CSVインポートに layout_inherit_flag(必須)を追加しました。
  • DefaultController::getLayout の継承・フォールバックをユニットテストで検証しました。

特記事項

  • 既存データ移行: base_layout はトップページの layout を初期値として追加し、layout_inherit_flag は全ページ1/トップのみ0に初期化します。
  • CSVの layout_inherit_flag は必須(0/1)のため、既存CSVは列追加が必要です。
  • テストは DefaultControllerLayoutTest を実行済みです。

レビュー完了希望日

軽微な改修なので急ぎません。

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

有り

  • configs に base_layout のレコード追加
  • pages.layout_inherit_flag 列追加

チェックリスト

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