The Wayback Machine - https://web.archive.org/web/20200919203235/https://github.com/MetaMask/metamask-extension/issues/5211
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

Improve wording of browser notifications #5211

Open
evabeylin opened this issue Sep 6, 2018 · 6 comments
Open

Improve wording of browser notifications #5211

evabeylin opened this issue Sep 6, 2018 · 6 comments

Comments

@evabeylin
Copy link

@evabeylin evabeylin commented Sep 6, 2018

Suggested improvement on the chrome pop-up confirming a transaction:

Currently the pop-up states "Transaction #X" is confirmed. With X being the number of the transaction from that user's ethereum address. Even better if instead of the transaction number, other metadata could be used as the description for the transaction confirmation - such as the dapp or browser the transaction was occuring on, the token that was transferred etc.

E.g., instead of Alice conducting a 6th transaction from her Ethereum address to purchase a Cryptokitty and the pop-up stating "Transaction 6 Confirmed" --> "Cryptokitties Transaction Confirmed".

Can enable better UX and ensure that if users are conducting ethereum transactions across multiple platforms in the same time frame, they can know which or where the transaction took place.

@bdresser
Copy link
Contributor

@bdresser bdresser commented Sep 6, 2018

@evabeylin good idea! The original tx notification was done by a community member and we were just stoked to have something better than the status quo!

It'd be nice to show (1) the name of the contract the user is interacting with and (2) the type of interaction ("sent ether", "approve", etc). It's possible we know neither (i.e. "unknown function"), in which case we should could fallback to the nonce.

@danfinlay thoughts?

@bdresser bdresser added the P3-soon label Sep 6, 2018
@danfinlay
Copy link
Contributor

@danfinlay danfinlay commented Sep 6, 2018

This design space can be pretty wide open for us. Our txMeta class has a lot of potentially useful properties (txMeta.txParams are the standard ethereum tx parameters). We could make this say whatever we want right here:

const message = `Transaction ${nonce} confirmed! View on EtherScan`

I like the idea of incorporating the source site. I also wouldn't mind including a method-name comprehension, maybe using eth-method-registry.

@danfinlay
Copy link
Contributor

@danfinlay danfinlay commented Sep 6, 2018

It strikes me that this string of text is not using our internationalization. That's bad. Also it probably shouldn't be in the platform definition file, since the wording probably isn't platform-specific. Anyways, just adding notes here.

@mrwillis
Copy link

@mrwillis mrwillis commented Feb 14, 2019

@danfinlay How many confirmations does MetaMask wait for to show this chrome extension? EDIT: for anyone interested in this, it's one confirmation. This should probably be configurable.

@bdresser bdresser changed the title Improved Description on Transaction Confirmation Chrome Pop-Up Improve wording of browser notifications Aug 14, 2019
@evabeylin
Copy link
Author

@evabeylin evabeylin commented Sep 12, 2019

Is this still being prioritized?

Honestly having no clarity on what the txn was creates poor UX for those using multiple ethereum dapps. If I'm making multiple transactions on various dapps in the same time period, I have no way of knowing what was confirmed unless I go to etherscan everytime...

@bdresser
Copy link
Contributor

@bdresser bdresser commented Sep 12, 2019

@evabeylin yep, we're working on re-vamping the copy used across the extension to describe transactions. thanks for your patience! cc @rachelcope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.