feat(client): converge Link text to Body 2 SB typography (#1753)#1793
feat(client): converge Link text to Body 2 SB typography (#1753)#1793
Conversation
Debaerdm
changed the title
There was a problem hiding this comment.
@Debaerdm Les changements que tu as fait sont incorrects.
Tu as supprimé du style client uniquement alors qu'il est correct, c'est le style apollo qui a besoin d'être uniformisé par rapport à la partie client.
Voici la dernière consigne de ce ticket "Le client est déja bon avec une font 18 en desktop (Body 2 desktop) et 16 en mobile (Body 2 Mobile). Seul la partie prospect est a corriger pour être de la même taille que la version client on passe tout en Body 2 desk et mobile."
Pour uniformiser les style apollo et client (LF) tu peux directement t'occuper du style common qui est la base de style d'apollo et client (LF)
…pr skill Add section "Conventions apprises" with 11 rules extracted from PR reviews: CSS Common over Apollo/LF duplication, classModifier for variants, getClassName mandatory, MDX in English, no invented token values, focus-visible vs focus, one story per variant, argTypes/decorators in meta. Sources: PRs #1793 #1801 #1807 #1805 #1783 #1792 #1765 #1766 #1775 #1771
|
Retours appliqués dans le commit 2db53ec :
Résultat : Apollo (prospect) hérite maintenant de Body 2 via le common, en iso avec le client/LF. Made with Claude |
GuillaumeKESTEMAN
left a comment
GuillaumeKESTEMAN
left a comment
There was a problem hiding this comment.
Les changements que tu as fait côté sont correct.
Maintenant qu'apollo et client ont la même font size pour .af-link, tu peux enlever la surcharge qu'il y a sur client concernant ce style.
D'ailleurs même plus besoin de token --link-font-size je pense vu qu'il n'y a plus de surcharge de font-size null part pour ce composant.
|
Fait dans le commit 5d37723 :
Made with Claude |
|
4 participants
Aligne la typographie du Link (Client) sur Body 2 SB (16px / 18px desktop, weight 600) en retirant la surcharge
--link-font-weight: 400et la règle.af-link--openInNewTab { --link-font-weight: 600 }qui devient redondante avec la nouvelle valeur par défaut héritée deLinkCommon.Mise à jour suite à l'évolution du ticket — la spec cible est Body 2 SB, pas Body 3 SB.
Closes #1753
Made with Claude