You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We can revisit this in a separate follow up PR, but I would suggest adding a variable to product-thumbnail to remove <modal-opener> structure from video and model medias.
I think it could be useful to change the logic to be based on the number of columns instead of the 1 used. This way we can tell the two/three first images to not lazy load.
The text was updated successfully, but these errors were encountered:
andrewetchen
changed the title
[Product thumbnail] Modal opener for right desktop media position
[Product thumbnail] Media position and lazy load fixes
Sep 8, 2022
andrewetchen
changed the title
[Product thumbnail] Media position and lazy load fixes
[Product thumbnail] Media position and lazy load
Sep 8, 2022
Captured improvements for
product-thumbnail
snippet:product-thumbnail
to remove the<modal-opener>
structure fromvideo
andmodel
medias since the right columns layout on mobile is never used.The text was updated successfully, but these errors were encountered: