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

Increase price precision in Offer Book chart (altcoins) #4311

Merged
merged 1 commit into from Jul 1, 2020
Merged

Increase price precision in Offer Book chart (altcoins) #4311

merged 1 commit into from Jul 1, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jun 16, 2020

In the Offer Book Chart, altcoins were displayed with prices having only 3 decimal places of precision. This resulted in not being able to see any meaningful prices across the x-axis because they were all the same, e.g. ETH=0.025

Instead it should show 8 digits of precision for altcoins. So for example ETH=0.02456914
Change the rule for display of altcoin Offer Book chart from:
if price begins 0.000 then display higher precision
to:
if price begins 0.0 then display higher precision
Fiat currency chart not affected by this rule.

Fixes #4251

In the Offer Book Chart, altcoins were displayed with prices
having only 3 decimal places of precision.  This resulted in
not being able to see any meaningful prices across the
x-axis because they were all the same, e.g. ETH=0.025
Instead it should show 8 digits of precision for altcoins.
So for example ETH=0.02456914
Change the rule from if price begins 0.000 then display
higher precision, to if price begins 0.0 then display higher
precision.

Fixes #4251
@ghost
Copy link
Author

ghost commented Jun 16, 2020

XMR before
image
XMR after
image
ZEC before
image
ZEC after
image
LTC before
image
LTC after
image
ETH before
image
ETH after
image

Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

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

utACK

@sqrrm sqrrm merged commit 594b1a4 into bisq-network:master Jul 1, 2020
@sqrrm sqrrm added this to the v1.3.6 milestone Jul 1, 2020
@ghost ghost mentioned this pull request Jul 12, 2020
@ghost ghost deleted the higher_precision_prices branch September 14, 2020 22:20
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.

Market Offer book graph needs higher precision x-axis values
1 participant