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

Updated asset selector component #163

Open
grod220 opened this issue Nov 25, 2024 · 0 comments · May be fixed by #204
Open

Updated asset selector component #163

grod220 opened this issue Nov 25, 2024 · 0 comments · May be fixed by #204
Assignees

Comments

@grod220
Copy link
Contributor

grod220 commented Nov 25, 2024

New pair-selected view. Has a star the save favorites (in local storage).
Screenshot 2024-11-22 at 4 57 26 PM

First screen in modal
Screenshot 2024-11-22 at 4 59 35 PM

Likely will need to add a new backend endpoint to get top X pairs by volume for suggested list.

Manual selecting pair screen.
Screenshot 2024-11-25 at 11 51 08 AM
Should be sorted by priority score. Recent searches from local storage. After selecting first asset, it should be removed from second asset list.

Able to search for individual assets:
Screenshot 2024-11-25 at 11 52 34 AM

Confirmation screen.
Screenshot 2024-11-25 at 11 53 26 AM

Figma reference: https://www.figma.com/design/xWsVprkEgEihweVN9isrGQ/DEX-(Incl.-UI-Lib)?node-id=3658-145100&node-type=canvas&t=MYKb27akVH0wi7kK-0

@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra web Nov 25, 2024
@grod220 grod220 moved this to 📝 Todo in Labs web Nov 25, 2024
@grod220 grod220 added this to Labs web Nov 25, 2024
@VanishMax VanishMax moved this from 📝 Todo to 🏗 In progress in Labs web Dec 6, 2024
@VanishMax VanishMax self-assigned this Dec 6, 2024
@VanishMax VanishMax linked a pull request Dec 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🗄️ Backlog
Development

Successfully merging a pull request may close this issue.

2 participants