From d26af59f149551cdbdb56576b38314131b4c0d57 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Sun, 3 May 2026 23:39:25 -0700 Subject: [PATCH] fix(tui): remove generic detail pane prefix --- tui/tui.go | 4 ++-- tui/tui_test.go | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tui/tui.go b/tui/tui.go index 5546088..5a4241e 100644 --- a/tui/tui.go +++ b/tui/tui.go @@ -1929,11 +1929,11 @@ func (m *model) syncDetailViewport() { } func (m *model) configureDetailViewport(rect rect, lines []string) { - title := m.detailPaneTitle() + " " + detailModeLabel(m.compactDetail) + title := detailModeLabel(m.compactDetail) if focus := paneFocusLabel(m.focus == focusDetail); focus != "" { title += " " + focus } - content := append([]string{paneTitleForWidth(focusDetail, m.focus, title, paneContentWidth(rect.w))}, lines...) + content := append([]string{paneTitleWithLabelForWidth(m.detailPaneTitle(), focusDetail, m.focus, title, paneContentWidth(rect.w))}, lines...) m.detailView.Width = paneContentWidth(rect.w) m.detailView.Height = maxInt(1, rect.h-2) m.detailView.MouseWheelEnabled = true diff --git a/tui/tui_test.go b/tui/tui_test.go index 9a68f2f..fed51dd 100644 --- a/tui/tui_test.go +++ b/tui/tui_test.go @@ -1108,7 +1108,7 @@ func TestDetailPaneTitleFollowsLayout(t *testing.T) { }) chat.width = 160 chat.height = 24 - if view := stripANSI(chat.View()); !strings.Contains(view, "Thread full") { + if view := stripANSI(chat.View()); !strings.Contains(view, "Thread full") || strings.Contains(view, "Detail Thread full") { t.Fatalf("chat detail pane should be labeled as a thread:\n%s", view) } @@ -1119,7 +1119,7 @@ func TestDetailPaneTitleFollowsLayout(t *testing.T) { }) doc.width = 160 doc.height = 24 - if view := stripANSI(doc.View()); !strings.Contains(view, "Page full") { + if view := stripANSI(doc.View()); !strings.Contains(view, "Page full") || strings.Contains(view, "Detail Page full") { t.Fatalf("document detail pane should be labeled as a page:\n%s", view) } }