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

feat: lazy load card images

Open totakoko opened this issue 1 year ago • 2 comments

cf https://github.com/codegouvfr/react-dsfr/issues/284

Ajoute loading="lazy" par défaut sur toutes les images des Card.

Je n'ai pas constaté de layout shift avec cette option (enfin je crois pas, j'ai testé avec réseau émulé 3G lent).

Par contre j'ai fait le strict minimum pour l'instant, cad aucune mention dans la doc. Je pourrai compléter demain.

totakoko avatar Jul 17 '24 16:07 totakoko

@garronej @ddecrulle Vous avez des avis sur ce qu'il manque pour intégrer ce changement ?

totakoko avatar Jul 22 '24 14:07 totakoko

Hello, personnellement je préfèrerais que ce soit paramétrable plutôt que le comportement par défaut. Je pense que le "layout shift" dépend de la taille de l'image ce que seul l'utilisateur de la librairie peut contrôler.

Par exemple ajouter la props : nativeImgProps

ddecrulle avatar Jul 22 '24 14:07 ddecrulle

La suite de cette PR ici :point_right: https://github.com/codegouvfr/react-dsfr/pull/294

totakoko avatar Sep 04 '24 14:09 totakoko