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

Add Typography component #890

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Typography component #890

wants to merge 1 commit into from

Conversation

pplancq
Copy link
Contributor

@pplancq pplancq commented Feb 17, 2025

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.

msedge_WFDAc18n1E

@pplancq pplancq added agent-slash Component for the agent theme component component needs to be worked on labels Feb 17, 2025
@pplancq pplancq self-assigned this Feb 17, 2025
@pplancq pplancq force-pushed the feature/slash-typography branch from 1171fad to 6e0c17b Compare February 17, 2025 16:33
@MartinWeb
Copy link
Contributor

j'ai peut être loupé un épisode, mais quel est le but exactement de ce composant ?

@pplancq
Copy link
Contributor Author

pplancq commented Feb 18, 2025

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.
Comme cela si nos UX vienne a changer la font style du h2, on a juste à changer une fois est tous nos composants seront a jour.

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.
Copy link
Contributor

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.

Copy link
Contributor Author

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.

@JLou
Copy link
Contributor

JLou commented Feb 19, 2025

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.

@pplancq
Copy link
Contributor Author

pplancq commented Feb 19, 2025

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.

@pplancq pplancq force-pushed the feature/slash-typography branch from 6e0c17b to 7db6e23 Compare February 26, 2025 09:39
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
56.9% Coverage on New Code (required ≥ 80%)
7.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-slash Component for the agent theme component component needs to be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants