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

Old UI component to suggest Beta UI #4811

Closed
bdresser opened this issue Jul 16, 2018 · 15 comments · Fixed by #4888
Closed

Old UI component to suggest Beta UI #4811

bdresser opened this issue Jul 16, 2018 · 15 comments · Fixed by #4888
Assignees

Comments

@bdresser
Copy link
Contributor

To finish up #3384, we need a UI component that asks users to upgrade to the new UI. Maybe a persistent banner or something.

@cjeria @whymarrh

@bdresser bdresser added the needs-design Needs design support. label Jul 16, 2018
@cjeria
Copy link
Contributor

cjeria commented Jul 16, 2018

The message should probably say that support for the old UI has been discontinued. No action is required, but that they should be aware that If they don't switch to the New UI, we will automatically be switching everyone over on XX date.

Maybe to encourage users to switch, we use both a modal and banner. The modal will provide us with plenty of room to make the announcement and a button to "Switch to the New UI". If modal is dismissed ("x") and the user doesn't switch, we then show a persistent banner with a CTA to switch to the old UI. Would be good to also have a "learn more" and link to an article describing our plans for the switch within the banner.

@cjeria cjeria self-assigned this Jul 16, 2018
@cjeria
Copy link
Contributor

cjeria commented Jul 19, 2018

@whymarrh Here are the suggested designs for the announcement in the old to ui.

image

@whymarrh
Copy link
Contributor

@cjeria thanks, this looks great! Where did you imagine the "Read more about our plans" and "learn more" links going?

@danfinlay
Copy link
Contributor

What will our "Read more about our plans" link go to?

@cjeria
Copy link
Contributor

cjeria commented Jul 19, 2018

Let's change that link to say "Learn more" and link to https://medium.com/metamask/metamasks-new-ui-has-begun-to-roll-out-74dba32cc7f7

thoughts? @whymarrh @danfinlay

@danfinlay
Copy link
Contributor

Sounds good. Maybe we edit the post to add info about the gas editing + tx parameter viewing.

@bdresser
Copy link
Contributor Author

looks great @cjeria except weird to say "No action is required right now" and then also ask them to click a shiny blue button :) so maybe we remove that little line?

@cjeria
Copy link
Contributor

cjeria commented Jul 20, 2018

good call @danfinlay

@bdresser @whymarrh See updated copy.

image

@bdresser bdresser added L03.0-oldUI and removed needs-design Needs design support. labels Jul 20, 2018
@bdresser
Copy link
Contributor Author

looks great, thanks Christian.

@whymarrh @kumavis what do you think about using Sentry to understand how many clicks we get? Okay by me if we can't de-dupe, would just love any signal on whether people are moving over.

@whymarrh whymarrh assigned whymarrh and unassigned cjeria Jul 24, 2018
@ghost ghost added the in progress label Jul 26, 2018
@bdresser
Copy link
Contributor Author

when will this display for brand-new users? right on startup?

@whymarrh
Copy link
Contributor

@bdresser right on startup

@bdresser
Copy link
Contributor Author

gotcha @whymarrh. can we link them to new UI's full-screen onboarding, rather than the extension? this might be what you're already planning, just want to double check.

@whymarrh
Copy link
Contributor

We can, that's a good idea—you're thinking that the switch will turn on the new UI and then open up a new tab?

@bdresser
Copy link
Contributor Author

Yeah. Full screen in the new UI is a little better formatted, fewer page breaks etc (vs. extension view).

i think that's how it used to work with the small "Try the Beta UI" link at the bottom of Old UI onboarding. But just found #4896, not super urgent, and even less urgent as we push more people off the old.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants
@danfinlay @cjeria @bdresser @whymarrh and others