From ca9e272ad8ac3455ddf970f28d5d02b1ddb962c9 Mon Sep 17 00:00:00 2001 From: Scarriffle Date: Thu, 13 Aug 2026 23:38:40 +0200 Subject: [PATCH] Einstellungen: die Reiter schalten wieder, und die Leiste passt ins Fenster MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Zwei Folgen der eigenen Reiterleiste, beide meine Schuld. Der Reiter oben sprang um, der Inhalt blieb stehen: der gewählte Reiter liegt im Fenster-Controller — er muss dort liegen, damit ein erneutes Öffnen ihn setzen kann, etwa auf „Berechtigungen", wenn eine fehlt. Der Controller war aber nicht beobachtbar. `TabView` fiel das nie auf, weil es seinen Zustand selbst führte und die Bindung nur benachrichtigte. Ein `switch` im Rumpf braucht dagegen eine Änderungsmeldung, und die kam nie. Jetzt ist der Controller `@Observable`, und beide Richtungen funktionieren. Zu breit war sie wegen `fixedSize`: sieben deutsche Beschriftungen brauchen rund 880 Punkte, und die nahm sich die Leiste auch dann, wenn das Fenster 780 breit war — sie ragte links und rechts über die Kante hinaus, der erste und der letzte Reiter waren nicht mehr erreichbar. Ohne feste Breite bleibt sie im Fenster, und das Fenster ist jetzt 920 breit statt 780, mit 900 als Mindestmaß. Darunter würde gestaucht. Co-Authored-By: Claude Opus 5 --- Onyx/SettingsView.swift | 12 +++++++++--- Onyx/SettingsWindowController.swift | 24 ++++++++++++++++-------- 2 files changed, 25 insertions(+), 11 deletions(-) diff --git a/Onyx/SettingsView.swift b/Onyx/SettingsView.swift index 9de829d..64178b2 100644 --- a/Onyx/SettingsView.swift +++ b/Onyx/SettingsView.swift @@ -22,6 +22,8 @@ struct SettingsView: View { let fanControl: FanControl let launchAtLogin: LaunchAtLogin let showOnboarding: () -> Void + /// Der gewählte Reiter liegt im Fenster-Controller, damit ein erneutes + /// Öffnen ihn setzen kann — etwa auf „Berechtigungen", wenn eine fehlt. @Binding var selectedTab: SettingsTab /// Die Reiter in der Reihenfolge, in der sie oben stehen. @@ -50,9 +52,13 @@ struct SettingsView: View { } .pickerStyle(.segmented) .labelsHidden() - // Nur so breit wie nötig und mittig: über die volle Fensterbreite - // gezogen sähen sieben Reiter aus wie eine Werkzeugleiste. - .fixedSize() + // Ohne feste Breite: sieben deutsche Beschriftungen brauchen rund + // 880 Punkte, und mit `fixedSize` nahm sich die Leiste die auch + // dann, wenn das Fenster schmaler war — sie ragte dann links und + // rechts über die Kante hinaus. Das Mindestmaß des Fensters sorgt + // dafür, dass hier trotzdem nichts gestaucht wird. + .frame(maxWidth: 1000) + .padding(.horizontal, 20) .padding(.top, 10) .padding(.bottom, 14) diff --git a/Onyx/SettingsWindowController.swift b/Onyx/SettingsWindowController.swift index 77c73b3..9bfe7df 100644 --- a/Onyx/SettingsWindowController.swift +++ b/Onyx/SettingsWindowController.swift @@ -16,10 +16,17 @@ import ShelfProvider /// /// Ein eigenes `NSWindow` hat diese Abhängigkeit nicht. Es kostet ein paar /// Zeilen mehr und funktioniert dafür. +/// +/// `@Observable`, und das ist keine Kür: der gewählte Reiter liegt hier, weil +/// ein erneutes Öffnen ihn setzen können muss. Ohne Beobachtung bemerkt die +/// View die Änderung nicht — weder die von außen noch die eigene. Genau daran +/// hing das Fenster, in dem der Reiter oben umsprang und der Inhalt stehen +/// blieb. @MainActor +@Observable final class SettingsWindowController: NSObject, NSWindowDelegate { - private var window: NSWindow? + @ObservationIgnored private var window: NSWindow? /// Muss außerhalb der View liegen, damit ein erneutes Öffnen den Reiter setzen kann. private var selectedTab: SettingsTab = .widgets @@ -76,15 +83,16 @@ final class SettingsWindowController: NSObject, NSWindowDelegate { // mehr, und ein Fenster, in dem man scrollen muss, obwohl der // Bildschirm frei ist, ist eine Zumutung. window.styleMask = [.titled, .closable, .miniaturizable, .resizable] - window.setContentSize(NSSize(width: 780, height: 660)) + window.setContentSize(NSSize(width: 920, height: 680)) // Breit genug für die Reiterleiste. // - // Sechs Reiter mit deutschen Beschriftungen brauchen mehr, als man - // beim Nachrechnen der reinen Textbreiten herausbekommt: macOS legt - // je Reiter Innenabstand und Trenner dazu. Ist das Fenster zu - // schmal, schneidet es die Leiste rechts einfach ab — der letzte - // Reiter ist dann nicht mehr erreichbar. - window.contentMinSize = NSSize(width: 740, height: 460) + // Sieben Reiter mit deutschen Beschriftungen brauchen rund 880 + // Punkte — mehr, als man beim Nachrechnen der reinen Textbreiten + // herausbekommt: macOS legt je Reiter Innenabstand und Trenner + // dazu. Ist das Fenster schmaler, ragt die Leiste über beide + // Kanten hinaus, und der erste wie der letzte Reiter sind nicht + // mehr erreichbar. + window.contentMinSize = NSSize(width: 900, height: 460) window.isReleasedWhenClosed = false window.delegate = self Self.place(window)