ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

admin:tableの背景色修正

Open riki-develop opened this issue 2 years ago • 1 comments

概要(Overview・Refs Issue)

tableの背景色が違う #5601

  1. admin→商品一覧のテーブルがBootstrap5の仕様でうっすらグレーになっている
  2. tbodyのborder-topの色がEC-CUBE4.1と異なる(黒になっている)

style.scssでインポートされているBootstrap5
@import "../../../../../node_modules/bootstrap/scss/bootstrap";
↓
.table > :not(caption) > * > * {
    padding: 0.5rem 0.5rem;
    background-color: var(--bs-table-bg);
    border-bottom-width: 1px;
    box-shadow: inset 0 0 0 9999px var(--bs-table-accent-bg);
}

が設定される
↓
_bootstrapOriginal.scss で【box-shadow】と【border-color】を上書き

テスト(Test)

src/Eccube/Resource/template/admin/Product/index.twig 【/admin/product】画面にてレイアウト確認

▼開発環境(M1 Mac:Docker)

EC-CUBE:4.2.0-beta2
DB:MySQL 5.7.39
Apache/2.4.54 (Debian)
PHP:7.4.30

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

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

レビュワー確認項目

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

riki-develop avatar Aug 30 '22 08:08 riki-develop

Codecov Report

Merging #5702 (d1f3812) into 4.2 (30e854f) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.2    #5702   +/-   ##
=========================================
  Coverage     78.83%   78.83%           
  Complexity     6273     6273           
=========================================
  Files           469      469           
  Lines         21058    21058           
=========================================
  Hits          16602    16602           
  Misses         4456     4456           
Flag Coverage Δ
E2E 64.82% <ø> (ø)
Unit 77.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/Eccube/Entity/TaxRule.php 85.18% <0.00%> (-1.24%) :arrow_down:
...be/Service/PurchaseFlow/Processor/TaxProcessor.php 84.21% <0.00%> (+1.75%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 30 '22 09:08 codecov-commenter

PRありがとうございます。 今後4.3では、新しいバージョンに合わせていきたいと思うため、Closeします。

ji-eunsoo avatar Feb 14 '24 07:02 ji-eunsoo