feat(Theme): add text tokens #1051
Conversation
|
Storybook for this build: https://ds.equisoft.io/pr-1051/ |
|
Webapp for this build: https://ds.equisoft.io/pr-1051/webapp/ |
cc5a178 to
4e9038d
Compare
f844301 to
64c7bf8
Compare
4e9038d to
f1df41f
Compare
| import type { AliasTokenMap } from '../tokens'; | ||
|
|
||
| export type TextAliasToken = | ||
| | 'text-heading-xl-font-family' |
There was a problem hiding this comment.
On avait déjà eu la discussion je pense pour les valeurs de props qui sont des sizes, et on met small, medium large plutôt que sm md lg. Est-ce qu'on veut garder la même convention pour les tokens? Ou revoir la convention pour les props?
There was a problem hiding this comment.
J'ai pas vraiment de préférence, @maboilard ?
There was a problem hiding this comment.
On peut s'adapter en fonction de ce qui existe déjà donc si on utilise déjà small, medium large restons constant avec cette nomenclature
There was a problem hiding this comment.
J'ai fait le changement, j'ai aussi utilisé xsmall et xlarge pour xs et xl.
DS-1276
Ajouter les tokens pour les styles liés au texte. Ils ne sont pas utilisés pour l'instant alors il n'y a pas grand-chose à tester.
Figma pour les tokens: https://www.figma.com/design/EIJItUlyeIymb2dfgeoQJp/Design-System-UI-Library?node-id=4-4&node-type=canvas&m=dev