From 94fa9e6a41078289d608f387c316bdc5b0c8d0f1 Mon Sep 17 00:00:00 2001 From: Anders Hafreager Date: Sat, 15 Aug 2026 21:49:19 +0200 Subject: [PATCH 1/2] fix: run console collapsed by default The console strip stole ~190px from the viewport on every run screen. Default it to collapsed; an explicit expand is still remembered for the session. Fixes #406 Co-Authored-By: Claude Fable 5 --- src/shell/RunDetail.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/shell/RunDetail.tsx b/src/shell/RunDetail.tsx index 88085c00..11d37341 100644 --- a/src/shell/RunDetail.tsx +++ b/src/shell/RunDetail.tsx @@ -68,10 +68,10 @@ const RunDetail = ({ runId }: { runId: string }) => { const [frameUrl, setFrameUrl] = useState(null); const [outputs, setOutputs] = useState([]); const consoleRef = useRef(null); - // Collapsible console strip: default expanded, choice remembered for the - // session so the viewport keeps the reclaimed height across run screens. + // Collapsible console strip: default collapsed so the viewport gets the + // height; an explicit expand is remembered for the session. const [consoleCollapsed, setConsoleCollapsed] = useState( - () => sessionStorage.getItem(CONSOLE_COLLAPSED_KEY) === "1", + () => sessionStorage.getItem(CONSOLE_COLLAPSED_KEY) !== "0", ); const toggleConsole = () => setConsoleCollapsed((previous) => { From 2fe26201be0474928ef70b6e365b01e6fd3164d8 Mon Sep 17 00:00:00 2001 From: Anders Hafreager Date: Sat, 15 Aug 2026 22:10:04 +0200 Subject: [PATCH 2/2] test: expand the console before asserting live output (e2e) Review fix: the console now defaults to collapsed, so the e2e run spec expands it before reading the live log; later console steps inherit the remembered expanded state. Co-Authored-By: Claude Fable 5 --- e2e/run.spec.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/e2e/run.spec.ts b/e2e/run.spec.ts index e9741e3f..9f046e3e 100644 --- a/e2e/run.spec.ts +++ b/e2e/run.spec.ts @@ -49,6 +49,9 @@ test("run a simulation end-to-end and inspect the recorded run", async ({ await expect(page.getByTestId("status-pill-running")).toBeVisible({ timeout: 60_000, }); + // The console defaults to collapsed; expand it to watch the live output + // (the choice is remembered for the session, so later steps see it open). + await page.getByTestId("run-console-toggle").click(); // Live console follows the engine output. await expect(page.getByTestId("run-console")).toContainText("LAMMPS", { timeout: 60_000,