基本設定 店名の修正

[nobuhiko]

概要(Overview・Refs Issue)

#5656

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5657 (493f1da) into 4.2 (16561c7) will decrease coverage by 0.08%. The diff coverage is n/a.

@@             Coverage Diff              @@
##                4.2    #5657      +/-   ##
============================================
- Coverage     78.82%   78.73%   -0.09%     
- Complexity     6260     6269       +9     
============================================
  Files           469      468       -1     
  Lines         21010    21024      +14     
============================================
- Hits          16561    16554       -7     
- Misses         4449     4470      +21     

Flag	Coverage Δ
E2E	64.75% <ø> (-0.02%)	:arrow_down:
Unit	77.49% <ø> (-0.04%)	:arrow_down:

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

Impacted Files	Coverage Δ
src/Eccube/Form/Type/Admin/ShopMasterType.php	100.00% <ø> (ø)
src/Eccube/Service/MailService.php	82.26% <0.00%> (-8.08%)	:arrow_down:
...orage/Handler/SameSiteNoneCompatSessionHandler.php	76.11% <0.00%> (-2.99%)	:arrow_down:
...ube/Controller/Admin/Order/CsvImportController.php	85.55% <0.00%> (-1.81%)	:arrow_down:
...ontroller/Admin/Product/ProductClassController.php	90.28% <0.00%> (-0.58%)	:arrow_down:
...ype/Admin/OrderItemForShippingRegistrationType.php
...Eccube/Controller/Admin/Content/FileController.php	84.86% <0.00%> (+0.06%)	:arrow_up:
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	84.21% <0.00%> (+1.75%)	:arrow_up:
.../Service/PurchaseFlow/Processor/PointProcessor.php	95.83% <0.00%> (+1.95%)	:arrow_up:

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

[chihiro-adachi]

@nobuhiko 他の画面でも同様の事象になるようで、issueにも記載をしておきました。 こちらの修正だと不十分かなと思います。

[nobuhiko]

全部のformにnovalidate属性をつけないとだめですか

[chihiro-adachi]

@nobuhiko すべてのformにnovalidate属性つけるのは影響が大きすぎるかと。 スクロールが発生する場合だけhtml5のエラー表示が行われないようなので、その原因の特定と修正ができるとよいかなと思いました。

[nobuhiko]

@chihiro-adachi 原因は @media (prefers-reduced-motion: no-preference) :root { scroll-behavior: smooth; } みたいです root ではなく、bodyだけにscroll-behavior: smoothがあれば表示されるようです。がこれはこれでって感じの見た目になります

[ji-eunsoo]

@nobuhiko 適切な修正方法が見つからないため、一旦close致します。