Skip to content

feat(mecatui): unify bounded scrolling and cursor selection #1615

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
jbeda wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

feat(mecatui): unify bounded scrolling and cursor selection #1615

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
d26eff1
docs(acceptance): amend scroll cursor anchors
jbeda Sep 15, 2026
8ef01a4
feat(mecatui): establish bounded scroll cursor control
jbeda Sep 15, 2026
c8058e6
fix(mecatui): reverse-page oversized cursor items
jbeda Sep 15, 2026
a2fa245
refactor(mecatui): split bounded viewport and list
jbeda Sep 15, 2026
29f07bc
feat(mecatui): adopt bounded browsing in agents and models
jbeda Sep 15, 2026
4d8d791
fix(mecatui): unify panel viewport navigation
jbeda Sep 15, 2026
c82b544
fix(mecatui): preserve bounded panel anchors
jbeda Sep 15, 2026
60cfbc0
fix(mecatui): finalize bounded selection mechanics
jbeda Sep 15, 2026
547c89e
fix(mecatui): preserve rendered list anchors
jbeda Sep 15, 2026
caed7fd
docs(acceptance): mark scroll cursor slice landed
jbeda Sep 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 10 additions & 3 deletions cmd/mecatui/ui/agents_detail_scrolling_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,13 @@ func testAgentsDetailScrolling(t *testing.T) {
m = tc.setup(m)

m = pressDetail(m, 'e')
activeOffset := m.team.detail.offset
if m.agentsTab == tabSubagents {
activeOffset = m.subagents.detail.offset
}
if activeOffset == 0 {
t.Fatal("JumpEnd did not update boundedViewport source offset")
}
end := stripANSIstr(m.View().Content)
if !strings.Contains(end, "detail-29") || strings.Contains(end, "detail-00") {
t.Fatalf("JumpEnd did not reveal only the final hidden content:\n%s", end)
Expand All @@ -72,7 +79,7 @@ func testAgentsDetailScrolling(t *testing.T) {
m = pressDetail(m, 'u')
up := stripANSIstr(m.View().Content)
if up == end {
t.Fatalf("Up did not scroll one rendered line (sub=%d team=%d):\n%s", m.subagents.scroll, m.team.scroll, up)
t.Fatalf("Up did not scroll one rendered line (sub=%d team=%d):\n%s", m.subagents.detail.offset, m.team.detail.offset, up)
}
m = pressDetail(m, 'p')
pageUp := stripANSIstr(m.View().Content)
Expand Down Expand Up @@ -100,9 +107,9 @@ func testAgentsDetailScrolling(t *testing.T) {
}
_, resizedWindow := m.agentsDetailMetrics()
wantMax := maxScrollOffset(30, resizedWindow)
gotOffset := m.team.scroll
gotOffset := m.team.detail.offset
if m.agentsTab == tabSubagents {
gotOffset = m.subagents.scroll
gotOffset = m.subagents.detail.offset
}
if gotOffset != wantMax {
t.Fatalf("resize left offset %d, want clamped %d", gotOffset, wantMax)
Expand Down
Loading
Loading