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

フロント側アラートメッセージが長い場合の崩れ修正

Open tatsuishi opened this issue 11 months ago • 2 comments

概要(Overview・Refs Issue)

フロント側アラートメッセージが長い場合の崩れ修正 Issue番号 カート メッセージが長いとアイコンが改行されてしまう #6092

方針(Policy)

メッセージが長いとアイコンが改行して表示されてしまいます(商品名が長い場合に発生しやすくなります)。 短い場合も長い場合も同様のデザインになるよう修正を行いました。

修正前 image

修正後 image

実装に関する補足(Appendix)

Issueにて報告したカートページ以外にも同様のアラートメッセージを表示している箇所があったため 合わせて修正いたしました。 ・カートページ ・お届け先選択画面 ・注文確認画面

テスト(Test)

各ページにて実際アラートを表示させ、崩れがないことを確認しました

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

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

レビュワー確認項目

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

tatsuishi avatar Feb 26 '24 03:02 tatsuishi

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.

codecov-commenter avatar Feb 26 '24 03:02 codecov-commenter

@tatsuishi PRありがとうございます👍 以下のテストが落ちてしまっていました。こちら対応して頂くことはできますか? もし難しい場合はお声がけください。 https://github.com/EC-CUBE/ec-cube/actions/runs/8043291350/job/21965143760?pr=6102

dotani1111 avatar Feb 26 '24 05:02 dotani1111

@dotani1111 お待たせしました。テストが通ったようですので、改めてよろしくお願いいたします。

tatsuishi avatar Feb 27 '24 01:02 tatsuishi

動作確認は問題ありませんでした。 ボックスの色も変わりませんでした。 スクリーンショット 2024-03-04 10 33 57

ji-eunsoo avatar Mar 04 '24 01:03 ji-eunsoo