From 6ad617931d9b24fb09b8cfa6fe30be5758737bba Mon Sep 17 00:00:00 2001
From: vidvuds <77242455+vidvudsc@users.noreply.github.com>
Date: Mon, 1 Jun 2026 16:25:16 +0300
Subject: [PATCH] Fix import-review list not scrolling in Brain modal (#509)

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
---
 static/style.css | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/static/style.css b/static/style.css
index e57b29e..2c8e342 100644
--- a/static/style.css
+++ b/static/style.css
@@ -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 {