[WIP] Fix Lighthouse scores below acceptable threshold#261
Conversation
|
|
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
DavidKRK
left a comment
There was a problem hiding this comment.
✅ Approve — Optimisations performances & nettoyage du code
Cette PR apporte des améliorations solides et bien ciblées :
Suppression de Howler.js et de tout son code associé : nettoyage bienvenu, la librairie n'était plus utilisée activement.
Fix du sélecteur de langue : la classe active est désormais posée directement dans le HTML (FR par défaut), ce qui élimine le risque de flash visuel (FOUC) au chargement. Bonne pratique.
Optimisations LCP sur index.html : fetchpriority="high" sur l'image principale + preload de l'image hero.
Iframes Mixcloud : suppression de l'autoplay + ajout de loading="lazy". Bon pour les perfs et l'expérience utilisateur.
Première vignette YouTube dans music.html : loading="eager" fetchpriority="high" — correct pour le LCP.
responsive.css retiré de index.html : vérifié — le fichier n'existe pas dans le repo, c'était une référence morte. ✅
Seuil Lighthouse performance relevé de 0.5 à 0.6 : exigence plus stricte, cohérent avec les optimisations.
Légère incohérence d'indentation dans shop.html (non bloquant).
→ Approuvé ✅
|
@copilot resolve the merge conflicts in this pull request |
…use-performance-seo # Conflicts: # contact.html # index.html # music.html
Les conflits de merge ont été résolus et intégrés dans le commit |
Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.