root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(dashboard): align List browser smoke fixture (#3316)

Browse Source 
## 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`

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## 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.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
Phil Larson
2026-08-03 00:21:14 -07:00
committed by GitHub
parent cb57093d03
commit 5d7fa7c6b6
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
packages/dashboard/app/__tests__/browser-layout-smoke-fixture.test.ts
View File

@@ -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(/<section\b[^>]*\bdata-smoke="list"[^>]*>/)?.[0];
expect(listSectionTag).toBeDefined();
expect(listSectionTag).toMatch(/\bclass="[^"]*\blist-view--single-pane\b[^"]*"/);
});
});

3
packages/dashboard/scripts/browser-layout-smoke.mjs
View File

@@ -379,7 +379,8 @@ export function createSmokeHtml() {
<main class="project-content project-content--with-footer project-content--with-mobile-nav">
<section class="board" data-smoke="board">${columnMarkup}</section>
<section class="list-view" data-smoke="list" hidden>
<!-- FNXC:ListView 2026-08-03-06:41: Mirror ListView's measured single-pane marker so the native mobile smoke exercises the production CSS selector introduced by FN-8754 instead of showing both the table and cards. -->
<section class="list-view list-view--single-pane" data-smoke="list" hidden>
<div class="list-create-area">
<div class="quick-entry-box quick-entry-box--collapsed" data-testid="quick-entry-box">
<div class="quick-entry-main-row">