-
Notifications
You must be signed in to change notification settings - Fork 77
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
Dark slider #2240
Dark slider #2240
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
I think this looks great!
@@ -76,6 +76,9 @@ const temporal = plugin( | |||
backgroundColor: css('--color-interactive-ghost-active'), | |||
}, | |||
}, | |||
'.surface-information': { | |||
backgroundColor: css('--color-surface-information'), |
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.
@GraceGardner just wanted to follow-up here before I make any changes! Was there a specific reason surface-information
was added here? Looks like we're just using backgroundColor
and that already exists ⬇️
Line 131 in 5f41480
information: css('--color-surface-information'), |
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.
I've gone ahead and removed it here.
Description & motivation 💭
Adjust Range input styling to match new designs.
Screenshots (if applicable) 📸
Design Considerations 🎨
Testing 🧪
How was this tested 👻
Steps for others to test: 🚶🏽♂️🚶🏽♀️
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
DT-2061
Docs
Any docs updates needed?