Skip to content

Supprimer les erreurs de console restantes du frontend - #404

Open
DnzzL wants to merge 2 commits into
chore/pnpm-12from
fix/console-errors
Open

DnzzL wants to merge 2 commits into
chore/pnpm-12from
fix/console-errors

Conversation

@DnzzL

@DnzzL DnzzL commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Six familles d'erreurs restaient dans la console du navigateur. Je les ai relevées en instrumentant le journal de next dev, qui relaie chaque message du navigateur avec le préfixe [browser]. Le départ comptait 193 messages sur seize pages ; il en reste dix-neuf, tous du même type et visibles en développement seulement.

BaseCard étalait ses propriétés sur son lien, ce qui faisait apparaître dans le HTML des attributs qui n'y ont rien à faire, rawDate et onSite entre autres. La bannière d'une page thématique absente du CMS était par ailleurs transmise à next/image sous forme de chaîne vide : ThematicHeroBlock ne rend plus l'image quand elle manque, à la manière d'EditoCard.

Le défaut le plus visible était ailleurs. transformPartners transmettait l'objet média de Strapi au lieu de son adresse, si bien que la carte du financeur s'affichait sans aucune image. Le logo de « La Concorde » est de nouveau visible sur /democratie.

Les balises script du layout passent par next/script. J'ai essayé beforeInteractive, puis renoncé : chargé avant le code de Next, le script iRaiser détourne console.error et casse la lecture du flux serveur, ce qui fait échouer les navigations. afterInteractive suffit.

Enfin, six couples width/height ne correspondaient plus à la boîte rendue, et deux index d'animation n'étaient pas bornés : avec deux images seulement, /nos-positions plantait.

Vérifications

La géométrie de /democratie est identique au pixel avant et après, et le contenu des seize URL de référence n'a pas bougé. Les 250 tests passent, le build aussi, et le lint ne bouge pas d'un point : 293 problèmes avec comme sans ces modifications.

Ce que cette branche ne corrige pas

Les dix-neuf messages restants concernent des images dont la boîte est volontairement contrainte en CSS : les portraits du carrousel, les bannières CMS, la puce dot-purple. Les faire taire demanderait soit de déclarer des dimensions fausses, soit de changer la conception, soit de passer en fill avec un parent positionné. Aucun effet sur le CLS, mesuré sous 0,015.

@DnzzL
DnzzL added this pull request to stack #400 October 7, 2026 17:43
DnzzL and others added 2 commits October 7, 2026 20:21
`BaseCard` laissait fuiter ses props dans le DOM (`rawDate`, `onSite`) et la
banniere des pages thematiques sans image recevait un `src` vide. Le logo des
financeurs recevait l'objet media au lieu de son URL, donc aucune image ne
s'affichait. Les scripts passent par next/script, six ratios width/height
correspondent a nouveau a la boite rendue, deux index d'animation sont bornes.
Animations : un seul acces optionnel par image au lieu d'une garde de longueur repetee.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@DnzzL
DnzzL force-pushed the fix/console-errors branch from 1709085 to 3d51429 Compare October 7, 2026 18:25

This branch has not been deployed

No deployments
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.

1 participant