-
Notifications
You must be signed in to change notification settings - Fork 128
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
Feature Request: Dynamic Scaling of Displayed Values on Home Page #1973
Comments
Yes please! Big zero in Difficulty and Hashrate look like something is broken. While at it we may also change |
I spent a long time trying to get familiar with the codebase so I could make this change. I'm nearly there, but it's more challenging than just plug-n-play Dave's code. I'm going to write some notes down because I'm about to go on vacation, and I'll need a reminder when I get back.
If anyone wants to pick this up in the next two weeks, feel free, or I'll just continue when I get back. |
@matthawkins90 are you back from vacation? I could look into this issue if you’re still away. |
I'm back, haha. Just haven't prioritized this yet. I was actually going to take a stab at it again this week based on Xaur's feedback about how the difficulty could be formatted. The difficulty on the front page is one thing, but the difficulty in the charts / graphs seems like it's a whole lot harder and I don't even know where to begin. |
Oh, okay. Sounds good. |
@ukane-philemon I just finished the difficulty, where I ended up just using threeSigFigs, which is possible because it doesn't need any units. But since the hash rate requires units, I don't think you can use threeSigFigs for it, and might need to use Dave's original code at the top. Can you take care of that? |
Do you have a PR for this yet? |
I'll be honest, I'm not sure why the build is failing. |
I think the Go test failure is not from you, but the JS, you can fix it by running |
Notice in the screenshot how the Mining difficulty says "0 Mil". I suggest this should be a dynamically scaled humanized value instead. Similarly, the hashrate is being reproted in Ph/s. It should scale dynamically too.
Example code:
Output:
The text was updated successfully, but these errors were encountered: