From 5d7fa7c6b66be383279fd830fc82d0e6ce0b58f5 Mon Sep 17 00:00:00 2001 From: Phil Larson Date: Mon, 3 Aug 2026 00:21:14 -0700 Subject: [PATCH] fix(dashboard): align List browser smoke fixture (#3316) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary - mark the native mobile List fixture with the production single-pane class introduced by FN-8754 - add fixture coverage so the production selector cannot drift silently ## Test Plan - `pnpm --filter @fusion/dashboard exec vitest run app/__tests__/browser-layout-smoke-fixture.test.ts --silent=passed-only --reporter=dot` - `pnpm check:changesets` - `pnpm build` - `pnpm --filter @fusion/dashboard test:browser-smoke -- --require-browser` ## Summary by CodeRabbit * **Bug Fixes** * Updated the mobile list layout fixture to include the production single-pane styling class, improving alignment with the app’s mobile layout behavior. * **Tests** * Added a browser smoke test to verify the expected mobile list layout class is generated reliably. --- .../__tests__/browser-layout-smoke-fixture.test.ts | 11 +++++++++++ packages/dashboard/scripts/browser-layout-smoke.mjs | 3 ++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/packages/dashboard/app/__tests__/browser-layout-smoke-fixture.test.ts b/packages/dashboard/app/__tests__/browser-layout-smoke-fixture.test.ts index 571bbaf648..6d727a6623 100644 --- a/packages/dashboard/app/__tests__/browser-layout-smoke-fixture.test.ts +++ b/packages/dashboard/app/__tests__/browser-layout-smoke-fixture.test.ts @@ -84,4 +84,15 @@ describe("browser layout smoke fixture", () => { expect(html).toContain(label); } }); + + /* + FNXC:ListView 2026-08-03-07:00: + The mobile List smoke fixture must carry the production list-view--single-pane marker without coupling the regression to HTML attribute order or spacing. + */ + it("marks the mobile List fixture as the production single-pane surface", () => { + const html = createSmokeHtml(); + const listSectionTag = html.match(/]*\bdata-smoke="list"[^>]*>/)?.[0]; + expect(listSectionTag).toBeDefined(); + expect(listSectionTag).toMatch(/\bclass="[^"]*\blist-view--single-pane\b[^"]*"/); + }); }); diff --git a/packages/dashboard/scripts/browser-layout-smoke.mjs b/packages/dashboard/scripts/browser-layout-smoke.mjs index f836ce14d3..6646ca001a 100644 --- a/packages/dashboard/scripts/browser-layout-smoke.mjs +++ b/packages/dashboard/scripts/browser-layout-smoke.mjs @@ -379,7 +379,8 @@ export function createSmokeHtml() {
${columnMarkup}
-