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

chore: update tokens #2766

Closed
wants to merge 2 commits into from

Conversation

boosted-bot
Copy link
Collaborator

This PR is generated automatically, it updates the tokens based on Figma Variables.

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 3f90b98
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/67172f713445cf00082e7432
😎 Deploy Preview https://deploy-preview-2766--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@julien-deramond
Copy link
Contributor

Pushed 3f90b98 to:

  • Ignore tokens files temporarily because at least the semantic file contains tokens not yet handled by OUDS Web
  • Fix the Sass tests because some calculated values are now:
    • 0px instead of 0
    • "dashed" instead of dashed

@julien-deramond julien-deramond added this to the OUDS milestone Oct 22, 2024
Copy link
Member

@louismaximepiton louismaximepiton left a 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;
Copy link
Member

Choose a reason for hiding this comment

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

$ouds-border-style-dashed: "dashed" !default;
// $ouds-border-style-dotted: "dotted" !default;
// $ouds-border-style-none: "none" !default;
$ouds-border-style-solid: "solid" !default;
Copy link
Member

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
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
$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 ?

@julien-deramond julien-deramond removed this from the OUDS milestone Oct 22, 2024
@julien-deramond julien-deramond deleted the tokenator-update-tokens-20241022064859 branch October 22, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants