From 1407c9a4602c57b6dac442e569329373e7b3a0db Mon Sep 17 00:00:00 2001
From: jaredvu <jaredvu@gmail.com>
Date: Sun, 26 Nov 2023 14:32:20 -0800
Subject: [PATCH] Orderbook Row: Handle locale change

---
 .../orderbook/useDrawOrderbookHistograms.ts   | 113 ++++++++++--------
 1 file changed, 66 insertions(+), 47 deletions(-)

diff --git a/src/hooks/orderbook/useDrawOrderbookHistograms.ts b/src/hooks/orderbook/useDrawOrderbookHistograms.ts
index c974ec2..7f1c7a4 100644
--- a/src/hooks/orderbook/useDrawOrderbookHistograms.ts
+++ b/src/hooks/orderbook/useDrawOrderbookHistograms.ts
@@ -1,8 +1,9 @@
-import { useEffect, useMemo, useRef } from 'react';
+import { useCallback, useEffect, useMemo, useRef } from 'react';
 import { useSelector } from 'react-redux';
 
 import type { Nullable } from '@/constants/abacus';
 import { SMALL_USD_DECIMALS, TOKEN_DECIMALS } from '@/constants/numbers';
+import { useLocaleSeparators } from '@/hooks/useLocaleSeparators';
 
 import { getAppTheme } from '@/state/configsSelectors';
 
@@ -130,54 +131,63 @@ export const useDrawOrderbookHistograms = ({
     ctx.fill();
   };
 
-  const drawText = ({
-    ctx,
-    canvasWidth,
-    idx,
-    size,
-    price,
-    mine,
-  }: {
-    ctx: CanvasRenderingContext2D;
-    canvasWidth: number;
-    idx: number;
-    size?: Nullable<number>;
-    price?: Nullable<number>;
-    mine?: Nullable<number>;
-  }) => {
-    const y = getYFromIndex(idx, 'text');
+  const { decimal: LOCALE_DECIMAL_SEPARATOR } = useLocaleSeparators();
 
-    // Size text
-    if (size) {
-      ctx.fillStyle = 'white';
-      ctx.fillText(
-        MustBigNumber(size).toFixed(stepSizeDecimals ?? TOKEN_DECIMALS),
-        getXByColumn({ canvasWidth, colIdx: 0 }) - ROW_PADDING_RIGHT,
-        y
-      );
-    }
-
-    // Price text
-    if (price) {
-      ctx.fillStyle = 'white';
-      ctx.fillText(
-        MustBigNumber(price).toFixed(tickSizeDecimals ?? SMALL_USD_DECIMALS),
-        getXByColumn({ canvasWidth, colIdx: 1 }) - ROW_PADDING_RIGHT,
-        y
-      );
-    }
-
-    // Mine text
-    if (mine) {
-      ctx.fillStyle = 'white';
-      ctx.fillText(
-        MustBigNumber(mine).toFixed(stepSizeDecimals ?? TOKEN_DECIMALS),
-        getXByColumn({ canvasWidth, colIdx: 2 }) - ROW_PADDING_RIGHT,
-        y
-      );
-    }
+  const formatOptions = {
+    decimalSeparator: LOCALE_DECIMAL_SEPARATOR,
   };
 
+  const drawText = useCallback(
+    ({
+      ctx,
+      canvasWidth,
+      idx,
+      size,
+      price,
+      mine,
+    }: {
+      ctx: CanvasRenderingContext2D;
+      canvasWidth: number;
+      idx: number;
+      size?: Nullable<number>;
+      price?: Nullable<number>;
+      mine?: Nullable<number>;
+    }) => {
+      const y = getYFromIndex(idx, 'text');
+
+      // Size text
+      if (size) {
+        ctx.fillStyle = 'white';
+        ctx.fillText(
+          MustBigNumber(size).toFormat(stepSizeDecimals ?? TOKEN_DECIMALS, formatOptions),
+          getXByColumn({ canvasWidth, colIdx: 0 }) - ROW_PADDING_RIGHT,
+          y
+        );
+      }
+
+      // Price text
+      if (price) {
+        ctx.fillStyle = 'white';
+        ctx.fillText(
+          MustBigNumber(price).toFormat(tickSizeDecimals ?? SMALL_USD_DECIMALS, formatOptions),
+          getXByColumn({ canvasWidth, colIdx: 1 }) - ROW_PADDING_RIGHT,
+          y
+        );
+      }
+
+      // Mine text
+      if (mine) {
+        ctx.fillStyle = 'white';
+        ctx.fillText(
+          MustBigNumber(mine).toFormat(stepSizeDecimals ?? TOKEN_DECIMALS, formatOptions),
+          getXByColumn({ canvasWidth, colIdx: 2 }) - ROW_PADDING_RIGHT,
+          y
+        );
+      }
+    },
+    [LOCALE_DECIMAL_SEPARATOR]
+  );
+
   /**
    * Scale canvas using device pixel ratio to unblur drawn text
    * @returns adjusted canvas width used in coordinates for drawing
@@ -292,7 +302,16 @@ export const useDrawOrderbookHistograms = ({
         mine,
       });
     });
-  }, [canvasWidth, data, histogramRange, selectedTheme, stepSizeDecimals, tickSizeDecimals, to]);
+  }, [
+    canvasWidth,
+    data,
+    drawText,
+    histogramRange,
+    selectedTheme,
+    stepSizeDecimals,
+    tickSizeDecimals,
+    to,
+  ]);
 
   return { canvasRef, hoverCanvasRef };
 };