-
Notifications
You must be signed in to change notification settings - Fork 54
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: update tokens #2766
chore: update tokens #2766
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Pushed 3f90b98 to:
|
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.
Some few structuring things to tackle before merging it
$ouds-border-radius-9999: 2000px !default; | ||
$ouds-border-style-dashed: "dashed" !default; |
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.
This doesn't work anymore, see: https://deploy-preview-2766--boosted.netlify.app/docs/0.0/utilities/borders/#drag-and-drop
$ouds-border-style-dashed: "dashed" !default; | ||
// $ouds-border-style-dotted: "dotted" !default; | ||
// $ouds-border-style-none: "none" !default; | ||
$ouds-border-style-solid: "solid" !default; |
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.
Same here I think, it's just the default so it looks good for now.
// $ouds-border-style-dotted: "dotted" !default; | ||
// $ouds-border-style-none: "none" !default; | ||
$ouds-border-style-solid: "solid" !default; | ||
$ouds-border-radius-0: $ouds-border-base * 0 !default; // 0 |
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.
$ouds-border-radius-0: $ouds-border-base * 0 !default; // 0 | |
$ouds-border-radius-0: strip-units($ouds-border-base * 0) !default; // 0 |
Is there any way to do it on Tokenator side ? I mean when it's * 0
?
This PR is generated automatically, it updates the tokens based on Figma Variables.