Skip to content

Rendre la définition du style cohérente en supprimant les withStyles

Benoît Courty requested to merge with-styles into master

Created by: LoicPoullain

Cette PR fait un nettoyage du code avant que je quitte le navire 🚢 .

Elle supprime l'ensemble des appels à withStyles et déplace tout le style dans des fichiers SCSS séparés. Cela permet de rendre le code cohérent et anticipe le nettoyage de l'initialization de l'application rendant npm run dev plus rapide.

L'objectif de la PR n'est pas de revoir le code en entier, ce qui est serait ambitieux étant donné les délais (compte-tenu du nombre du jours qu'il me reste) et le fait qu'il s'agit simplement de style.

Les parties de l'application touchées sont listées ci-dessous. Il faudrait en gros vérifier que le style (qui n'est pas amélioré) est le même entre Scalingo et https://leximpact.an.fr.

  • la fenêtre "vous êtes déconnecté."
  • la fenêtre "ajout d'un cas type"
  • les cartes IR "cas types"
  • la fenêtre "vous êtes connecté"
  • la carte "gagnant-perdant"
  • la carte "budget de l'Etat" avec son tableau
  • le tooltip gris quand je survole un élément
  • la fenêtre de connexion

PS : Le seul truc qui change un peu, c'est la popin qui a un padding-top un peu plus élevé des fois, mais je pense que c'est acceptable.

PS 2 : Quelques tests ont été temporairement enlevés, le temps que je trouve comment régler le problème d'import CSS/SCSS dans des fichiers Jest (sinon je suis complètement bloqué pour toute la suite ...).

Merge request reports