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

NewUI: Show eth values on confirm screen #4298

Closed
danfinlay opened this issue May 18, 2018 · 3 comments
Closed

NewUI: Show eth values on confirm screen #4298

danfinlay opened this issue May 18, 2018 · 3 comments
Assignees
Labels
area-UI Relating to the user interface. needs-design Needs design support. type-bug

Comments

@danfinlay
Copy link
Contributor

User reported the eth value of a transfer isn't reflected on the NewUI confirm screen. We should definitely still show ether. Does our design not include ether value? @cjeria

image 2

@danfinlay danfinlay added type-bug needs-design Needs design support. area-UI Relating to the user interface. labels May 18, 2018
@cjeria
Copy link
Contributor

cjeria commented May 18, 2018

We do show it, but user needs to scroll to the bottom to see it (see screenshot). This is on our radar to fix asap. @danfinlay

image

@cjeria
Copy link
Contributor

cjeria commented Jun 4, 2018

Solution to this was to remove the to and from rows in this screen as it's redundant.

Quick fix design:
image

@bdresser
Copy link
Contributor

bdresser commented Jun 4, 2018

looks good @cjeria

do you want the "total" row to stay gray or change to white? also what about the helper text up top?

screen shot 2018-06-04 at 12 16 08 pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-UI Relating to the user interface. needs-design Needs design support. type-bug
Projects
None yet
Development

No branches or pull requests

4 participants