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

Add new account component design - Closes #553 #603

Merged
merged 5 commits into from
Mar 29, 2018

Conversation

michaeltomasik
Copy link
Contributor

What was the problem?

Old account component design

How did I fix it?

I added new design ex. reset timer

How to test it?

Go to issue ticket and check each scenario.

Review checklist

Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please check again the font-family, font-size, and font-weight of balance and address in the design. It doesn't look right.
  • The copy-to-clipboard icon should be bigger compared to the font-size of the address
  • The copy-to-clipboard icon should be on the left in the header, but still on the right in all other occurrences.

@slaweet slaweet force-pushed the 553-update-account-component-design branch from 4dc83a3 to c594198 Compare March 29, 2018 07:20
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@michaeltomasik michaeltomasik merged commit ba430fb into 0.4.0 Mar 29, 2018
@slaweet slaweet deleted the 553-update-account-component-design branch March 29, 2018 07:56
faival pushed a commit that referenced this pull request Mar 29, 2018
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 this pull request may close these issues.

2 participants