-
Notifications
You must be signed in to change notification settings - Fork 51
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
Update metamask popup #151
Comments
@akolotov please assign the issue |
@fernandomg could you take a look? |
Open questions and recommendation can be found in #147 |
Just want to clarify and maybe have a better UX. Currently the modal with the 'Error' is being displayed despite if the wallet is not installed or locked. The message is quite generic and may lead to confusion. I can identify each type of error and display:
Both errors are triggered already, but just need to be properly handled. Screenshots (check the console error message displayed at the bottom) UnlockInstallAny thoughts @akolotov @igorbarinov? |
My bid is that we need to handle these two cases in different way. |
Old view
New view
Zeplin: https://zpl.io/a8P3Bjx
The text was updated successfully, but these errors were encountered: