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

MAJ bibliothèques de CSS et sécurité #478

Merged
merged 95 commits into from Jul 19, 2022
Merged

MAJ bibliothèques de CSS et sécurité #478

merged 95 commits into from Jul 19, 2022

Conversation

laem
Copy link
Contributor

@laem laem commented Mar 14, 2022

[Origine de la PR] : Honnêtement je ne suis plus à jour sur les paquets CSS à utiliser pour
Webpack. Il faut qu'on investigue ça. Le setup actuel date de bien avant
Webpack 5. MiniCSSextract semble avoir un autre rôle maintenatn.

Je me suis finalement lancé dans une grosse MAJ des dépendances. C'était bien nécessaire, à en juger par le nombre de changements.

Un travail de fond qui a pris beaucoup de retard, mais qui n'apportera aucune amélioration graphique. Juste une diminution significative du poids de l'app.

Et qui dit poids de l'app 📉 dit petite amélioration d'indexation google, et de rapidité d'affichage sur les puces peu puissantes.

Et c'est loin d'être fini encore.

image

  • chargement fainéant de publicodes-react, FAQ
  • useDatabase ?
  • mais aussi peut-être la conversation entière ? <- à voir dans une prochaine étape, on a bcp de taf sur le chargement des données avant
  • taille LogoADEME, ecolabclimat dessin (voir branche animée), Marianne, ABC
  • Supprimer le parser yaml de publicodes, PR lancée ici No yaml dependency publicodes/publicodes#243
  • suppression de react-markdown comme sur futureco
  • MAJ publicodes
  • l'appli compile, mais bug dans extractCategories. Changement cassant publicodes ?
  • re-vérifier le graphe Lighthouse pour voir ce qu'on charge en trop
  • réimplémenter les notes de bas de page en markdown
  • tester pour trouver des régressions
  • tester tout le parcours de questions, on a une MAJ publicodes substantielle
  • reprendre Corrige des erreurs de configuration #562

Tout cela fera déjà un énorme gain.

Ensuite

Honnêtement je ne suis plus à jour sur les paquets CSS à utiliser pour
Webpack. Il faut qu'on investigue ça. Le setup actuel date de bien avant
Webpack 5. MiniCSSextract semble avoir un autre rôle maintenatn.
@laem laem added the tech label Mar 14, 2022
@laem laem changed the title Dernière version de css-loader MAJ bibliothèques de CSS et sécurité Mar 14, 2022
@laem laem added this to En cours in Kanban Site Mar 14, 2022
@laem laem force-pushed the maj-lib-css branch 2 times, most recently from 0fa83d4 to bde9c79 Compare April 27, 2022 16:39
@@ -84,7 +86,7 @@ export const Markdown = ({
...otherProps.options,
forceBlock: true,
overrides: {
a: LinkRenderer,
a: noRouter ? undefined : LinkRenderer,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A quoi correspond noRouter ?

@laem laem mentioned this pull request Jul 19, 2022
@laem laem merged commit 8112cec into master Jul 19, 2022
@laem laem deleted the maj-lib-css branch July 19, 2022 10:48
@Clemog Clemog moved this from En cours to Fermé in Kanban Site Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants