From 4ee76fb7666bcb20562cf07d95a9e7c03ed0bb86 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Sat, 28 Jun 2025 12:09:06 +0300 Subject: fix: enable scrolling in help screen viewport MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Fixed viewport update handling in main Update function - Viewport now properly receives mouse and scroll events - Content is set immediately when help is toggled - Resolves issue where help screen couldn't scroll on small terminals 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- internal/ui/keyhandlers.go | 4 +++- internal/ui/table.go | 7 +++++-- 2 files changed, 8 insertions(+), 3 deletions(-) (limited to 'internal') diff --git a/internal/ui/keyhandlers.go b/internal/ui/keyhandlers.go index 428f28e..419bc14 100644 --- a/internal/ui/keyhandlers.go +++ b/internal/ui/keyhandlers.go @@ -121,7 +121,9 @@ func (m *Model) handleToggleHelp() (tea.Model, tea.Cmd) { height = 20 // Default height } m.helpViewport = viewport.New(width, height) - m.helpViewport.SetContent("") // Content will be set in renderHelpScreen + // Set the content immediately + content := m.buildHelpContent() + m.helpViewport.SetContent(content) return m, nil } diff --git a/internal/ui/table.go b/internal/ui/table.go index a0a82c5..a430dcf 100644 --- a/internal/ui/table.go +++ b/internal/ui/table.go @@ -420,9 +420,12 @@ func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) { return m.handleNormalMode(msg) } - // Default case - pass through to table + // Default case - pass through to appropriate component if m.showHelp { - return m, nil + // Update help viewport for mouse wheel and other events + var cmd tea.Cmd + m.helpViewport, cmd = m.helpViewport.Update(msg) + return m, cmd } var cmd tea.Cmd -- cgit v1.2.3