From 075804bb70761c2016bca2e62902c6161c76f7a4 Mon Sep 17 00:00:00 2001 From: mantikoros Date: Sat, 19 Mar 2022 12:32:22 -0500 Subject: [PATCH] Revert "formatPercent: only show decimal to at most 2 places" This reverts commit aebb4c54832b100ec16666ccffba74271f0bc537. --- common/util/format.ts | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/common/util/format.ts b/common/util/format.ts index 611e5da4..7f352cf2 100644 --- a/common/util/format.ts +++ b/common/util/format.ts @@ -18,19 +18,20 @@ export function formatWithCommas(amount: number) { return formatter.format(amount).replace('$', '') } -export const decimalPlaces = (x: number) => Math.ceil(-Math.log10(x)) - 2 +const decimalPlaces = (x: number) => Math.ceil(-Math.log10(x)) - 2 export function formatPercent(decimalPercent: number) { - const decimalFigs = + const displayedFigs = (decimalPercent >= 0.02 && decimalPercent <= 0.98) || decimalPercent <= 0 || decimalPercent >= 1 ? 0 - : decimalPercent >= 0.01 && decimalPercent <= 0.99 - ? 1 - : 2 + : Math.max( + decimalPlaces(decimalPercent), + decimalPlaces(1 - decimalPercent) + ) - return (decimalPercent * 100).toFixed(decimalFigs) + '%' + return (decimalPercent * 100).toFixed(displayedFigs) + '%' } export function toCamelCase(words: string) {