Преглед изворни кода

Navbar: do not select the first value on unknown id (#2403)

resolves #2401

Co-authored-by: Fred Lefévère-Laoide <Fred.Lefevere-Laoide@Taipy.io>
Fred Lefévère-Laoide пре 4 месеци
родитељ
комит
1632da3c2a

+ 6 - 6
frontend/taipy-gui/src/components/Taipy/NavBar.spec.tsx

@@ -53,18 +53,18 @@ describe("NavBar Component", () => {
     });
     it("is disabled", async () => {
         const { getAllByRole } = render(<BrowserRouter><NavBar lov={lov} active={false} /></BrowserRouter>);
-        const elts = getAllByRole("tab");
-        elts.forEach(elt => expect(elt).toBeDisabled());
+        const tabElements = getAllByRole("tab");
+        tabElements.forEach(elt => expect(elt).toBeDisabled());
     });
     it("is enabled by default", async () => {
         const { getAllByRole } = render(<BrowserRouter><NavBar lov={lov} /></BrowserRouter>);
-        const elts = getAllByRole("tab");
-        elts.forEach(elt => expect(elt).not.toBeDisabled());
+        const tabElements = getAllByRole("tab");
+        tabElements.forEach(elt => expect(elt).not.toBeDisabled());
     });
     it("is enabled by active", async () => {
         const { getAllByRole } = render(<BrowserRouter><NavBar lov={lov} active={true} /></BrowserRouter>);
-        const elts = getAllByRole("tab");
-        elts.forEach(elt => expect(elt).not.toBeDisabled());
+        const tabElements = getAllByRole("tab");
+        tabElements.forEach(elt => expect(elt).not.toBeDisabled());
     });
     it("dispatch a well formed message", async () => {
         const focusSpy = jest.fn()

+ 1 - 3
frontend/taipy-gui/src/components/Taipy/NavBar.tsx

@@ -66,9 +66,7 @@ const NavBar = (props: LovProps) => {
         [state.locations, navigate]
     );
 
-    const selectedVal =
-        lov.find((it) => getBaseURL() + it.id.substring(1) === location.pathname)?.id ||
-        (lov.length ? lov[0].id : false);
+    const selectedVal = lov.find((it) => getBaseURL() + it.id.substring(1) === location.pathname)?.id || "";
 
     return isMobile ? (
         <Tooltip title={hover || ""}>