From 72463af5084fbe6f32199dafa00cc09395bbb7d5 Mon Sep 17 00:00:00 2001 From: jiajia Date: Fri, 22 May 2026 17:56:35 +0800 Subject: [PATCH] Show Popi.TV in the header brand The app header still exposed the internal popiart-node label while the requested product brand is Popi.TV. This keeps the existing icon asset but makes it decorative so the visible header label is the accessible brand signal. Constraint: Keep the change scoped to the top-left header logo area. Rejected: Rename all popiart-node strings globally | the request only targeted the top-left Logo. Confidence: high Scope-risk: narrow Directive: Do not broaden this into a product-wide rename without checking welcome and onboarding copy. Tested: npm run test:run -- src/components/__tests__/Header.test.tsx --- src/components/Header.tsx | 4 ++-- src/components/__tests__/Header.test.tsx | 8 +++++--- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/components/Header.tsx b/src/components/Header.tsx index 71f27eb1..b7462612 100644 --- a/src/components/Header.tsx +++ b/src/components/Header.tsx @@ -251,9 +251,9 @@ export function Header() { className="flex items-center gap-2 hover:opacity-80 transition-opacity" title={t("header.openWelcome")} > - PopiArt +

- popiart-node + Popi.TV

diff --git a/src/components/__tests__/Header.test.tsx b/src/components/__tests__/Header.test.tsx index 84a41691..ce618df4 100644 --- a/src/components/__tests__/Header.test.tsx +++ b/src/components/__tests__/Header.test.tsx @@ -75,14 +75,16 @@ describe("Header", () => { describe("Basic Rendering", () => { it("should render the app title", () => { render(
); - expect(screen.getByText("popiart-node")).toBeInTheDocument(); + expect(screen.getByText("Popi.TV")).toBeInTheDocument(); }); it("should render the brand icon", () => { - render(
); - const icon = screen.getByAltText("PopiArt"); + const { container } = render(
); + const icon = container.querySelector('img[src="/banana_icon.png"]'); expect(icon).toBeInTheDocument(); expect(icon).toHaveAttribute("src", "/banana_icon.png"); + expect(icon).toHaveAttribute("alt", ""); + expect(icon).toHaveAttribute("aria-hidden", "true"); }); it("should hide external author link", () => {