freighter icon indicating copy to clipboard operation
freighter copied to clipboard

Improve UX for manually add assets

Open JakeUrban opened this issue 1 year ago • 2 comments

"curious if we've thought about trying to get rid of the "add an asset" vs "add manually" choice in this flow? I've seen users get confused by this because they don't know when to search manually or how it's different from the regular search. Under the hood, one does an asset search and the other does either a contract id or issuer lookup depending on if you enter a G or C key. I think we can pretty much do the same thing while using one input by parsing the input into: asset code or contract id or public key and doing the search that matches the input type." - @aristidesstaffieri

JakeUrban avatar Jan 14 '25 22:01 JakeUrban

"curious if we've thought about trying to get rid of the "add an asset" vs "add manually" choice in this flow? I've seen users get confused by this because they don't know when to search manually or how it's different from the regular search. Under the hood, one does an asset search and the other does either a contract id or issuer lookup depending on if you enter a G or C key. I think we can pretty much do the same thing while using one input by parsing the input into: asset code or contract id or public key and doing the search that matches the input type." - @aristidesstaffieri

Yeah absolutely, this has been flagged as an improvement for a long time but never prioritized. There is no reason why we can't do this now. I think along with this, we should improve our ability to reject contracts that are not sep41 compliant. Right now we accept any valid contract ID.

aristidesstaffieri avatar Jan 14 '25 22:01 aristidesstaffieri

Designs here: https://www.figma.com/design/C3G0a4Gd6RQyplRBppGDsL/Freighter-(SDS-v3)?node-id=2932-3383&t=NNZIouOPrlmRWQS4-1

Notes:

  • remove "Add Manually" as a cta on Manage Assets
  • combine into one input
  • searching should either try to find the address that was pasted or if they search by name e.g. "USDC" should match the query as well

sdfcharles avatar Feb 04 '25 18:02 sdfcharles