Skip to content

Nettoyage des classes Tailwind redondantes avec les valeurs par défaut - #2375

Merged
Mopolo merged 1 commit into
afup:masterfrom
vgreb:front/cleanup-default-tailwind-classes
Aug 28, 2026
Merged

Nettoyage des classes Tailwind redondantes avec les valeurs par défaut#2375
Mopolo merged 1 commit into
afup:masterfrom
vgreb:front/cleanup-default-tailwind-classes

Conversation

@vgreb

@vgreb vgreb commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Suite à la review de #2370, où les trois suggestions portaient toutes sur le même point : les conversions Figma → Tailwind transcrivent littéralement font-family: Inter, font-weight: 400 et font-size: 16px, alors que ces valeurs sont déjà celles du body.

Le motif était présent sur 12 fichiers déjà mergés, d'où cette PR dédiée plutôt qu'une correction au coup par coup.

Ce qui est retiré

Classe Pourquoi c'est un no-op
font-sans --font-sans: "Inter" est déjà la police du body (assets/styles/fonts.css). Vérifié : font-titre n'est jamais posée sur un conteneur parent, uniquement sur l'élément lui-même.
font-normal Graisse 400 par défaut. Le preflight remet h1h6 à font-weight: inherit, donc même sur un titre la classe ne change rien. Vérifié : aucun conteneur des fichiers touchés n'impose une graisse.
text-base 1rem par défaut. Vérifié : aucun conteneur des fichiers touchés n'impose une autre taille.

Ce qui est conservé, et pourquoi

Toutes ces occurrences surchargent réellement quelque chose, elles sont laissées en place :

  • text-base et font-semibold passées à <twig:Button> — le composant a text-sm font-medium en base, et tailwind_merge donne la priorité aux attributs. Les retirer changerait le rendu des boutons. Idem pour le préfixe du bloc button_widget dans le thème de formulaire.
  • font-normal sur les libellés du thème de formulaire (tailwind.html.twig lignes 32, 61, 126) — écrase le font-medium de form_label.
  • text-base dans les bases de composants (Benefit, SectionTitle, Card/Subtitle) — y fixe l'échelle typographique du composant indépendamment du contexte, comme le text-sm de Button.
  • text-base sm:text-xl sur la page d'accueil — moitié basse d'une paire responsive volontaire (Réduction de la taille du texte sur mobile #2374).

Vérifications

  • Diff vérifié ligne à ligne : les 88 lignes modifiées ne diffèrent que par la suppression de ces trois classes, aucune autre modification.
  • bin/console lint:twig templates/[OK] All 328 Twig files contain valid syntax.
  • Aucun changement de rendu attendu.

@vgreb
vgreb force-pushed the front/cleanup-default-tailwind-classes branch from d6a51e1 to ebf9474 Compare August 27, 2026 20:49
@Mopolo
Mopolo merged commit 7f9c85d into afup:master Aug 28, 2026
7 checks passed
@vgreb
vgreb deleted the front/cleanup-default-tailwind-classes branch August 28, 2026 22:51
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.

2 participants