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

Make the number easier to read #2602

Closed
iketheadore opened this issue May 2, 2021 · 5 comments · Fixed by #2616 · May be fixed by #2619
Closed

Make the number easier to read #2602

iketheadore opened this issue May 2, 2021 · 5 comments · Fixed by #2616 · May be fixed by #2619
Assignees
Labels
wallet Desktop wallet

Comments

@iketheadore
Copy link
Member

Requested by Synth:

When typing numbers in wallet, like 10 000.

Can we put a slight space, every 3 digits to make it easier to read number, and make it easier to see difference between 10,000 and 100,000

or put an implicit comma, that the user doesn't type and which is not in the text field

@iketheadore iketheadore added the wallet Desktop wallet label May 2, 2021
@iketheadore
Copy link
Member Author

Hi @Senyoret1 would you mind to handle this when you got time, thanks.

@Senyoret1
Copy link
Contributor

Sorry for the delay. Fixed in #2616

However, I think another problem is that it is nor very easy to see the decimal values in the UI. Maybe it would be good to use a different style for the decimal parts. Instead of this:
v1
we could show something like this:
v2
The explorer would also need something similar. What do you think?

@iketheadore
Copy link
Member Author

I agree, the second one looks better.

@iketheadore
Copy link
Member Author

Hi @Senyoret1 , I checked the code and had a test on local env. All looks good, except that some of the decimal values are not processed, also when coins value is not big, such as
image
, the different style of the decimal value would look a little weird. So perhaps we should keep the old style.

@Senyoret1
Copy link
Contributor

I didn’t use the new format in the USD values, but I can use it in them if you want. Appart from that, did you found other coin values not using the new format?

About the format in small numbers, I was also thinking about that while making the PR but found 2 things.

The first one is that using the new format only for some values would make the UI appear to be inconsistent, as the user don’t have to kanow the rules used by the system for deciding when to show small decimals and when not. The use of the 2 formats in the same place at different times could appear to the users just like erratic behavior.

The other thing is that the change made in this PR should be for practical reasons and not really for aesthetic reasons, so in the case of “1.234” it still serves for quickly identify that the value is not more than one thousand, because the point is very small. Also, in a lot of regions the thousands are separated using points and the decimals with a comma, which is just the contrary of the format used in the US, and that could make the numbers even more confusing for some users.

These are the reasons for the current behavior, but if you still want changes to be made, just tell me.

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