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] Utiliser les designs tokens sur Mon Pix (PIX-15177). #10807

Merged
merged 6 commits into from
Jan 2, 2025

Conversation

Libouk
Copy link
Member

@Libouk Libouk commented Dec 13, 2024

🎄 Problème

Les design tokens ne sont pas encore utilisés partout sur Mon Pix. On cherche à répandre leur usage.

🎁 Proposition

Remplacer les attributs CSS de certaines classes par les Design Tokens définis dans le Design System.

🧦 Remarques

Travail partiel issu d'un point avec Quentin C. D'autres points seront à prévoir.

🎅 Pour tester

  • Sur l'ensemble des pages impactées, vérifier que l'affichage n'est pas cassé.

@Libouk Libouk added 👀 Tech Review Needed 👀 Func Review Needed team-evaluation PR relatives à l'expérience d'évaluation labels Dec 13, 2024
@Libouk Libouk self-assigned this Dec 13, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

Copy link
Member

@AndreiaPena AndreiaPena left a comment

Choose a reason for hiding this comment

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

func review pour la page Certif ✅ (_certification-not-certifiable.scss)

Comment on lines +262 to +263
margin: 0 3px;
padding-top: 10px;
Copy link
Member

Choose a reason for hiding this comment

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

je comprends que le code a juste été déplacé mais suggestion : pourquoi ne pas utiliser les tokens ici ?

Copy link
Member

Choose a reason for hiding this comment

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

(ça s'applique à pas mal de fichiers modifiées j'ai l'impression même si tout n'a pas été touché)

Copy link
Member Author

Choose a reason for hiding this comment

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

C'est vrai qu'on n'a pas tout fait, j'étais en pair sur une séance à temps limité avec QuentinC et il avait ciblé des valeurs à changer. On a dû passer à côté d'autres endroits où les tokens pouvaient être utilisés

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15177-apply-design-token-to-pix-app branch from 620e210 to d185a89 Compare January 2, 2025 14:34
@pix-service-auto-merge pix-service-auto-merge merged commit eca93de into dev Jan 2, 2025
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15177-apply-design-token-to-pix-app branch January 2, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Review OK Func Review OK PO validated functionally the PR 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants