-
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
[OUDS] feat: Add "Elevation" tokens, utilities and documentation - new version #2741
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…into ouds/main-his-tokens-elevation
The comment has been split in 2 lines because Hugo doesn't generate a p tag if the line starts with an html comment
scss/tokens/_semantic.scss
Outdated
$ouds-elevation-spread-focus: $ouds-elevation-spread-300 !default; | ||
|
||
$ouds-elevation-color-none: rgba(0, 0, 0, 0) !default; | ||
$ouds-elevation-color-raised: rgba(0, 0, 0, .32) !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.
We'll need to be careful here when we'll tackle the colors, as it doesn't work for the dark mode actually.
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.
Looks great! Good job!
Note: Please transform
- [ ]
into- (NA)
in the description when things are not applicableRelated issues
Closes #2596
Description
To add in the main issue:
site/content/docs/0.0/utilities/shadows.md
when we'll have components and/or$enable-shadows
site/content/docs/0.0/layout/z-index.md
when some components are present (navigation, tooltips, popovers, etc.)Motivation & Context
Types of change
Live previews
Checklist
Contribution
Accessibility
Design
Development
Documentation
Checklist (for Core Team only)
After the merge