PathOfBuilding icon indicating copy to clipboard operation
PathOfBuilding copied to clipboard

Reduced Precision Number Display

Open Regisle opened this issue 5 months ago • 1 comments

This reduces the number of digits shown, and instead opts to show SI prefixes (k, M, B, T) image

I just ended up applying this to everything for now, its much harder to only apply it to dps becouse then I need to tag what does and does not get it applied to, I kept the same level of precision as the base number if it had any, (and 2 decimals if it didnt), I can standardise this to 2 if we want it.

This does not modify the values stored in the xml, only the values displayed in PoB itself.

This does have issues above 15 digits, I can either add more prefixes, or can let it continue in the function through showThousandsSeparators to still get separated, but people shouldnt have numbers that big so I think this is fine

I can cleanup code to not use if statements, but it shouldnt change much, so up to maintainer what they prefer for style

Regisle avatar Sep 07 '24 09:09 Regisle