-
Notifications
You must be signed in to change notification settings - Fork 190
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
feat: desaturate unapproved input token #289
Conversation
This reverts commit 0c9cf78.
Co-authored-by: Kristie Huang <kristiehuang01@gmail.com>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
${TokenImg} { | ||
filter: ${({ approved }) => approved === false && 'grayscale(1)'}; | ||
} |
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 prefer to have some kind of explicit unset value here instead of creating a floating invalid filter
rule: filter:
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.
lgtm--This would have been much easier to review as two PRs though. The actual desaturation code is very small, and I had to kinda figure out what all the rest of the code was doing on my own.
🎉 This PR is included in version 2.17.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
See DE-76. Replaces #215.