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

ETQ dev, je souhaiterais que purgeCSS ne retire pas les classes CSS dont j'ai besoin #515

Open
Volubyl opened this issue Oct 24, 2022 · 1 comment
Assignees
Labels
bug Something isn't working front technical

Comments

@Volubyl
Copy link
Collaborator

Volubyl commented Oct 24, 2022

Contexte

En travaillant sur #509, je me suis rendu compte que le fix inclut dans #514 marchait en local mais pas sur staging.

En fouillant, je me suis rappelé que la manière dont on buildait l'app est différent entre les env de Prod (utilisant la variable d'env NODE_ENV=production) et l'env local.

En production, l'on minifie et l'on purge le CSS tandis qu'en local non

Etape pour reproduire

  1. Inclure le code supprimé sur cette PR HOTFIX: remove purgecss #516
  2. builder le client avec la commande npm run build
  3. Lancer le server avec make serve-server
  4. Lancer le client avec la commande cd client && node build
  5. 6.En local aller sur la page de modification ou de contribution.
    7.Modifier un champs texte, cliquer sur la croix
  6. Constater que rien ne se passe

D'où viendrait le problème ?

Certaines lib, dont le DSFR que l'on utilise, injecte du CSS via Javascript. Au moment du build, purge CSS analyse statiquement le code HTML à la recherche de classe CSS. Si il trouve des classes css, il va les inclure dans le css "purgé"

Si

À l'heure actuelle, je pense qu'il y a un problème de config avec purgecss

Pourquoi ?

Si l'on modifie le fichier vite.config.js et que l'on retire les lignes 17 à 29 et que l'on refait les étapes pour reproduire, il n'y a plus le problème.

Proposition de HOTFIX

Comme hotfix, je propose d'enlever purgeCSS le temps d'approfondir la recherche d'une solution plus pérenne

Voir : #516

@Volubyl Volubyl added bug Something isn't working front technical labels Oct 24, 2022
@Volubyl Volubyl added this to Backlog in Outil de catalogage de données via automation Oct 24, 2022
@Volubyl Volubyl moved this from Backlog to Exploration en cours in Outil de catalogage de données Oct 24, 2022
@Volubyl Volubyl self-assigned this Oct 24, 2022
@florimondmanca
Copy link
Collaborator

Il y a aussi make build && make serve-dist pour lancer le client en mode buildé et débugger en local (à vérifier si ça build bien en faisait un NODE_ENV=production)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working front technical
Projects
Development

No branches or pull requests

2 participants