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

feat: desaturate unapproved input token #289

Merged
merged 12 commits into from
Oct 24, 2022
Merged

feat: desaturate unapproved input token #289

merged 12 commits into from
Oct 24, 2022

Conversation

zzmp
Copy link
Contributor

@zzmp zzmp commented Oct 19, 2022

  • Simplifies approval codepaths, and includes approval in swap info state
    • I wanted to clean these codepaths, but don't think it is worth the effort to consolidate them further.
  • Renders an unapproved input token in grayscale

See DE-76. Replaces #215.

@zzmp zzmp requested review from JFrankfurt and a team October 19, 2022 00:26
@vercel
Copy link

vercel bot commented Oct 19, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
widgets ✅ Ready (Inspect) Visit Preview Oct 24, 2022 at 7:12PM (UTC)

Base automatically changed from skip-ux to main October 24, 2022 19:08
@vercel vercel bot temporarily deployed to Preview October 24, 2022 19:12 Inactive
Comment on lines +24 to +26
${TokenImg} {
filter: ${({ approved }) => approved === false && 'grayscale(1)'};
}
Copy link
Contributor

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:

Copy link
Contributor

@JFrankfurt JFrankfurt left a 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.

@zzmp zzmp merged commit be77a8d into main Oct 24, 2022
@zzmp zzmp deleted the simplify-approval branch October 24, 2022 21:31
@github-actions
Copy link

🎉 This PR is included in version 2.17.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants