MrSphay/odysseus
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix import-review list not scrolling in Brain modal (#509)

Browse Source 
The memory import-review list (.memory-suggestions) is shown inside the
overflow:hidden .admin-card but, unlike the sibling .memory-list, it had
no scroll bounding of its own (no flex:1 / min-height:0 / overflow-y).
A long review list therefore grew past the card and was clipped, leaving
lower entries and their controls unreachable with no usable scroll area.

Give .memory-suggestions the same flex:1 + min-height:0 + overflow-y:auto
bounding the memories list already uses so the review list scrolls
internally within the modal. Pin the review header (the title and the
save all / back controls) with position:sticky so they stay visible while
the items scroll under them, and add a small scrollbar gutter so the bar
does not sit flush against the item cards.

Fixes #455
This commit is contained in:
vidvuds
2026-06-01 16:25:16 +03:00
committed by GitHub
parent 04fd963394
commit 6ad617931d
1 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
static/style.css
View File

@@ -10503,6 +10503,16 @@ textarea.memory-add-input {
display: flex;
flex-direction: column;
gap: 6px;
/* Bound the import-review list to the modal like the sibling .memory-list,
so a long list scrolls internally instead of overflowing the
overflow:hidden .admin-card — which clipped lower entries and their
save/discard controls with no usable scroll area. */
flex: 1;
min-height: 0;
overflow-y: auto;
overflow-x: hidden;
/* Small gutter so the scrollbar doesn't sit flush against the item cards. */
padding-right: 4px;
}
.memory-suggestions.hidden {
@@ -10517,6 +10527,13 @@ textarea.memory-add-input {
color: color-mix(in srgb, var(--fg) 70%, transparent);
padding-bottom: 4px;
border-bottom: 1px solid var(--border);
/* Pin the title + save all/back controls to the top of the scrolling
review list so they stay reachable while the items scroll under them.
Opaque background masks items passing beneath. */
position: sticky;
top: 0;
z-index: 1;
background: var(--panel);
}
.memory-suggestions-actions,
.memory-suggestion-actions {