-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore(design-tokens): link alias tokens to component tokens #871
Conversation
Tous les problèmes soulevés dans les discussions ont été adressés! Je crois qu'on est proche du green light! 🟢 |
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.
LGTM
un petit detail à corriger, mais rien en lien avec vos tokens
/** | ||
* Use for backgrounds of elements in an empty state, | ||
* such as carousel dots, progress elements, etc. | ||
*/ | ||
'color-bg-empty': 'color-neutral-15', | ||
'color-background-empty': 'color-neutral-15', | ||
/** | ||
* Use for the background of elements in a selected state, | ||
* such as naviaation elements like pagination pages or items like table rows. |
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.
petit detail mais il y a une typo dans navigation: * such as naviaation elements like pagination pages or items like table rows.
@meriouma @pylafleur @savutsang Je crois que nous serions good, si l'un d'entre vous pouvait faire une passe, nous pourrions merger et avancer vers de nouvelles aventures. |
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.
Je pense que c'est good! On va voir à l'utilisation. Let's go theme-directory!
DS-1120
Description
Ici on vient assigner les alias tokens à tous les component tokens.
Tests fonctionnels
Travail en collaboration avec @maboilard