Console: layout fixes from the 2026-10-23 review #912

Part of #906.

Scope

Seven layout faults in the console are fixed. After this PR the page no longer shifts when a scrollbar appears; the worker page has no breadcrumb; the warm-pool tab has a line between its summary and its table; the fleet table's row ⋯ sits inside its cell; Inbox row buttons have room above and below; a toast's close button sits beside its text, not over it; and a ⋯ list never repeats the button beside it. UI only: no route, contract or daemon change.

In short

The console's layout stops shifting, cramping and overlapping in the seven places the 2026-10-23 review found. The one change in behaviour: a ⋯ list beside a button (in a page header, a drawer or an Inbox row) no longer lists that button's action. When that leaves the list empty, the ⋯ is not drawn.

flowchart LR
  E[Item's actions] --> B[First safe action → button]
  E --> M["⋯ list: the rest (changed: without the button's action)"]
  M -->|empty| N[No ⋯ drawn]

Before and after, a drained worker's page header:

If this goes wrong, a user would notice an action missing from both the button and the ⋯, or a box that moved.

Technical spec

Facts

Modules touched

Searches: rg "crumbs|table-menu|inbox-actions|toast-close|ActionBar|scrollbar" over ui/src, e2e/console, docs/internal/DESIGN.md, website/src/content/docs.

flowchart LR
  WP[screens/fleet/worker-page.tsx *] --> DS[ds/menu.tsx ActionBar *]
  IN[screens/inbox/index.tsx *] --> DS
  IN --> A[model/actions.ts]
  WP --> T[ds/table.css *]
  TO[ds/toast.tsx, toast.css *]
  F[ds/foundation.css *]
  MD[ds/master-detail.css *]

Also screens/inbox/inbox.css, screens/fleet/fleet.css (warm divider), docs/internal/DESIGN.md (Menu and Toast paragraphs), e2e/console/long-lists.spec.ts, e2e/console/workers.spec.ts, e2e/console/actions.spec.ts, and a new e2e/console/layout.spec.ts. The Inbox row drops the button's entry in screens/inbox/index.tsx (InboxActions), not in inboxActions or state/inbox.ts, so the Inbox drawer keeps Undrain (as its ActionBar button) and #915's files are untouched (F17).

Contract and event changes

None (searched: no file under src/ changes).

Other code on the same state

UI state only: the ⋯ list's entries. Searched rg "RowMenu|ActionBar|actions.menu" ui/src (12 hits). Every caller that shows a button beside a ⋯ goes through ActionBar (F10) or InboxActions (F9); DataTable's row ⋯ shows no button beside it and keeps every entry. #910 adds tabs to the device drawer and keeps its ActionBar as is; no shared lines.

Failure modes

None: the task crosses no module boundary and writes no state.

Rules in play

• DESIGN.md Spacing: "Pick from the scale; never a value between" — every new value is a --space-* token.
• DESIGN.md Spacing: "Buttons and inputs are at least 40px tall" — the toast close keeps a 40px hit area; only its border goes.
• DESIGN.md Menu: "A list with no entry draws no ⋯" — kept; the paragraph gains "and the list does not repeat that action".
• documentation rule 9 — the console's public look changes in no documented way (console.mdx names no ⋯ entries; checked rg "⋯|menu" website/src/content/docs/docs/worker/console.mdx), so no website page changes.

Tests

Seam: the browser lane, e2e/console/ against a started daemon (model: e2e/console/long-lists.spec.ts), plus ActionBar with renderToStaticMarkup (model: ui/src/ds/chart.test.tsx).

• The page reserves room for a scrollbar: the computed scrollbar-gutter of html is stable on every page (Overview, Fleet, a worker page). A layout measure cannot fail in the lane, which hides scrollbars (F13).
• An operator opens a worker page: no element named "Breadcrumb"; on the Warm pool tab a 1px --border line sits between the facts and the table. The tests that clicked the breadcrumb to return with the Fleet's page kept (long-lists.spec.ts:378-423, workers.spec.ts:933,1133) use the browser's Back, which keeps it; the keyboard-only test (workers.spec.ts:1252,1290-1293) follows the sidebar's Fleet link by keyboard.
• An operator looks at fleet and Inbox rows: the fleet row's ⋯ lies inside its cell with 24px to the table's right edge; each Inbox row button has 8px between it and the row's top and bottom lines.
• An operator sees a toast: the close button lies inside the toast's box; the title's box does not overlap it; on a one-line toast the text's and the button's vertical centres are within 4px. Cases: a one-line success toast, a drain toast with a line and Undo.
• An operator opens a ⋯ beside a button: drained, connected worker page header → Undrain button and no ⋯, its only action being Undrain (actions.spec.ts:477-478); drained, disconnected worker → Undrain button, ⋯ entries ["Remove worker"] (actions.spec.ts:485); Inbox "drained" row → Undrain button, ⋯ without Undrain; Inbox drained drawer → Undrain button still there; lease drawer, whose only action is dangerous → no button, ⋯ keeps Release lease (as actions.spec.ts:460-462 pins); an ActionBar whose only safe entry is the button draws no ⋯ (static markup, which can see the trigger, F14).

Done when

• pnpm test and pnpm test:console pass, with the five Tests lines above.
• DESIGN.md's Menu paragraph says the ⋯ beside a button does not repeat it; its Toast paragraph says the close button sits beside the text.
• rg -n 'crumb' ui/src finds nothing.

Out of scope

• Removing WorkerPageView.back, which only the breadcrumb read: it sits in useWorkerPage, whose reads The console reads per page through a query cache, and events mark data stale #914 rewrites, so it stays until a later change (F16).

• Any change to which actions an item has, or their order.

• The device drawer's tabs and Screen (Console: watch a device from its drawer's Screen tab; the lease stream route goes #910), the worker Events tab (Console: the worker's Events tab scrolls back through its history #911).

• The toast hover timing (Console e2e: a success toast leaves after 6 s fails when the mouse rests over the new toast #900, PR fix(e2e): move the mouse off the menu so a new toast is not hovered (#900) #903).

Depends on

Written by an agent.