Scarriffle/onyx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ladelimit gehört zum Akku, nicht zu den Lüftern

Browse Source 
Der Regler stand in der Lüfterkachel, weil dort der Helfer schon angebunden
war. Das ist die Sicht des Programmierers, nicht die des Nutzers: gesucht
wird er beim Akku, und zwar genau dann, wenn man auf den Ladestand schaut.

Er steht jetzt in der Akku-Kachel des Panels und im Akku-Popover der
Menüleiste. Angebunden über ein Protokoll statt über einen direkten Zugriff:
geschrieben wird vom privilegierten Helfer, und der gehört zur App, nicht zu
MetricsProvider — die Akku-Ansicht soll ihn bedienen können, ohne ihn zu
kennen. Ohne laufenden Helfer ist der Regler sichtbar, aber gesperrt, mit dem
Hinweis, wo er herkommt. Ein Regler, der still nichts tut, ist schlimmer als
einer, der sagt warum.

Dazu Protokollierung, warum das Panel schließt: Zeigerposition,
Auslösefläche, Fensterrahmen. „Es ging zu, obwohl ich noch drin war" ist
ohne diese drei Zahlen eine Behauptung gegen eine andere — und meine
automatisierten Proben halten den Zeiger perfekt still, treffen den Fall
also nie.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Scarriffle
2026-08-11 12:20:00 +02:00
parent a54b332723
commit 5ba9e551a4
13 changed files with 201 additions and 79 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Packages/OnyxKit/Package.swift
View File

@@ -58,7 +58,7 @@ let package = Package(
resources: [.process("Localizable.xcstrings")]),
.testTarget(name: "MediaProviderTests", dependencies: ["MediaProvider"]),
.target(name: "MetricsProvider", dependencies: ["OnyxDesign", "OnyxWidgetKit", "OnyxMenuBar"],
.target(name: "MetricsProvider", dependencies: ["OnyxDesign", "OnyxWidgetKit", "OnyxMenuBar", "OnyxHelperProtocol"],
resources: [.process("Localizable.xcstrings")]),
.testTarget(name: "MetricsProviderTests", dependencies: ["MetricsProvider"]),

92
Packages/OnyxKit/Sources/MetricsProvider/ChargeLimitView.swift Normal file
View File

@@ -0,0 +1,92 @@
import SwiftUI
import OnyxDesign
import OnyxHelperProtocol
/// Wer das Ladelimit setzen kann.
///
/// Ein Protokoll und kein direkter Zugriff: das Limit schreibt der privilegierte
/// Helfer, und der gehört zur App, nicht zu diesem Package. Die Akku-Ansicht
/// soll ihn bedienen können, ohne ihn zu kennen.
@MainActor
public protocol ChargeLimitControlling: AnyObject {
/// Das eingestellte Limit in Prozent, `nil` wenn keines gilt.
var chargeLimit: Int? { get }
/// Ob überhaupt geschrieben werden kann — ohne Helfer geht nichts.
var canControlCharge: Bool { get }
func refreshChargeLimit()
func setChargeLimit(_ percent: Int)
}
/// Der Ladelimit-Regler, wie er im Akku-Widget und im Akku-Popover steht.
///
/// Ein Akku, der dauerhaft bei 100 % am Netz hängt, altert schneller. Deshalb
/// gibt es die Einstellung — und deshalb ist ihr Bereich so schmal: 80 bis
/// 100 % ist, was macOS selbst anbietet und wofür gemessen wurde, wie der
/// zuständige SMC-Key aussieht. Darunter wäre geraten.
public struct ChargeLimitControl: View {
private let control: any ChargeLimitControlling
/// Im Popover ist Platz für die Erklärung, in der Kachel nicht.
private let roomy: Bool
@State private var pending: Double?
public init(control: any ChargeLimitControlling, roomy: Bool) {
self.control = control
self.roomy = roomy
}
private var current: Double {
pending ?? Double(control.chargeLimit ?? ChargeSafety.noLimit)
}
private var isOff: Bool { current >= Double(ChargeSafety.noLimit) }
public var body: some View {
VStack(alignment: .leading, spacing: 5) {
HStack(spacing: 6) {
Text("charge.limit", bundle: .module)
.font(Onyx.Font.caption)
.foregroundStyle(Onyx.Color.textSecondary)
Spacer(minLength: 4)
Group {
// Hundert Prozent ist kein Limit — und soll auch nicht so
// heißen, sonst sucht man den Ausschalter.
if isOff {
Text("charge.limit.off", bundle: .module)
} else {
Text(verbatim: "\(Int(current)) %")
}
}
.font(Onyx.Font.metricSmall).monospacedDigit()
.foregroundStyle(isOff ? Onyx.Color.textTertiary : Onyx.Color.accent)
}
Slider(value: Binding(get: { current }, set: { pending = ($0 / 5).rounded() * 5 }),
in: Double(ChargeSafety.minimumPercent)...Double(ChargeSafety.maximumPercent),
onEditingChanged: { editing in
// Erst beim Loslassen schreiben. Bei jedem Zwischenwert
// in den Ladepfad zu schreiben wäre ein Dutzend
// Schreibvorgänge für eine einzige Entscheidung.
guard !editing, let pending else { return }
control.setChargeLimit(Int(pending))
self.pending = nil
})
.controlSize(.mini)
.disabled(!control.canControlCharge)
if !control.canControlCharge {
Text("charge.limit.needsHelper", bundle: .module)
.font(Onyx.Font.metricSmall)
.foregroundStyle(Onyx.Color.textTertiary)
.fixedSize(horizontal: false, vertical: true)
} else if roomy {
Text("charge.limit.hint", bundle: .module)
.font(Onyx.Font.metricSmall)
.foregroundStyle(Onyx.Color.textTertiary)
.fixedSize(horizontal: false, vertical: true)
}
}
.onAppear { control.refreshChargeLimit() }
}
}

64
Packages/OnyxKit/Sources/MetricsProvider/Localizable.xcstrings
View File

@@ -56,6 +56,70 @@
}
}
},
"charge.limit": {
"localizations": {
"de": {
"stringUnit": {
"state": "translated",
"value": "Ladelimit"
}
},
"en": {
"stringUnit": {
"state": "translated",
"value": "Charge limit"
}
}
}
},
"charge.limit.hint": {
"localizations": {
"de": {
"stringUnit": {
"state": "translated",
"value": "80 bis 100 % — der Bereich, den macOS selbst anbietet."
}
},
"en": {
"stringUnit": {
"state": "translated",
"value": "80 to 100 % — the range macOS itself offers."
}
}
}
},
"charge.limit.needsHelper": {
"localizations": {
"de": {
"stringUnit": {
"state": "translated",
"value": "Braucht den privilegierten Dienst aus Einstellungen → Lüfter."
}
},
"en": {
"stringUnit": {
"state": "translated",
"value": "Needs the privileged service from Settings → Fans."
}
}
}
},
"charge.limit.off": {
"localizations": {
"de": {
"stringUnit": {
"state": "translated",
"value": "Aus"
}
},
"en": {
"stringUnit": {
"state": "translated",
"value": "Off"
}
}
}
},
"metric.battery": {
"localizations": {
"de": {

12
Packages/OnyxKit/Sources/MetricsProvider/MetricPopovers.swift
View File

@@ -1,4 +1,5 @@
import SwiftUI
import OnyxDesign
/// Die Detailansichten hinter den Menüleisten-Symbolen.
///
@@ -15,7 +16,7 @@ struct MetricPopoverContent: View {
switch metric {
case .cpu, .gpu: ProcessorDetail(model: model, metric: metric)
case .memory: MemoryDetail(snapshot: model.snapshot)
case .battery: BatteryDetail(snapshot: model.snapshot)
case .battery: BatteryDetail(snapshot: model.snapshot, control: model.chargeControl)
case .temperature: SensorDetail(snapshot: model.snapshot)
}
}
@@ -231,6 +232,7 @@ private struct MemoryDetail: View {
private struct BatteryDetail: View {
let snapshot: MetricsSnapshot
let control: (any ChargeLimitControlling)?
var body: some View {
if let battery = snapshot.battery {
@@ -256,6 +258,14 @@ private struct BatteryDetail: View {
}
}
// Das Ladelimit steht hier und nicht in den Einstellungen: es ist
// eine Akkusache, und man greift danach, während man auf den
// Ladestand schaut.
if let control {
Divider().overlay(Onyx.Color.hairline)
ChargeLimitControl(control: control, roomy: true)
}
SectionTitle(text: "popover.battery.condition")
if let health = battery.health {
Row(label: String(localized: "popover.battery.health", bundle: .module),

8
Packages/OnyxKit/Sources/MetricsProvider/MetricWidgets.swift
View File

@@ -54,6 +54,14 @@ private struct MetricWidgetView: View {
// alles andere zeigt Zahl und Verlauf.
if metric == .temperature {
SensorList(snapshot: model.snapshot, detailed: true)
} else if metric == .battery, let control = model.chargeControl {
// Die Akku-Kachel bekommt den Ladelimit-Regler direkt: dort
// schaut man ohnehin hin, wenn man ihn braucht.
VStack(alignment: .leading, spacing: 8) {
WideMetric(model: model, metric: metric)
Divider().overlay(Onyx.Color.hairline)
ChargeLimitControl(control: control, roomy: false)
}
} else {
WideMetric(model: model, metric: metric)
}

4
Packages/OnyxKit/Sources/MetricsProvider/MetricsModel.swift
View File

@@ -14,6 +14,10 @@ import SwiftUI
@Observable
public final class MetricsModel {
/// Wer das Ladelimit setzen kann — die App reicht den Helfer herein.
/// `nil` heißt: kein Regler, weil es nichts zu regeln gibt.
public weak var chargeControl: (any ChargeLimitControlling)?
public private(set) var snapshot = MetricsSnapshot()
/// Verlauf für die Graphen, jüngster Wert zuletzt.
public private(set) var history: [MetricsSnapshot] = []

10
Packages/OnyxKit/Sources/OnyxNotch/NotchScreenController.swift
View File

@@ -66,7 +66,11 @@ public final class NotchScreenController {
// MARK: - Eingaben
/// Zeigerposition in globalen Bildschirmkoordinaten.
/// Wo der Zeiger beim letzten Verlassen stand — nur fürs Protokoll.
private var lastPoint: CGPoint = .zero
public func pointerMoved(to point: CGPoint) {
lastPoint = point
// Nur die tatsächlich sichtbare Fläche zählt. Das Fenster ist immer so
// groß wie das ausgefahrene Panel, damit die Animation Platz hat —
// würde man `panel.frame` nehmen, hielte ein Zeiger in der noch leeren
@@ -119,6 +123,12 @@ public final class NotchScreenController {
// umgeschaltet hat oder ob die Unterdrückung dazwischenkam.
if machine.phase != before {
log.notice("Notch \(self.displayID, privacy: .public): \(String(describing: before), privacy: .public) → \(String(describing: self.machine.phase), privacy: .public) durch \(String(describing: input), privacy: .public)")
// Beim Schließen zusätzlich, **warum** der Zeiger als draußen galt.
// Ohne diese drei Zahlen bleibt „es ging zu, obwohl ich noch drin
// war" eine Behauptung gegen eine andere.
if before == .open, self.machine.phase == .closing {
log.notice(" Zeiger \(Int(self.lastPoint.x), privacy: .public)/\(Int(self.lastPoint.y), privacy: .public), Auslösefläche \(String(describing: self.geometry.triggerRect), privacy: .public), Fenster \(String(describing: self.panel.frame), privacy: .public), ausgefahren \(self.presentation.isExpanded, privacy: .public)")
}
}
if machine.isVisible != wasVisible { onVisibilityChanged?(machine.isVisible) }
}