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

UI fixes to new Onboarding flow #6222

Closed
bdresser opened this issue Feb 28, 2019 · 1 comment
Closed

UI fixes to new Onboarding flow #6222

bdresser opened this issue Feb 28, 2019 · 1 comment
Labels
area-onboarding area-UI Relating to the user interface.

Comments

@bdresser
Copy link
Contributor

Couple tweaks to #6192, apologies for not QA'ing before merge

"Protect Your Seed" step

  • I think this step needs to be AFTER the seed backup, before the "Congrats!" step. Otherwise, the user might not know what we're talking about, since the seed hasn't been introduced yet.
  • Can we say "MetaMask will never spontaneously ask for your seed phrase"
  • Can we make [email protected] a mailto: link so it's more visible

Other

  • Checkbox string on both flows should be "I have read and agree to the Terms of Use" where "Terms of Use" links to this page
  • On the New vs Import step, can we re-introduce the helper text from Christian's designs? My mockup was just meant to indicate switching the main options, I think the details in @cjeria's designs can remain. My bad this was not clear.

screen shot 2019-02-28 at 10 49 15 am

* Under "Yes, let's get set up!" we can say "This will create a new wallet and seed phrase" * Under "No, I already have a seed phrase" we can say "Import your existing wallet using a 12 word seed phrase"

I think this screen needs a quick once-over from @cjeria -- at the very least, let's make each phrase below "Tips on storing it safely" into a bullet point with consistent styling

screen shot 2019-02-28 at 10 52 09 am

@bdresser bdresser added area-UI Relating to the user interface. area-onboarding labels Feb 28, 2019
@bdresser bdresser mentioned this issue Feb 28, 2019
6 tasks
@danjm
Copy link
Contributor

danjm commented Mar 4, 2019

#6171 now has all of these changes except the mailto link on the phishing warning screen

We need to improve the way our translations system works before we can cleanly add that. I may get that in with #6171, or maybe with an additional PR

@bdresser bdresser closed this as completed Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-onboarding area-UI Relating to the user interface.
Projects
None yet
Development

No branches or pull requests

2 participants