From 4a812cd89ca8a952649dd1252f76b3f8c331562d Mon Sep 17 00:00:00 2001 From: Brett Chien Date: Sun, 13 Sep 2026 20:35:24 +0800 Subject: [PATCH] fix(console): cap drilldown row height instead of only flooring it MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --drilldown-row-h measures the window's remaining height, but the row only used it as min-height — a long chat transcript pushed the whole row past the window edge instead of clipping and scrolling inside .chat-log. Switch to height so the measured value is a real ceiling too (matches .is-split's behavior below), and give .drilldown-main's content its own overflow-y so capping the row doesn't silently clip a tall left column instead. --- console/src/styles.css | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/console/src/styles.css b/console/src/styles.css index 4d43f07..214bb1e 100644 --- a/console/src/styles.css +++ b/console/src/styles.css @@ -216,7 +216,12 @@ body { align-items: stretch; gap: 0; margin: 0 0 16px; - min-height: var(--drilldown-row-h, auto); + /* Was `min-height` — a floor only, so a long chat transcript (or any other + tall column content) pushed the row past the window instead of clipping + to it (Brett: chat panel not fitting the window / not autoscrolling). + `height` makes the live-measured var a real ceiling too, matching how + `.is-split` below already caps the row while the editor is open. */ + height: var(--drilldown-row-h, auto); } /* Nothing toggles `hidden` on this element today (deploy.ts used to; it now hides `#config`/`#fleet-detail` in place instead, see deploy.ts), but the @@ -252,6 +257,10 @@ body { min-height: 0; display: flex; flex-direction: column; + /* The row's height is now a real ceiling (see `.drilldown-row` above) — give + this column its own scrollbar for content taller than that, same as the + chat log does, instead of silently clipping it. */ + overflow-y: auto; } /* Same equal-specificity trap as `.debug-drawer[hidden]`/`.deploy-identity [hidden]` above — `.drilldown-main > *` and the UA `[hidden]` rule tie, and