react-dsfr icon indicating copy to clipboard operation
react-dsfr copied to clipboard

Card: feat lazy loading des images

Open totakoko opened this issue 1 year ago • 1 comments

:wave:

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.

https://github.com/codegouvfr/react-dsfr/blob/137c8a638bc70b8b0aa3181c8a02dd5d8975702f/src/Card.tsx#L226-L230

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 ?

totakoko avatar Jul 17 '24 14:07 totakoko

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

garronej avatar Jul 17 '24 15:07 garronej

Résolu comme la PR #294 a été mergée.

totakoko avatar Sep 17 '24 14:09 totakoko

Merci

garronej avatar Sep 17 '24 14:09 garronej