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 wallet balance to staking UI #826

Merged
merged 4 commits into from Apr 6, 2022
Merged

Conversation

Sirmorrison
Copy link
Contributor

@Sirmorrison Sirmorrison commented Mar 24, 2022

Fix #781

This PR add total wallet balance to the staking page with progress bar showing locked and spendable balance.

image

The use of a vertical alignment layout instead of the horizontal alignment was due to space constraint as the balance tend to wrap to the next line.

image

  • Possible options are

image

@Sirmorrison Sirmorrison changed the title move stake records into sub_files, code clean up Add wallet balance to staking UI Mar 24, 2022
@Sirmorrison Sirmorrison marked this pull request as ready for review April 5, 2022 18:54
Copy link
Collaborator

@dreacot dreacot left a comment

Choose a reason for hiding this comment

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

The texts above the progress bar are hardly readable, could we increase it a bit?

ui/page/main_page.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@dreacot dreacot left a comment

Choose a reason for hiding this comment

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

LGTM

@dreacot dreacot merged commit 7d41359 into planetdecred:master Apr 6, 2022
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.

Show available/locked balance in Staking UI
2 participants