From af98bad9dea42ed28b58affd9fc7f6f04b975f13 Mon Sep 17 00:00:00 2001 From: Michael Lam Date: Fri, 8 May 2026 09:19:41 -0700 Subject: [PATCH] fix: make kanban detail view scrollable --- static/style.css | 2 +- tests/test_kanban_ui_static.py | 11 +++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/static/style.css b/static/style.css index 5ca1ebfa..26e32b05 100644 --- a/static/style.css +++ b/static/style.css @@ -2275,7 +2275,7 @@ main.main.showing-settings > #mainSettings{display:flex;overflow-y:auto;} main.main.showing-skills > #mainSkills{display:flex;} main.main.showing-memory > #mainMemory{display:flex;} main.main.showing-tasks > #mainTasks{display:flex;} -main.main.showing-kanban > #mainKanban{display:flex;} +main.main.showing-kanban > #mainKanban{display:flex;overflow-y:auto;} main.main.showing-workspaces > #mainWorkspaces{display:flex;} main.main.showing-profiles > #mainProfiles{display:flex;} main.main.showing-logs > #mainLogs{display:flex;} diff --git a/tests/test_kanban_ui_static.py b/tests/test_kanban_ui_static.py index b9ed8372..89e019ba 100644 --- a/tests/test_kanban_ui_static.py +++ b/tests/test_kanban_ui_static.py @@ -93,6 +93,17 @@ def test_kanban_board_has_native_css_classes(): assert "overflow-x:auto" in COMPACT_STYLE +def test_kanban_main_view_scrolls_when_task_preview_is_tall(): + """The app shell keeps body overflow hidden, so the Kanban main view + must own vertical scrolling. Otherwise a selected task with a long body + can push the board below the viewport with no way to reach it. + """ + assert re.search( + r"main\.main\.showing-kanban\s*>\s*#mainKanban\s*\{[^}]*display:flex;[^}]*overflow-y:auto;", + COMPACT_STYLE, + ), "Kanban main view must expose a vertical scrollbar when detail content is taller than the viewport" + + def test_kanban_i18n_keys_exist_in_every_locale_block(): locale_blocks = re.findall(r"\n\s*([a-z]{2}(?:-[A-Z]{2})?): \{(.*?)\n\s*\},", I18N, flags=re.S) assert len(locale_blocks) >= 8