-
Notifications
You must be signed in to change notification settings - Fork 120
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
[redesign] Launcher views #3690
Conversation
ee7d8fd
to
70c06ae
Compare
8166bbf
to
d14b32e
Compare
d14b32e
to
079745f
Compare
079745f
to
93917ef
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking pretty slick!
UX wise, my only comment is that, after the initial sync process is done, the "Open Wallet" button looks a bit timid: I didn't realize I had to click it to continue opening the wallet.
Also, the "automatically load wallet" doesn't seem to work if you close the wallet (settings -> close wallet) and then try to open it again. It only works after an app reload.
These seem very minor, so i'm ok with merging this as is then working through the issues in an ongoing fashion.
Builds upon #3669
Requires
decred/pi-ui#417has been mergedCloses #3614
Notes:
GetStartedPage/hooks.js
. Same with the new onboarding tutorial pages.Setup a Trezor Wallet
, but it depends on [redesign] Trezor views #3570Automated Wallet Launching
modal and save the answer to the config.It can not be changed from the Settings yet. Planned to implement it after [redesign] Settings view #3631 will be merged.It can be toggled from the Settings now.Updated screenshots: