Skip to content

fix(frontend): importe chart.js sélectivement au lieu de chart.js/auto - #30

Merged
gilmry merged 1 commit into
mainfrom
fix/chartjs-tree-shaking
Aug 20, 2026
Merged

gilmry merged 1 commit into
mainfrom
fix/chartjs-tree-shaking

Conversation

@gilmry

@gilmry gilmry commented Aug 20, 2026

Copy link
Copy Markdown
Owner

Résumé

chart.js/auto enregistre tous les contrôleurs/échelles/plugins de Chart.js (radar, bulles, scatter, filler, decimation...) alors que le dashboard n'utilise que line/bar/pie. N'enregistrer que ce qui sert : 212774 → 182160 octets (-14%), 63,4 Ko une fois compressé (gzip), sans changement de comportement.

Test plan

  • npx svelte-check : 0 erreur
  • Vérifié en dev : 4 graphiques toujours rendus à l'identique, 0 erreur console, capture comparée
  • Suite e2e complète
  • CI

chart.js/auto enregistre tous les contrôleurs/échelles/plugins que
Chart.js embarque (radar, bulles, scatter, filler, decimation...),
alors que le dashboard n'utilise que line/bar/pie. N'enregistrer que
ce qui sert réduit le bundle de 212774 à 182160 octets (-14%), sans
changement de comportement.

Vérifié en dev : 4 graphiques toujours rendus à l'identique (courbes,
barres, camembert, titres, légendes, tooltips), 0 erreur console,
capture d'écran comparée. Suite e2e complète.
@gilmry
gilmry merged commit ef51104 into main Aug 20, 2026
2 checks passed
@gilmry
gilmry deleted the fix/chartjs-tree-shaking branch August 20, 2026 06:49
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