From d4a0b88d9f4ddb7c4249dff0818056503feb8a0c Mon Sep 17 00:00:00 2001 From: armada-lookout Date: Tue, 21 Jul 2026 13:07:38 +0000 Subject: [PATCH] =?UTF-8?q?XS=E2=9C=94=20=E2=97=BE=20Replace=20stop=20emoj?= =?UTF-8?q?i=20with=20sad=20face=20and=20remove=20dash=20in=20My=20Shaves?= =?UTF-8?q?=20empty=20state?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The empty state used a stop-sign emoji with a dash before the message, which reads as harsh for a first-run/no-data state. Swap in a sad face emoji and drop the dash so it reads naturally. Closes #966 --- .../src/components/shaves/NoShaves.test.tsx | 21 +++++++++++++++++++ src/ui/src/components/shaves/NoShaves.tsx | 2 +- 2 files changed, 22 insertions(+), 1 deletion(-) create mode 100644 src/ui/src/components/shaves/NoShaves.test.tsx diff --git a/src/ui/src/components/shaves/NoShaves.test.tsx b/src/ui/src/components/shaves/NoShaves.test.tsx new file mode 100644 index 00000000..e199a34d --- /dev/null +++ b/src/ui/src/components/shaves/NoShaves.test.tsx @@ -0,0 +1,21 @@ +import { render, screen } from "@testing-library/react"; +import { describe, expect, it } from "vitest"; +import { NoShaves } from "./NoShaves"; + +describe("NoShaves (#966)", () => { + it("uses a sad face emoji instead of the stop-sign emoji (AC1)", () => { + render(); + + const title = screen.getByText(/You don't have any YakShaves yet!/i); + expect(title.textContent).toContain("😢"); + expect(title.textContent).not.toContain("⛔️"); + }); + + it("has no dash between the emoji and the message text (AC2)", () => { + render(); + + const title = screen.getByText(/You don't have any YakShaves yet!/i); + expect(title.textContent).toBe("😢 You don't have any YakShaves yet!"); + expect(title.textContent).not.toMatch(/-\s*You don't have any YakShaves yet!/); + }); +}); diff --git a/src/ui/src/components/shaves/NoShaves.tsx b/src/ui/src/components/shaves/NoShaves.tsx index c01ea7dc..3d212d15 100644 --- a/src/ui/src/components/shaves/NoShaves.tsx +++ b/src/ui/src/components/shaves/NoShaves.tsx @@ -10,7 +10,7 @@ export function NoShaves() { return ( - ⛔️ - You don't have any YakShaves yet! + 😢 You don't have any YakShaves yet! Get started in 3 easy steps: