From c6d073e1193afa1646f6a9c20f36665dc172289f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Mon, 9 Mar 2026 14:10:09 +0800 Subject: [PATCH] Fix getYTicksByIndex truncating integers by using toFixed Co-Authored-By: Claude Opus 4.6 --- src/module/timeLabel.js | 6 +----- tests/unit/module/timeLabel.test.js | 2 +- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/src/module/timeLabel.js b/src/module/timeLabel.js index 9d3212c..e6f5cba 100644 --- a/src/module/timeLabel.js +++ b/src/module/timeLabel.js @@ -76,11 +76,7 @@ const getTimeUnitAndValueToUse = (secondToDecide) => { * @returns {string} The formatted tick label (e.g. "2.5h"). */ export function getYTicksByIndex(stepSize, index, unitToUse) { - const rawStepsizeMultIndex = (stepSize * index).toString(); - const shortenStepsizeMultIndex = rawStepsizeMultIndex.substring( - 0, - rawStepsizeMultIndex.indexOf(".") + 1 + TOFIXED_DECIMAL, - ); + const shortenStepsizeMultIndex = (stepSize * index).toFixed(TOFIXED_DECIMAL); return `${shortenStepsizeMultIndex}${unitToUse}`; } diff --git a/tests/unit/module/timeLabel.test.js b/tests/unit/module/timeLabel.test.js index 2eff072..6d4ad25 100644 --- a/tests/unit/module/timeLabel.test.js +++ b/tests/unit/module/timeLabel.test.js @@ -123,7 +123,7 @@ describe("getStepSizeOfYTicks", () => { describe("getYTicksByIndex", () => { it("formats tick label with unit", () => { - expect(getYTicksByIndex(2, 3, "h")).toBe("6h"); + expect(getYTicksByIndex(2, 3, "h")).toBe("6.0h"); }); it("truncates to 1 decimal place", () => {