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
Auparavant on utilisait directement le markup HTML DSFR des Card pour lazy loader les images (grosse liste d'actus).
Avec react-dsfr, il n'y a pas d'attributs loading, donc toutes les images sont téléchargées.
Est-ce que ça vous irait si je fais une PR pour ajouter le paramètre loading="lazy" à l'élément img ?
Je dirais qu'on peut l'ajouter par défaut, mais si on veut être sûr de ne rien casser, le mettre derrière une propriété lazyLoadImage marche aussi.
Qu'en pensez-vous ?
The text was updated successfully, but these errors were encountered:
Oui top, par default ça me parais bien juste a voir pour la taille peut être.
Il faut s'assurer que ça marche correctement quand l'immage est en train de charger et qu'on a pas un mega layout shift
👋
Auparavant on utilisait directement le markup HTML DSFR des Card pour lazy loader les images (grosse liste d'actus).
Avec react-dsfr, il n'y a pas d'attributs loading, donc toutes les images sont téléchargées.
react-dsfr/src/Card.tsx
Lines 226 to 230 in 137c8a6
Est-ce que ça vous irait si je fais une PR pour ajouter le paramètre
loading="lazy"
à l'élément img ?Je dirais qu'on peut l'ajouter par défaut, mais si on veut être sûr de ne rien casser, le mettre derrière une propriété
lazyLoadImage
marche aussi.Qu'en pensez-vous ?
The text was updated successfully, but these errors were encountered: