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

[FEATURE] Remise à jour des typographies des titres (PIX-9969) #497

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Nov 15, 2023

🎄 Problème

On souhaite mettre à jour les typographies utilisées pour nos titres de la police Open Sans vers la police Nunito. Les détails des variantes de typos sont sur Figma : https://www.figma.com/file/8RJ3aCSfdeQ8AZZVBBYKS8/Design-System-Pix?type=design&node-id=2598-16269&mode=design&t=WFMIRmEDI58shetS-0.

🎁 Proposition

Pour avoir le maximum d'impact, remplacer directement les valeurs données aux Design Tokens de nos titres, via les placeholders SCSS et classes CSS.

Par soucis de rétrocompatibilité, on conserve la possibilité de récupérer Open Sans via Pix UI mais on note que ça sera déprécié. Je pense donc que ça peut être une montée de version mineure ou patch. La suppression d'Open Sans sera une montée de version majeure. On pourra peut être en profiter pour supprimer les line-height qui posent problème quand les fonts sont utilisées dans d'autres contextes des paragraphes de texte.

On n'expose pas plus de variables "privées" de Pix UI. L'objectif des Design Tokens est de pouvoir correspondre aux cas d'usages le plus fidèlement possible.

On met à disposition 3 variables CSS pour ajuster le font-weight quand c'est nécessaire, en complément des 2 classes utilitaires déjà existantes (rien ne change sur leurs valeurs). On a utilisé en interne des variables CSS pour simplifier le code :)

🌟 Remarques

Je ne sais pas comment tester la fallback font, j'ai l'impression qu'elles ne sont pas utilisées car le loader Ember attend le chargement de notre font.

🎅 Pour tester

Dans Pix UI :

  • Vérifier que la documentation est bien à jour
  • Vérifier que la nouvelle police est bien utilisée dans la doc
  • Vérifier que la nouvelle police est bien utilisée dans les composants : PixIndicatorCard, PixModal et PixSidebar.

Dans les différentes applications qui l'utilisent :

@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr497.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr497/environment

@yannbertrand yannbertrand force-pushed the replace-title-font-with-nunito branch 2 times, most recently from 2be3459 to e048547 Compare November 15, 2023 08:47
@yannbertrand yannbertrand force-pushed the replace-title-font-with-nunito branch 2 times, most recently from 649f7e4 to f9e5ae9 Compare November 16, 2023 19:32
@yannbertrand yannbertrand removed the 🚧 Development in progress Work In Progress label Nov 17, 2023
@yannbertrand yannbertrand marked this pull request as ready for review November 17, 2023 16:18
@mcampourcy
Copy link
Contributor

mcampourcy commented Nov 20, 2023

Tests func :

  • MàJ de la doc OK
  • La doc Storybook est toujours en Nunito Sans chez moi, je regarde si on peut la dégager (si c'était bien le but ?)
  • Reste à faire : la connexion avec Certif

PixModal est bien passée en Nunito
Capture d’écran 2023-11-20 à 13 58 47

Les titres de PixIndicatorCard aussi :

Capture d’écran 2023-11-20 à 14 00 50

et PixSidebar aussi pour le texte "42" :

Capture d’écran 2023-11-20 à 13 59 53

@yannbertrand
Copy link
Member Author

* La doc Storybook est toujours en `Nunito Sans` chez moi, je regarde si on peut la dégager (si c'était bien le but ?)

Côté Storybook je propose de ne rien faire.

Quelques KO pour les components dans SB :

Qu'est-ce qui est KO ?

@mcampourcy
Copy link
Contributor

* La doc Storybook est toujours en `Nunito Sans` chez moi, je regarde si on peut la dégager (si c'était bien le but ?)

Côté Storybook je propose de ne rien faire.

Quelques KO pour les components dans SB :

Qu'est-ce qui est KO ?

Nope, c'était une erreur, pas de KO

@yannbertrand yannbertrand force-pushed the replace-title-font-with-nunito branch 2 times, most recently from 5d3ee58 to 4d4f6d3 Compare November 21, 2023 14:49
@github-actions github-actions bot force-pushed the replace-title-font-with-nunito branch from 4d4f6d3 to 02e85a0 Compare November 21, 2023 15:00
@github-actions github-actions bot merged commit 8bceb48 into dev Nov 21, 2023
8 of 9 checks passed
@github-actions github-actions bot deleted the replace-title-font-with-nunito branch November 21, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants