Skip to content

Conversation

@pierrepougetpix
Copy link
Contributor

🎄 Problème

La font appliquée sur la version large est également sur la version small mais le rendu visuel n'est pas optimal

🎁 Proposition

Appliquer le token font body-s (0.875rem)

🌟 Remarques

Le token body-s semblait déjà être appliqué mais ne fonctionne pas, j'ai le souvenir qu'à l'époque de la construction du composant, il était question d'utiliser la propriété CSS en font-size directement.

🎅 Pour tester

Vérifier que la font est à 0.875rem en version small

MathieuGilet and others added 7 commits May 9, 2025 16:32
Merge pull request #4 from 1024pix/MathieuGilet-patch-4
Merge pull request #5 from 1024pix/MathieuGilet-patch-5
## [55.17.2](v55.17.1...v55.17.2) (2025-05-09)

### 🐛 Correction

- [#4](#4) Update README.md
- [#5](#5) Update release.yml
@pierrepougetpix pierrepougetpix self-assigned this May 12, 2025
@pix-bot-github
Copy link

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

font-weight: var(--pix-font-bold);

@extend %pix-body-s;
font-size: 0.875rem;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

le pix body s est déjà en 0.875 🤔

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.

6 participants