Merge pull request 'fix(explore): bound 3-pane grid row so a tall rail can't scroll the page' (#135) from dev into main
Build images / sign-extension (push) Successful in 3s
CI / lint (push) Successful in 3s
CI / frontend-build (push) Successful in 23s
CI / backend-lint-and-test (push) Successful in 28s
Build images / build-web (push) Successful in 2m5s
Build images / build-ml (push) Successful in 2m37s
CI / integration (push) Successful in 3m20s

This commit was merged in pull request #135.
This commit is contained in:
2026-06-26 23:37:43 -04:00
+9 -1
View File
@@ -235,11 +235,17 @@ onUnmounted(() => document.removeEventListener('keydown', onKeyDown))
.fc-ex__crumb:focus-visible { outline: 2px solid rgb(var(--v-theme-accent)); outline-offset: 1px; }
.fc-ex__reseed { margin-left: auto; }
/* The three panes fill the remaining height; each scrolls on its own. */
/* The three panes fill the remaining height; each scrolls on its own.
grid-template-rows: minmax(0, 1fr) BOUNDS the single row to the container
height — without it the row is `auto` and a tall pane (Provenance + Tags +
a long Suggestions list) grows the row past the fixed-height view, spilling
over and scrolling the whole page (the weird top gap). Bounded, the panes'
own overflow-y:auto kicks in and they scroll internally instead. */
.fc-ex__panes {
flex: 1 1 auto; min-height: 0;
display: grid;
grid-template-columns: 340px minmax(0, 1fr) var(--fc-side-w, 320px);
grid-template-rows: minmax(0, 1fr);
gap: 1px;
background: rgb(var(--v-theme-surface-light));
}
@@ -303,6 +309,8 @@ onUnmounted(() => document.removeEventListener('keydown', onKeyDown))
.fc-ex { height: auto; }
.fc-ex__panes {
grid-template-columns: 1fr;
/* Stacked: let each pane take its natural height and the page scroll. */
grid-template-rows: none;
background: transparent;
}
.fc-ex__viewer { order: -1; min-height: 60vh; }