search

decred / decrediton

Cross-platform GUI for Decred.
https://docs.decred.org/wallets/decrediton/decrediton-setup/
ISC License
195 stars 121 forks source link

Launcher + Trezor UX improvments - discussion and validation #3297

Open ta-lind opened 3 years ago

ta-lind commented 3 years ago

See drafts here: https://www.figma.com/file/EanUbqJmPs8EMqY26j9zwo/decred-piui-master?node-id=613%3A0

Figured these updates would make sense with further changes to the launcher. If that anything causes technical issues, lmk.

Overall would simplify the launcher view and provide better feedback what's going on with each wallet: i.e. does the wallet have privacy enabled, ln, trezor. These can be communicated via both icons and labels. It makes sense to then also use simpler cards for wallets as they'll be visually busier.

For wallet management, i'd recommend having separate flow for Trezor. Having everything under Restore is somewhat confusing, as for example Trezor and Decrediton already have different seed word formats.

If there's an accepted device found, creating a Trezor wallet would open an isolated view. In there we'd provide a bit more feedback on what can be done with the device to continue. As Trezor's native wallet has a well refined setup, I think it's also fine to refer to that as an option.

As one of the changes, i'd recommend moving the loader bar to the bottom edge of the launcher view. This makes for more practical use of space as there's now much more going on than when the initial loader layout was designed. During the Trezor setup, the loader could shown in half size, while displaying the trezor status on the left side. This would be practical also when there's more tutorial content made, as that can be shown during the blockchain loading.

Setup accordions are slightly refined to make them more informative.

PIN is reworked to use bullets/dots instead of letters to make a more consistent experience with Trezor's native wallet.

When the wallet is open, recommend moving the Trezor setup to a separate tab under settings, instead of keeping it in the main nav.

guilhermemntt commented 3 years ago

Looks good! Could we please add some spec for https://github.com/decred/decrediton/issues/2955?

guilhermemntt commented 3 years ago

I'd like to implement these improvements, if there are no objections.