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

fix: token select hover ux #7

Merged
merged 4 commits into from
May 11, 2022
Merged

fix: token select hover ux #7

merged 4 commits into from
May 11, 2022

Conversation

zzmp
Copy link
Contributor

@zzmp zzmp commented May 6, 2022

  • Scrolls to the hovered currency when the tokens change (eg through querying)
  • Fixes state usage to prevent spurious renders

@vercel
Copy link

vercel bot commented May 6, 2022

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

Name Status Preview Updated
widgets ✅ Ready (Inspect) Visit Preview May 9, 2022 at 11:10PM (UTC)

@zzmp zzmp requested a review from JFrankfurt May 6, 2022 17:55
@vercel vercel bot temporarily deployed to Preview May 6, 2022 17:56 Inactive
zzmp added 2 commits May 6, 2022 11:12
- Scrolls to the hovered currency when the tokens change (eg through querying)
- Fixes state usage to prevent spurious renders
@zzmp zzmp force-pushed the token-select-persistence branch from 86c0ef5 to efe7b66 Compare May 6, 2022 19:51
@vercel vercel bot temporarily deployed to Preview May 6, 2022 19:53 Inactive
@zzmp zzmp marked this pull request as draft May 9, 2022 19:36
@zzmp zzmp marked this pull request as ready for review May 9, 2022 23:09
@vercel vercel bot temporarily deployed to Preview May 9, 2022 23:10 Inactive
@zzmp zzmp merged commit 47007b6 into main May 11, 2022
@zzmp zzmp deleted the token-select-persistence branch May 11, 2022 19:28
@github-actions
Copy link

github-actions bot commented Jun 1, 2022

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

tinaszheng added a commit that referenced this pull request Feb 25, 2023
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