From 079969d375ff94d574f3e92b5c89d81da5161059 Mon Sep 17 00:00:00 2001 From: Stu Leak Date: Wed, 31 Dec 2025 15:38:43 -0500 Subject: [PATCH] fix(settings): Fix double scrollbar issue with single scroll container MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Removed individual VScroll containers from each tab - Added single VScroll around entire tabs container - Matches convert module pattern of one scroll for main content - Eliminates overlapping scrollbars and janky scrolling behavior 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 --- settings_module.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/settings_module.go b/settings_module.go index 72ace01..cb969c0 100644 --- a/settings_module.go +++ b/settings_module.go @@ -165,7 +165,10 @@ func buildSettingsView(state *appState) fyne.CanvasObject { ) tabs.SetTabLocation(container.TabLocationTop) - return container.NewBorder(topBar, bottomBar, nil, nil, tabs) + // Single scroll container for entire tabs area + scrollableTabs := container.NewVScroll(tabs) + + return container.NewBorder(topBar, bottomBar, nil, nil, scrollableTabs) } func buildDependenciesTab(state *appState) fyne.CanvasObject { @@ -266,7 +269,7 @@ func buildDependenciesTab(state *appState) fyne.CanvasObject { }) content.Add(refreshBtn) - return container.NewVScroll(content) + return content } func buildPreferencesTab(state *appState) fyne.CanvasObject { @@ -283,7 +286,7 @@ func buildPreferencesTab(state *appState) fyne.CanvasObject { content.Add(widget.NewLabel("• UI theme preferences")) content.Add(widget.NewLabel("• Automatic updates")) - return container.NewVScroll(content) + return content } func (s *appState) showSettingsView() {