-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add Typography component #890
base: main
Are you sure you want to change the base?
Conversation
1171fad
to
6e0c17b
Compare
j'ai peut être loupé un épisode, mais quel est le but exactement de ce composant ? |
Le but c'est d'avoir la gestion du font style à un seul endroits. bien sur à therme il faudra utiliser ce composant/les class css utilitaire par tout ou il y aura du text |
|
||
### Prerequisite | ||
|
||
To use the typography, you must install the font "Source Sans Pro" in your project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On importe la font dans l'index.ts a la racine pour éviter que les utilisateurs aient a faire ca.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oui dans le package react, mais pas dans le package css qui n'a pas la font en dépendance. Il est donc nécessaire de mettre dans la documentation le fait de devoir installer la font.
j'ai du mal a voir l’intérêt de la PR au delà des headings, sachant qu'en plus on a deja un composant pour les titres. Quel est l'intéret pour le "copyright" par exemple qui n'est présent que dans le footer. |
Le but est de centraliser la gestion des styles de police décrits dans le DS ici en un seul endroit. Cela simplifiera la maintenance à long terme. Par exemple, si les UX décident de changer certaines tailles de police ou d'adapter les tailles de police en fonction de l'appareil (desktop ou tablette), nous n'aurons qu'à effectuer les modifications à un seul endroit, ce qui réduira le risque d'oublier un composant (idem côté consommateur). En effet, j'imagine qu'à terme, nous utiliserons tous ce composant au sein du toolkit, mais également les consommateurs dans leurs applications. |
6e0c17b
to
7db6e23
Compare
|
Cette PR à pour but d'implémenter les fonts style de slash, présent ici https://zeroheight.com/4b1e27a45/v/latest/p/9506de-fonts-style
elle permet d'ajouter des class css utilitaire afin d'avoir la bonne font style en fonction de la situation.
Coté react le composant Typography permet d'automatiser le travail.