Phase 1 Testing

[riongull]

Phase 1 Testing

Testing dates: Feb 1, 2024 - Feb 5, 2024

This was originally authored as a Google Doc, converting to a GitHub issue because @jojobyte likes Microsoft more than Google (</sarcasm>)

Most of these issues won’t need dedicated GitHub issues or PRs, but I will categorize them like conventional commits to help organize them. Some user experience (UX) issues will be tagged as “feat” if they affect a current feature that may need to be reconsidered, others may be more simple changes, which could go under “style” or “doc”.

I’m basically going through the whole app and testing each feature (link, button, etc). I’m not necessarily trying to break the app, but I’m monkeying around and being more picky than a normal user.

This document incorporates feedback from me and juliushamilton1

Feedback by Application Feature

Welcome screen

• [x] #22
  • I want to be able to navigate to either button and hit enter and have it work.
• [ ] style: Some text is redundant, but no need to change for now.
• [x] #23
• [ ] style(julius): Is there some reason it is necessary to use the terms “disconnect” and “add wallet” as opposed to just “log in” and “log out”?

New Wallet modal

• [x] #24
• [x] #25
• [ ] doc(julius): Minor aspects of user understanding: how does the wallet actually work? What information is your browser storing when you create a wallet? Is it downloading a temporary self custodial wallet into the browser session? How does that work, where is it saved to? How does locking work?
• [ ] doc(julius): User experience: should you be able to create a new alias every time you log back in?

Existing Wallet modal

• [x] #26
• [x] fix: The seed phrase eye toggle doesn’t seem to work (at all, or at least reliably).
• [x] fix(julius): the show/hide button for the seed wallet entry field doesn’t work
• [x] #36
• [x] doc: Here’s a good description of the alias. Let’s copy this over to the other modal.

Decrypt Wallet modal

• [x] #28
• [ ] #42

Main screen

• [ ] style(julius): Minor visual design notes: the landing page feels slightly “empty”. Awkward side columns.
• [x] feat: Same as welcome screen item 1. #22
  • I want all buttons, fields, links, etc accessible by keyboard navigation
  • I can see it going from #home to #me, and hitting enter works, but not all action options (like the buttons) are available.
• [ ] feat: I kind of want some hover options right on each contact in the list. This would bring some life and color to the otherwise sparse list.
  • [ ] send arrow - shortcut to send to this contact
  • [ ] receive arrow - shortcut to request from this contact
  • [ ] trash icon - quickly get rid of contacts that I know I didn’t share
  • [ ] star - not now, but later on, to bring up to a top section of the list
• [ ] feat: Maybe we should use the big empty space in the contacts list to recommend pairing with other prominent wallets (mobile wallet, Desktop wallet, etc).
  • [ ] DCG’s mobile wallet definitely needs to display a QR for its xpub address.
  • [ ] We need to support importing raw xpubs if we don’t already.

Alias Button

• [x] #29

Edit Profile modal

• [ ] #40

Backup Wallet modal

• [ ] style: Top button is “adjective noun”, bottom two buttons are “verb noun”. Maybe make it consistent.

Lock modal

• [x] style: The bottom button lines don’t line up, maybe because of emoji in one but not the other. Or maybe because of box-sizing: border-box

Disconnect modal

• [x] style: Still not sure “disconnect” is the right word. Leave for now. Either way, maybe change the bomb to something else? A broom or something else could be used if we changed it to “Clean”, but I’m not sure there’s anything better than a bomb for now. Leave it for now.

Dash Balance field

• [ ] #34

Send Funds modal

• [x] #41
• [x] #33
  • Scanning a normal Dash address captures it, but doesn’t parse and enter in the to field.
  • I don’t like the up/down buttons on the input field
  • Does anyone adjust the amount like that?
  • Either way, we should make the step = 0.001 DASH (last digit in white area, about 3 cents USD)
• [x] #37
  • When a leading decimal exists (e.g. ".01") I want it to automatically change it to "0.01".
  • I like that the validation disallows numbers without a leading decimal if we add the above fix.
  • The validation allows 9 post-decimal values; it should only allow 8.

Request Funds modal

• [x] #38
• [x] feat: Every time I click Request the wallet generates a new address. This might be unsettling for users. Need to rethink this functionality. Side note/question: Is this cycling through the 0 index on the HD path when no contact is selected? Either way, we may want to put our own username in the alias input box by default, which would be the default funding mechanism.

Add a New Contact modal

• [ ] #39
• [ ] What about putting data into a normal web link (https://wallet.dashincubator.dev with parameters in the query string) that people can just click and accept your friend request)?
• [x] #32
  • I made a QR code out of the text, but it didn’t read it.
  • It does show a green square when you line it up right, but it doesn’t do anything other than that.
• [x] Adding contacts doesn’t seem to be working at all. Saving has two issues:
  • The address/xpub isn’t saving
  • The name is appending “- Finish pairing with contact” to every alias, even if adding all info in one fresh and complete step.
  • Might only be when using a raw xpub (text starting with “xpub…”

Clicked Contact modal

• [ ] style: The modal could use a header like I think all the other modals have.

[coolaj86]

• Needs a way to skip the passphrase (i.e. have an empty string "" or a constant, such as "NONE" that's automatically entered when the "skip passphrase" setting)
• We need a way to see the list of what money has been sent and received
• I shouldn't have to click Send 3 times to send \ (but there definitely should be an animation and a 2 second "sending" - even if it doesn't take that long)