Clean up Settings layout

josh · Apr 24, 2024 · 77accfe · 77accfe
1 parent 872fdaa
commit 77accfe
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 7 deletions.
diff --git a/Aware/macOS/MenuBar.swift b/Aware/macOS/MenuBar.swift
@@ -89,6 +89,7 @@ private func findStatusBarItem() -> NSStatusItem? {
 struct MenuBarContentView: View {
     var body: some View {
         SettingsLink()
+            .keyboardShortcut(",")
         // FIXME: Remove before next release
         Button("Export Logs") {
             Task<Void, Never>(priority: .background) {

diff --git a/Aware/macOS/SettingsView.swift b/Aware/macOS/SettingsView.swift
@@ -18,27 +18,49 @@ struct SettingsView: View {
 
     var body: some View {
         Form {
-            Section(header: Text("Timer")) {
-                Picker("Format Style", selection: $timerFormatStyle) {
+            Section {
+                Picker("Format Style:", selection: $timerFormatStyle) {
                     ForEach(TimerFormatStyle.Style.allCases, id: \.self) { style in
                         Text(style.exampleText)
                     }
                 }
 
                 Toggle("Show Seconds", isOn: $showSeconds)
+            }
 
-                TextField(value: $userIdleSeconds, format: .number) {
-                    Text("Idle Seconds")
-                }
+            Spacer()
+                .frame(width: 0, height: 0)
+                .padding(.top)
 
-                Toggle("Open at Login", isOn: openAtLogin)
-                    .toggleStyle(.checkbox)
+            Section {
+                LabeledContent("Reset after:") {
+                    TextField("Idle Seconds", value: $userIdleSeconds, format: .number)
+                        .multilineTextAlignment(.trailing)
+                        .labelsHidden()
+                        .frame(width: 50)
+                    Stepper("Idle Seconds", value: $userIdleSeconds, step: 30)
+                        .labelsHidden()
+                    Text("seconds of inactivity")
+                        .padding(.leading, 5)
+                }
 
                 Button("Reset Timer") {
                     self.resetTimer = true
                 }
             }
+
+            Spacer()
+                .frame(width: 0, height: 0)
+                .padding(.top)
+
+            Section {
+                LabeledContent("Login Item:") {
+                    Toggle("Open at Login", isOn: openAtLogin)
+                }
+            }
         }
+        .padding()
+        .frame(width: 350)
     }
 
     var openAtLogin: Binding<Bool> {