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

Update transaction indicator #1414

Closed
cjeria opened this issue Mar 10, 2020 · 2 comments · Fixed by #1475
Closed

Update transaction indicator #1414

cjeria opened this issue Mar 10, 2020 · 2 comments · Fixed by #1475
Assignees

Comments

@cjeria
Copy link

cjeria commented Mar 10, 2020

Two usability issues have come up with regards to the transaction status indicator.

Issue 1

After submitting a transaction, a status indicator appears at the bottom of the screen. If tapped, the full transaction history view is loaded moving the user away from the context the user was previously on which could be a dapp where they might be in the middle of a flow.

Solution: Instead of moving the user away from the context if they tap on the tx status indicator, we could instead use a modal to show the transaction status which keeps the user in-context and prevents them from having to navigate back to the browser.

Issue 2

The transaction indicator is not dismissable and overlaps important controls like the browsers nav bar that the user might want to access while waiting for the transaction to be processed (see screenshot below).

Solution: make the tx status indicator dismissible with and float above the navigation bar when in the browser

@cjeria
Copy link
Author

cjeria commented Mar 10, 2020

The suggested solution for issue 1.

The suggested solution for the notification banner (issue 2)

image

@mightypenguin
Copy link

Using compound.finance or kyberswap is super annoying because of this UI overlay currently.
Especially if you want to queue multiple pending transactions at the same time.

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.

3 participants