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.

Sign up for GitHub

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

Jump to bottom

new ui - does not show tx error message #3768

Closed
kumavis opened this issue Mar 28, 2018 · 2 comments · Fixed by #5223
Closed

new ui - does not show tx error message #3768

kumavis opened this issue Mar 28, 2018 · 2 comments · Fixed by #5223
Assignees
@whymarrh
Labels
area-UI Relating to the user interface.

Comments

@kumavis
Copy link
Member

kumavis commented Mar 28, 2018

image

nowhere to inspect error message
@kumavis kumavis added area-UI Relating to the user interface. needs-design Needs design support. labels Mar 28, 2018
@jennypollack jennypollack mentioned this issue Jul 24, 2018
Open
5 tasks
@bdresser
Copy link
Contributor

bdresser commented Aug 7, 2018

In the old UI, hovering over the error message shows details - let's add this in to new UI

short term: can include the error message where it says Failed - long term, this will be handled nicely in the transaction detail view #4786

@bdresser
Copy link
Contributor

bdresser commented Aug 7, 2018

can just do a simple hover-tooltip, no design necessary

@bdresser bdresser removed the needs-design Needs design support. label Aug 7, 2018
@whymarrh whymarrh mentioned this issue Aug 22, 2018
Merged
@ghost ghost added the in progress label Aug 22, 2018
This was referenced Sep 7, 2018
Closed
Merged
@ghost ghost removed the in progress label Sep 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@whymarrh whymarrh

Labels
area-UI Relating to the user interface.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Show transaction error message tooltips for statuses whymarrh/metamask-extension
3 participants
@bdresser @kumavis @whymarrh