-
Notifications
You must be signed in to change notification settings - Fork 717
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
[Qt] Hide zPIV balances when they are zero #1323
[Qt] Hide zPIV balances when they are zero #1323
Conversation
Concept ACK. Good follow-up on #1306 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concept ACK 👍 . Left some code improvements.
Since there will be no **new** zerocoins anymore, there is no need to take up UI space with zero value zerocoin balance information.
606c732
to
cd1f806
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looking good, utACK cd1f806
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK cd1f806 and merging...
cd1f806 [Qt] Hide zPIV balances when they are zero (Fuzzbawls) Pull request description: Since there will be no **new** zerocoins anymore, there is no need to take up UI space with zero value zerocoin balance information. This hides the related UI labels if/when the wallet doesn't have any zerocoin balance. Before: ![Screen Shot 2020-02-06 at 3 42 28 PM](https://user-images.githubusercontent.com/7393257/73988398-711bf100-48f7-11ea-8872-94c2b653b6b9.png) After: ![Screen Shot 2020-02-06 at 3 43 08 PM](https://user-images.githubusercontent.com/7393257/73988406-7711d200-48f7-11ea-9a01-adfb9ecb4f3c.png) ACKs for top commit: furszy: Code looking good, utACK cd1f806 random-zebra: ACK cd1f806 and merging... Tree-SHA512: 84fdfa7790d137efe319e7da8caf9f46a7787f51de7feb7ce164c738ac4f002515d3946857a1837d18480d88e49e62e43457de755d436922e7ce229ca736498e
Since there will be no new zerocoins anymore, there is no need to
take up UI space with zero value zerocoin balance information.
This hides the related UI labels if/when the wallet doesn't have any zerocoin
balance.
Before:
![Screen Shot 2020-02-06 at 3 42 28 PM](https://user-images.githubusercontent.com/7393257/73988398-711bf100-48f7-11ea-8872-94c2b653b6b9.png)
After:
![Screen Shot 2020-02-06 at 3 43 08 PM](https://user-images.githubusercontent.com/7393257/73988406-7711d200-48f7-11ea-9a01-adfb9ecb4f3c.png)