Merge pull request #202 from charmbracelet/charm-367

Kujtim Hoxha created 

fix(tui): permissions: ensure content viewport has a minimum height

Change summary

internal/tui/components/dialogs/permissions/permissions.go | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

Detailed changes

internal/tui/components/dialogs/permissions/permissions.go 🔗 

@@ -459,7 +459,11 @@ func (p *permissionDialogCmp) render() string {
 	contentFinal := p.getOrGenerateContent()
 
 	// Always set viewport content (the caching is handled in getOrGenerateContent)
-	contentHeight := min(p.height-9, lipgloss.Height(contentFinal))
+	const minContentHeight = 9
+	contentHeight := min(
+		max(minContentHeight, p.height-minContentHeight),
+		lipgloss.Height(contentFinal),
+	)
 	p.contentViewPort.SetHeight(contentHeight)
 	p.contentViewPort.SetContent(contentFinal)