Skip to content

fix: Bootstrap 5 で廃止されたクラスを置き換える - #101

Open
dotani1111 wants to merge 1 commit into
EC-CUBE:4.4from
dotani1111:dev/4.4-bootstrap5
Open

fix: Bootstrap 5 で廃止されたクラスを置き換える#101
dotani1111 wants to merge 1 commit into
EC-CUBE:4.4from
dotani1111:dev/4.4-bootstrap5

Conversation

@dotani1111

@dotani1111 dotani1111 commented Aug 11, 2026

Copy link
Copy Markdown

概要(Overview・Refs Issue)

#95 のレビューでご指摘いただいた、Bootstrap 5 で廃止されたクラスの残存を置き換えます。
ご指摘の form-group / text-right に加え、同種の廃止クラスを管理画面テンプレート全体で洗い出して置き換えています。

方針(Policy)

  • 置き換え先は本体 4.4 の管理画面テンプレート(admin/Product/index.twig 等)の記法に合わせる
  • 対象は管理画面の 3 テンプレート(index / edit / config)。フロント側テンプレートに廃止クラスはなく、data-toggle 等の data-bs-* 化は移行済みでした

実装に関する補足(Appendix)

置き換え内容は次のとおりです。

Bootstrap 4 Bootstrap 5 件数
form-group mb-3 6
text-right text-end 4
text-left text-start 2
mr-1 / mr-2 / mr-3 me-1 / me-2 / me-3 6
ml-2 ms-2 1

index.twig の検索フォームにある range クラスは Bootstrap のクラスではない(本体にも定義なし)ため、そのまま残しています。

テスト(Test)

  • PHPUnit: 18 tests / 47 assertions すべて成功(管理画面のレンダリングを含む)
  • 実画面で表示確認: レビュー一覧(検索フォーム・一覧・削除モーダル)/ レビュー編集 / プラグイン設定の各画面で、余白・右寄せ・モーダル開閉が本体管理画面と同じ見た目になることを確認

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

※ 変更は twig の class 属性のみで、PHP の変更はありません。

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

🤖 Generated with Claude Code

- form-group → mb-3 / text-right → text-end / text-left → text-start /
  mr-* → me-* / ml-* → ms-*
- 置き換え先は本体 4.4 の管理画面テンプレート (admin/Product/index.twig) の
  記法に合わせる

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c6f87636-0777-498b-8048-7a50e1493227

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant