From 0948a82eb5052eba36afc55d1d204756ac5a9f06 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Sun, 8 Mar 2026 18:58:01 +0800 Subject: [PATCH] Align compare navbar state mapping with MAP and PERFORMANCE tabs Co-Authored-By: Codex --- src/components/AppNavbar.vue | 6 ++++-- src/utils/pageUtils.js | 5 +++-- tests/stores/pageAdmin.test.js | 5 +++++ tests/unit/utils/pageUtils.test.js | 4 ++-- 4 files changed, 14 insertions(+), 6 deletions(-) diff --git a/src/components/AppNavbar.vue b/src/components/AppNavbar.vue index 1ca31ee..c475065 100644 --- a/src/components/AppNavbar.vue +++ b/src/components/AppNavbar.vue @@ -325,9 +325,11 @@ function getNavViewName() { break; case "COMPARE": switch (name) { - case "dummy": + case "MapCompare": + valueToSet = "MAP"; + break; case "CompareDashboard": - valueToSet = "DASHBOARD"; + valueToSet = "PERFORMANCE"; break; default: break; diff --git a/src/utils/pageUtils.js b/src/utils/pageUtils.js index 2bd4413..62dc37e 100644 --- a/src/utils/pageUtils.js +++ b/src/utils/pageUtils.js @@ -10,7 +10,8 @@ * Maps a raw route page name to a unified uppercase display name. * * Converts "Check"-prefixed route names (e.g. "CheckMap") to their - * base names (e.g. "MAP"), and "CompareDashboard" to "DASHBOARD". + * base names (e.g. "MAP"), and "CompareDashboard" to "PERFORMANCE" + * to match the compare navbar tab label. * All other names are simply uppercased. * * @param {string} rawPageName - The raw route name to convert. @@ -27,7 +28,7 @@ const mapPageNameToCapitalUnifiedName = (rawPageName) => { case "CHECKPERFORMANCE": return "PERFORMANCE"; case "COMPAREDASHBOARD": - return "DASHBOARD"; + return "PERFORMANCE"; default: return rawPageName.toUpperCase(); } diff --git a/tests/stores/pageAdmin.test.js b/tests/stores/pageAdmin.test.js index 8d1d515..c2abad6 100644 --- a/tests/stores/pageAdmin.test.js +++ b/tests/stores/pageAdmin.test.js @@ -50,6 +50,11 @@ describe("pageAdminStore", () => { expect(store.pendingActivePage).toBe("MAP"); }); + it("setPendingActivePage maps CompareDashboard to PERFORMANCE", () => { + store.setPendingActivePage("CompareDashboard"); + expect(store.pendingActivePage).toBe("PERFORMANCE"); + }); + it("copyPendingPageToActivePage transfers value", () => { store.setPendingActivePage("CheckConformance"); store.copyPendingPageToActivePage(); diff --git a/tests/unit/utils/pageUtils.test.js b/tests/unit/utils/pageUtils.test.js index a1a0c38..fbd9fa6 100644 --- a/tests/unit/utils/pageUtils.test.js +++ b/tests/unit/utils/pageUtils.test.js @@ -24,9 +24,9 @@ describe("pageUtils", () => { ); }); - it("converts CompareDashboard to DASHBOARD", () => { + it("converts CompareDashboard to PERFORMANCE", () => { expect(mapPageNameToCapitalUnifiedName("CompareDashboard")).toBe( - "DASHBOARD", + "PERFORMANCE", ); });