-
Notifications
You must be signed in to change notification settings - Fork 654
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CWVのCLS対応のために商品詳細画像のwidth、height追加 #5023
Conversation
@tao-s コンフリクト解消お願い致します |
Codecov Report
@@ Coverage Diff @@
## 4.1 #5023 +/- ##
=========================================
Coverage 68.34% 68.34%
Complexity 6070 6070
=========================================
Files 456 456
Lines 24946 24946
=========================================
Hits 17050 17050
Misses 7896 7896
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@tao-s
正方形ではない画像を登録されているサイトもあり(ec-cube.coで確認)、固定でのwidth/heightの指定だと既存サイトへの影響が大きそうです。 サーバサイドで事前に計算しておき、それをtwigで出力するようにするなどの実装が望ましいかもしれません。 |
あと標準でaltくらい出しとこうよ
概要(Overview・Refs Issue)
ref: #4807 #4901 #5022
Core Web VitalでCumulative Layout Shiftのスコアが悪くなるので、ブラウザがアスペクト比を計算するためにデフォルトの縦横サイズを入れとく。あと標準でaltくらい出しとこうよ
方針(Policy)
メイン画像の2目枚目以降はlazy loadに
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目