商品一覧・商品詳細ページの商品画像 imgタグに、alt属性を追加

[matsuoshi]

概要(Overview・Refs Issue)

フロントの商品一覧・商品詳細ページの商品画像 imgタグに、alt属性を追加する
alt属性の値は商品名としています

関連issue
#5052

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[codecov-commenter]

Codecov Report

Merging #5058 (63c0629) into 4.1-feature (60f8669) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             4.1-feature    #5058   +/-   ##
==============================================
  Coverage          76.38%   76.38%           
  Complexity          6190     6190           
==============================================
  Files                445      445           
  Lines              20963    20963           
==============================================
  Hits               16013    16013           
  Misses              4950     4950           

Flag	Coverage Δ
tests	76.38% <ø> (ø)

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

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60f8669...63c0629. Read the comment docs.

[chihiro-adachi]

確認しました。
問題ないかと思います。

[k-yamamura]

商品詳細画面の以下の箇所ですが、

{% for ProductImage in Product.ProductImage %}
    <div class="slide-item"><img src="{{ asset(ProductImage, 'save_image') }}" alt="{{ Product.name }}"></div>
{% else %}
    <div class="slide-item"><img src="{{ asset(''|no_image_product, 'save_image') }}" alt="{{ Product.name }}" /></div>
{% endfor %}

画像が少なければ余り問題ありませんが、画像が多いと同一のalt内容が使われることになるので好ましくありません。
解決方法は1枚目だけにaltを入れるとかになりそうです。

サムネイル部分の

{% for ProductImage in Product.ProductImage %}
    <div class="slideThumb" data-index="{{ loop.index0 }}"><img src="{{ asset(ProductImage, 'save_image') }}" alt="{{ Product.name }}"></div>
{% endfor %}

は、 alt="" だけでも問題ないかと思います。

[matsuoshi]

@k-yamamura コメントありがとうございます！
ご指摘反映しまして、1枚目の商品画像 alt にのみ商品名を入れ、その他サムネイルは alt="" とするようにしました

@chihiro-adachi お手数ですが再度確認おねがいします！

[chihiro-adachi]

@matsuoshi
ありがとうございます、マージしました