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

Separate thousands place with a comma #159

Closed
krazyito65 opened this issue May 10, 2020 · 3 comments
Closed

Separate thousands place with a comma #159

krazyito65 opened this issue May 10, 2020 · 3 comments

Comments

@krazyito65
Copy link

Is there anyway you could add an option to separate the numbers with commas for stuff above 1000? I feel like it makes it much easier to read.

@Sundava
Copy link
Contributor

Sundava commented May 8, 2021

In the same spirit, would it be possible to add the option to shorten values using K suffix, like MopiMopi does ?
ACT's "Simplify numbers" option does not do anything for Kagerou

@hibiyasleep
Copy link
Owner

hibiyasleep commented May 11, 2021

'Smaller font for under-k digit for dps' was added with the purpose of actually this. (I don't like commas imo, since it has localization/readability/and more issues especially with .)

and... 'abbreviate numbers' option was added at 50a993c and version v0.8.6.

@Sundava
Copy link
Contributor

Sundava commented May 21, 2021

Is it possible to extend the abbreviate numbers option to the total damage/healing columns too ?

Edit: I took the liberty of opening a PR for this

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

No branches or pull requests

3 participants