vrr/codeburn
2
0
Fork 0
mirror of https://github.com/AgentSeal/codeburn.git synced 2026-08-20 14:04:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

fix(menubar): Usage Refresh picker label now reflects the selection

Browse source 
The computed Binding wrote UserDefaults but nothing invalidated the
view, so the picker visually stayed on Auto while the value silently
landed. Back it with AppStorage so selection updates render.

Refs #647
This commit is contained in:
AgentSeal 2026-07-09 22:38:46 +02:00
parent 121181e13b
commit 3befd8554c
1 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

10
mac/Sources/CodeBurnMenubar/Views/SettingsView.swift
View file

@ -46,6 +46,12 @@ private struct GeneralSettingsTab: View {
@State private var costText = ""
@State private var tokenText = ""
// AppStorage (not a computed Binding over UsageRefreshCadence.current):
// a plain UserDefaults write does not invalidate the view, so the picker
// label would never reflect the selection even though the value landed.
@AppStorage(UsageRefreshCadence.defaultsKey)
private var usageRefreshSeconds: Int = UsageRefreshCadence.default.rawValue
private let costPresets: Set<Double> = [25, 50, 100, 200, 500]
private let tokenPresets: Set<Double> = [1_000_000, 5_000_000, 10_000_000, 25_000_000, 50_000_000, 100_000_000]
@ -124,8 +130,8 @@ private struct GeneralSettingsTab: View {
Section("Usage Refresh") {
Picker("Update every", selection: Binding(
get: { UsageRefreshCadence.current },
set: { UsageRefreshCadence.current = $0 }
get: { UsageRefreshCadence(rawValue: usageRefreshSeconds) ?? .default },
set: { usageRefreshSeconds = $0.rawValue }
)) {
ForEach(UsageRefreshCadence.allCases) { cadence in
Text(cadence.label).tag(cadence)