Skip to content

feat: introduce $ouds-border-radius-9999 raw token and create a px-to-rem Sass function#2751

Merged
julien-deramond merged 4 commits into
ouds/mainfrom
main-jd-reintegrate-border-radius-9999
Oct 3, 2024
Merged

feat: introduce $ouds-border-radius-9999 raw token and create a px-to-rem Sass function#2751
julien-deramond merged 4 commits into
ouds/mainfrom
main-jd-reintegrate-border-radius-9999

Conversation

@julien-deramond
Copy link
Copy Markdown
Contributor

@julien-deramond julien-deramond commented Oct 2, 2024

Description

We finally decided to (re)introduce $ouds-border-radius-9999 that comes from Figma.

This value is set to 2000px so we needed to transform it to 125rem. Hence, the creation of a new px-to-rem Sass function, that will be useful for other use cases in the future.

Live previews

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 2, 2024

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit e1fe493
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/66fe4b3b3a49300008693fc1
😎 Deploy Preview https://deploy-preview-2751--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 julien-deramond force-pushed the main-jd-reintegrate-border-radius-9999 branch from df3ed04 to df0aca9 Compare October 2, 2024 10:04
@julien-deramond julien-deramond force-pushed the main-jd-reintegrate-border-radius-9999 branch from df0aca9 to f5a2c44 Compare October 2, 2024 10:08
@julien-deramond julien-deramond marked this pull request as ready for review October 2, 2024 10:10
@louismaximepiton louismaximepiton changed the title feat: introduce '$ouds-border-radius-9999 raw token and create a px-to-rem` Sass function feat: introduce $ouds-border-radius-9999 raw token and create a px-to-rem Sass function Oct 2, 2024
@louismaximepiton
Copy link
Copy Markdown
Member

louismaximepiton commented Oct 2, 2024

I've added a new strip-units function to allow people using it with or without unit. Feel free to better or revert these changes. (0e03ca3 and 2fa20a5)

Comment thread scss/tokens/_raw.scss Outdated
@julien-deramond julien-deramond merged commit 08b7004 into ouds/main Oct 3, 2024
@julien-deramond julien-deramond deleted the main-jd-reintegrate-border-radius-9999 branch October 3, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants