[bisq-network/bisq] Touch-ups to the DAO BSQ Supply page (#4785)

deusmax notifications at github.com
Thu Nov 12 11:08:15 CET 2020


<!-- 
- make yourself familiar with the CONTRIBUTING.md if you have not already (https://github.com/bisq-network/bisq/blob/master/CONTRIBUTING.md)
- make sure you follow our [coding style guidelines][https://github.com/bisq-network/style/issues)
- pick a descriptive title
- provide some meaningful PR description below
- create the PR
- in case you receive a "Change request" and/or a NACK, please react within 30 days. If not, we will close your PR and it can not be up for compensation.
- After addressing the change request, __please re-request a review!__ Otherwise we might miss your PR as we tend to only look at pull requests tagged with a "review required".
-->
## Intro

The BSQ supply page on the DAO GUI tab, provides 3 charts and some numerical data (not statistics) on the BSQ supply.
For simplicity, I will refer here to the charts as chart#1 (top), chart#2 (middle) and chart#3 (bottom) to avoid any confusion with the names.
The charts and the page itself seems a little confusing. I decided to do some touch-ups to improve it. This is not a redesign of the page.

## Current Status 

The DAO BSQ pages, as of version 1.42 is like this:
![Bisq-Dao-Page-BSQ_Supply_top-2020-11-08](https://user-images.githubusercontent.com/142019/98921446-fb723080-24d9-11eb-9454-20d8057e55fe.png)
![Bisq-Dao-Page-BSQ_Supply_bottom-2020-11-08](https://user-images.githubusercontent.com/142019/98921479-075df280-24da-11eb-8af6-09415f6323c5.png)

## Results
The result of this PR is currently:
![Bisq-Dao-Page-BSQ_Supply_NEW_top-2020-11-12](https://user-images.githubusercontent.com/142019/98922113-c0bcc800-24da-11eb-8c2a-ff06e78acb77.png)
![Bisq-Dao-Page-BSQ_Supply_NEW_bottom-2020-11-12](https://user-images.githubusercontent.com/142019/98923219-0e860000-24dc-11eb-8420-7418eee13dc7.png)

## Description of changes

1. Chart touch-ups affect the display of lines and axes.
2. Removed Chart#2, it has no new information from Chart#1, so it is not needed.
3. Did not:
    - Check or touch any of the calculations in the number sections.
    - Removed chart#2 by commenting-out the relevant code and left as a placeholder. Easy to re-insert another chart in its place if/when needed.

### Remaining issues 

1. Chart#1 is a monthly summary, does not align well with the tick marks. Can be worked on.
2. Section **BSQ burnt** show a negative number for _Burned BSQ (fees)_. **Q:** Is that by design ?

You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/4785

-- Commit Summary --

  * Improve view of market offers chart
  * Touch ups on the DAO BSQ supply page.

-- File Changes --

    M core/src/main/java/bisq/core/locale/CurrencyUtil.java (4)
    M core/src/main/resources/i18n/displayStrings.properties (1)
    M desktop/src/main/java/bisq/desktop/bisq.css (14)
    M desktop/src/main/java/bisq/desktop/main/dao/economy/supply/SupplyView.java (91)
    M desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartView.java (50)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/4785.patch
https://github.com/bisq-network/bisq/pull/4785.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4785
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201112/810141a3/attachment.html>


More information about the bisq-github mailing list