ec-cube
ec-cube copied to clipboard
フロント側アラートメッセージが長い場合の崩れ修正
概要(Overview・Refs Issue)
フロント側アラートメッセージが長い場合の崩れ修正 Issue番号 カート メッセージが長いとアイコンが改行されてしまう #6092
方針(Policy)
メッセージが長いとアイコンが改行して表示されてしまいます(商品名が長い場合に発生しやすくなります)。 短い場合も長い場合も同様のデザインになるよう修正を行いました。
修正前
修正後
実装に関する補足(Appendix)
Issueにて報告したカートページ以外にも同様のアラートメッセージを表示している箇所があったため 合わせて修正いたしました。 ・カートページ ・お届け先選択画面 ・注文確認画面
テスト(Test)
各ページにて実際アラートを表示させ、崩れがないことを確認しました
マイナーバージョン互換性保持のための制限事項チェックリスト
- [ ] 既存機能の仕様変更はありません
- [ ] フックポイントの呼び出しタイミングの変更はありません
- [ ] フックポイントのパラメータの削除・データ型の変更はありません
- [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.77%. Comparing base (
8e96ee9
) to head (f79de9a
). Report is 2 commits behind head on 4.3.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## 4.3 #6102 +/- ##
=========================================
Coverage 82.77% 82.77%
Complexity 6509 6509
=========================================
Files 479 479
Lines 26029 26029
=========================================
Hits 21546 21546
Misses 4483 4483
Flag | Coverage Δ | |
---|---|---|
E2E | 70.08% <ø> (ø) |
|
Unit | 79.57% <ø> (-0.04%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@tatsuishi PRありがとうございます👍 以下のテストが落ちてしまっていました。こちら対応して頂くことはできますか? もし難しい場合はお声がけください。 https://github.com/EC-CUBE/ec-cube/actions/runs/8043291350/job/21965143760?pr=6102
@dotani1111 お待たせしました。テストが通ったようですので、改めてよろしくお願いいたします。
動作確認は問題ありませんでした。
ボックスの色も変わりませんでした。