info icon indicating copy to clipboard operation
info copied to clipboard

add basic keyboard navigation to search

Open JFrankfurt opened this issue 4 years ago • 3 comments

Came back to this after some sleep and it looks like it still needs work. Don't merge as is.

Looks like the lint:fix script added useEffect dependencies that broke the intended behavior.

JFrankfurt avatar Jul 15 '20 01:07 JFrankfurt

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/uniswap/uniswap-info/mi98n9ikt ✅ Preview: https://uniswap-info-git-fork-jfrankfurt-add-keyboard-search-nav.uniswap.vercel.app

vercel[bot] avatar Jul 15 '20 01:07 vercel[bot]

Thanks for working on this regardless - been on the to do list forever. Let me know you have questions

ianlapham avatar Jul 15 '20 18:07 ianlapham

@ianlapham finally came back to this. The vercel review app is still the initial commit though..

JFrankfurt avatar Sep 05 '20 08:09 JFrankfurt