From 2f040bf81047cb348e8e4b0bde9dcaf3b75415bd Mon Sep 17 00:00:00 2001
From: Levi Neely <lneely@pascom.net>
Date: Fri, 2 Oct 2026 12:14:12 +0200
Subject: [PATCH] Make the Settings dialog content scrollable

Adding the Org Path group pushed the settings content past the dialog's
fixed height, so the lower groups and the Close button overflowed the
dialog bounds. Wrap the content ColumnLayout in a ScrollView (vertical
only; the inner layout matches the viewport width so Layout.fillWidth
children still size correctly) and raise the default height from 400 to
520 so the common case fits without scrolling. Content that still exceeds
the dialog now scrolls instead of overflowing.
---
 ui/qml/SettingsDialog.qml | 37 ++++++++++++++++++++++++-------------
 1 file changed, 24 insertions(+), 13 deletions(-)

diff --git a/ui/qml/SettingsDialog.qml b/ui/qml/SettingsDialog.qml
index f6f609d..28986fb 100644
--- a/ui/qml/SettingsDialog.qml
+++ b/ui/qml/SettingsDialog.qml
@@ -9,7 +9,7 @@ Dialog {
     modal: true
     standardButtons: Dialog.Close
     width: 450
-    height: 400
+    height: 520
 
     x: (parent.width - width) / 2
     y: (parent.height - height) / 2
@@ -80,10 +80,20 @@ Dialog {
         }
     }
 
-    contentItem: ColumnLayout {
-        spacing: 16
+    contentItem: ScrollView {
+        id: contentScroll
+        clip: true
+        // Only scroll vertically; the inner layout matches the viewport width.
+        ScrollBar.horizontal.policy: ScrollBar.AlwaysOff
+        contentWidth: availableWidth
 
-        // --- Org Path ---
+        ColumnLayout {
+            // Match the scroll viewport width so Layout.fillWidth children size
+            // correctly and no horizontal scrolling is needed.
+            width: contentScroll.availableWidth
+            spacing: 16
+
+            // --- Org Path ---
         GroupBox {
             title: qsTr("Org Path")
             Layout.fillWidth: true
@@ -296,17 +306,18 @@ Dialog {
             }
         }
 
-        // --- Reset buttons ---
-        RowLayout {
-            Layout.fillWidth: true
-            spacing: 8
+            // --- Reset buttons ---
+            RowLayout {
+                Layout.fillWidth: true
+                spacing: 8
 
-            Button {
-                text: qsTr("Reset Font")
-                onClicked: theme.resetFont()
+                Button {
+                    text: qsTr("Reset Font")
+                    onClicked: theme.resetFont()
+                }
+
+                Item { Layout.fillWidth: true }
             }
-
-            Item { Layout.fillWidth: true }
         }
     }
 }