patch: allow ints in chart min/max values

acheronfail · Sep 28, 2024 · fae1ec8 · fae1ec8
1 parent fae19be
commit fae1ec8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/components/Chart.svelte b/src/components/Chart.svelte
@@ -352,15 +352,15 @@
             {/if}
             {#if minShown}
               <div class="flex flex-row gap-2">
-                <span style:color={line.color}>min: {formatFloat(min)}{unit}</span>
+                <span style:color={line.color}>min: {formatFloat(min, true)}{unit}</span>
               </div>
             {/if}
             {#if maxShown && minShown}
               <span class="text-slate-500">-</span>
             {/if}
             {#if maxShown}
               <div class="flex flex-row gap-2">
-                <span style:color={line.color}>max: {formatFloat(max)}{unit}</span>
+                <span style:color={line.color}>max: {formatFloat(max, true)}{unit}</span>
               </div>
             {/if}
           {/if}

diff --git a/src/lib/misc.ts b/src/lib/misc.ts
@@ -21,4 +21,8 @@ export const speedMapper = (inUnit: Units, outUnit: Units): ((input: number) =>
   return miToKm;
 };
 
-export const formatFloat = (n: number) => (Number.isNaN(n) ? '??' : n.toFixed(1));
+export const formatFloat = (n: number, allowInt = false) => {
+  if (Number.isNaN(n)) return '??';
+  if (allowInt && Number.isInteger(n)) return n.toString();
+  return n.toFixed(1);
+};