lerkolabs/uptop
1
1
Fork 0
Code Issues 17 Pull Requests Actions Packages Projects 2 Releases 4 Wiki Activity

fix(tui): logs tab use viewport for scrollable content
CI / test (pull_request) Successful in 2m39s
Details
CI / lint (pull_request) Successful in 56s
Details
CI / vulncheck (pull_request) Successful in 51s
Details

Browse Source 
Logs were dumping all lines directly, pushing the dashboard
footer off screen. Now uses logViewport with proper height
accounting so footer stays visible and scrolling works.
This commit is contained in:
Tyler Koenig
2026-06-04 15:36:21 -04:00
parent ba75be194d
commit e0f189efe9
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/tui/tab_logs.go
+2 -1
View File
@@ -118,5 +118,6 @@ func (m Model) viewLogsTab() string {
header += subtleStyle.Render(fmt.Sprintf(" (%d hidden)", total-shown)) header += subtleStyle.Render(fmt.Sprintf(" (%d hidden)", total-shown))
} }
return "\n" + header + "\n\n" + strings.Join(rendered, "\n") m.logViewport.SetContent(strings.Join(rendered, "\n"))
return "\n" + header + "\n\n" + m.logViewport.View()
} }
internal/tui/update.go
+1 -1
View File
@@ -127,7 +127,7 @@ func (m *Model) handleResize(msg tea.WindowSizeMsg) (tea.Model, tea.Cmd) {
m.maxTableRows = 1 m.maxTableRows = 1
} }
m.logViewport.Width = msg.Width - chromePadH m.logViewport.Width = msg.Width - chromePadH
m.logViewport.Height = msg.Height - (chromePadV + chromeHeader + chromeGaps + chromeFooter) m.logViewport.Height = msg.Height - (chromePadV + chromeHeader + chromeGaps + chromeFooter + 3)
m.historyViewport.Width = msg.Width - chromePadH m.historyViewport.Width = msg.Width - chromePadH
m.historyViewport.Height = msg.Height - 10 m.historyViewport.Height = msg.Height - 10
m.slaViewport.Width = msg.Width - chromePadH m.slaViewport.Width = msg.Width - chromePadH