ec-cube
ec-cube copied to clipboard
admin:tableの背景色修正
概要(Overview・Refs Issue)
tableの背景色が違う #5601
- admin→商品一覧のテーブルがBootstrap5の仕様でうっすらグレーになっている
- 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 テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
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.
PRありがとうございます。 今後4.3では、新しいバージョンに合わせていきたいと思うため、Closeします。