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

align all balances with decimal #313

Open
mdpfeiffer opened this issue Oct 28, 2015 · 10 comments
Open

align all balances with decimal #313

mdpfeiffer opened this issue Oct 28, 2015 · 10 comments
Labels

Comments

@mdpfeiffer
Copy link

Align all balance displays along decimal.

I want to be able to tell at a glance which is larger, even in a long list of transactions:

100

or

0.25106631

In a long list, it's kinda messy.

One way to achieve this would be to display 8 decimal places for each transaction. It's not elegant, but would work.

@IngCr3at1on
Copy link
Contributor

Sounds nasty to me actually but I think we'll need other feedback on this to see if it's what others want.

@jwrb
Copy link

jwrb commented Nov 2, 2015

Displaying 8 decimal places is the easiest way to fix this.

Alternatively we could calculate the amount of white space needed on the end of a balance; and then display that amount of white space at the end to alight it.

@MitchellCash
Copy link
Contributor

This is not my comment either way but Paycoin is actually 6 decimal places.

@jwrb
Copy link

jwrb commented Nov 2, 2015

I always forget @MitchellCash. You are correct.

@IngCr3at1on
Copy link
Contributor

@MitchellCash is it bad that I realized that but chose not to comment?

@MitchellCash
Copy link
Contributor

@IngCr3at1on tsk tsk ;)

@mdpfeiffer
Copy link
Author

I really like @jwrb's idea:

we could calculate the amount of white space needed on the end of a balance; and then display that amount of white space at the end to align it.

So when cryptsy shows 8 decimals that's just following BTC convention on their internal ledger?

Whoever designed this coin surely lacked ambition about its future value.

@IngCr3at1on
Copy link
Contributor

@mdpfeiffer the original coin spec called from it going out to 8 decimal places and the original devs never changed the source (which was peercoin) from 6 to 8. Leaving us sort of stuck with 6.

@MitchellCash
Copy link
Contributor

I always wondered if we could push it back to 8 but it seemed like it would mess a lot of shit up

@IngCr3at1on
Copy link
Contributor

@MitchellCash I had a theory/idea on how to do that at one point but I've never attempted it. That's a discussion for somewhere else lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants