-
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
Sord 564 update button #250
Conversation
124087e
to
a0e236e
Compare
marginHorizontal: size === 's' ? 16 : 22, | ||
fontSize: | ||
size === 's' | ||
? theme.sw.typography.button.n3!.fontSize |
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.
pourquoi le point d'exclamation a 'n3', il est potentiellement null ?
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.
Pour typer les typographies (button, headline, body et label) on utilise un même type et il y a des typographies qui s'arrête à n2 :-/. Mais pas button.
? (theme.sw.typography.button.n3!.fontWeight as never) | ||
: (theme.sw.typography.button.n1.fontWeight as never), |
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.
as never ?
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.
Oui, notre type ne match pas parfaitement celui attendu, mais fonctionne très bien... Il faudrait peut-être revoir les types retournés par la librairie de token (https://github.com/ZeroGachis/smartway-design-token)
No description provided.