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

Resubmitting transaction should be worded clearer and better highlighted #5075

Closed
Cygnusfear opened this issue Aug 15, 2018 · 4 comments
Closed

Comments

@Cygnusfear
Copy link

What problem are you trying to solve?
Currently transactions that are not processed hold up all future transactions, the offending transaction is marked by Metamask with a small blue subtext saying 'Taking too long? Retry with a higher gas price here'. However, as much as I'd not consider myself fully stupid, the wording of this subtext is vague enough that I had no idea this was how I could resubmit the transaction.

Describe the solution you'd like
Replace the text with 'Click here to resubmit your transaction with a higher gas price'. Alternatively:
'Click here to increase the gas price for speeding up your transaction'

Additional comments
If future transactions are being blocked by that one transaction, this should be made apparent to the user; novice users may probably not be aware that their transactions are being blocked and will resubmit new transactions which does not help. Suggested solutions:

  • Grey out new transactions making it clearer what the last 'active' transaction is and thus highlighting the increase gas feature
  • Notify the user in a new transaction screen that there is a transaction pending that may block any further transactions

Issue described above may cost unknowing users money due to their Metamask seemingly being 'broken':
https://www.reddit.com/r/ethtrader/comments/975ybz/daily_general_discussion_august_14_2018/e45viqx/

@Cygnusfear Cygnusfear changed the title Resubmitting transaction Resubmitting transaction should be worded clearer and better highlighted Aug 15, 2018
@Cygnusfear
Copy link
Author

Cygnusfear commented Aug 15, 2018

Additionally: Showing a duplicate/reference/notification of the offending transaction at the top of the transaction list might help make the user more aware of the transaction blocking all other transactions.

@bdresser
Copy link
Contributor

@Cygnusfear check out our designs for a new transaction detail view - it's ordered with pending tx first, like you suggest, and changes the way we suggest users speed up pending tx: #4786

@Cygnusfear
Copy link
Author

That looks great! Only suggestions I could think of is a warning if a transaction suddenly takes extremely long (pending label could be colored red) due to a spontaneous increase in network usage. I'll head over to the design corner and start rummaging around there!

@bdresser
Copy link
Contributor

Sounds good. Closing this, thanks for the feedback.

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

No branches or pull requests

2 participants