Skip to content
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

perf (lazysizes): remove lazysizes, use native lazy attribute instead #433

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

firestar300
Copy link
Contributor

@firestar300 firestar300 commented Nov 5, 2024

Supprimer les occurrences de lazysizes.

Suite à cet article, il est recommandé d'utiliser les attributs natifs de lazyload en HTML plutôt que d'utiliser une librairie JavaScript.

Load the image using an element with the src or srcset attribute. Do not use non-standard attributes like data-src that require JavaScript in order to render, as that will always be slower. 9% of pages obscure their LCP image behind data-src.

@francoistibo
Copy link
Contributor

@asadowski10 discuter avec @firestar300 pour compatibilité avec ARI

@francoistibo
Copy link
Contributor

en attente d'un point avec Alexandre sur les changements ARI

@firestar300
Copy link
Contributor Author

Abandonner les fichiers .tpl et utiliser un fichier JSON pour la déclaration des data-location:

image

Puis créer un outil pour maj les JSON :

image

@firestar300 firestar300 force-pushed the perf/remove-lazysizes branch from 7464f7b to 964d17a Compare March 7, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants