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

Fix general UI issues according to proposal #5407

Merged
merged 9 commits into from
Oct 31, 2023

Conversation

ikem-legend
Copy link
Member

@ikem-legend ikem-legend commented Oct 27, 2023

What was the problem?

This PR resolves #5306

How was it solved?

Fixed stated UI issues

  • Network selector border radius
  • Verify message spacing
  • Page title font size
  • Wallet address alignment
  • Transaction broadcast "Back to wallet" button
  • Add Account flow font, color and spacing
  • Staking flow font, color and spacing
  • Empty state description font

How was it tested?

  • Visually
  • Unit and E2E tests

@ikem-legend ikem-legend changed the title Fix general UI issues #1 Fix general UI issues according to proposal Oct 27, 2023
@ikem-legend ikem-legend marked this pull request as ready for review October 31, 2023 11:38
oskarleonard
oskarleonard previously approved these changes Oct 31, 2023
@ManuGowda ManuGowda removed the request for review from eniolam1000752 October 31, 2023 14:35
@ManuGowda ManuGowda merged commit a14cb26 into release/3.0.0 Oct 31, 2023
3 checks passed
@ManuGowda ManuGowda deleted the 5306-adapt-ui-proposal branch October 31, 2023 14:35
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.

3 participants