vrr/codeburn
2
0
Fork 0
mirror of https://github.com/AgentSeal/codeburn.git synced 2026-04-28 06:59:37 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Merge pull request #95 from getagentseal/fix/trend-tooltip-per-provider

Browse source 
fix(mac): show correct cost in trend tooltip for per-provider views
This commit is contained in:
AgentSeal 2026-04-18 13:41:49 -07:00 • committed by GitHub
commit af3676a2b1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

14
mac/Sources/CodeBurnMenubar/Views/HeatmapSection.swift
View file

@ -209,7 +209,7 @@ private struct TrendChart: View {
// Floats below the chart without taking layout space. Opaque dark card hides
// whatever sits beneath it (mini stats, activity rows).
if let hoveredBar {
BarTooltipCard(bar: hoveredBar, formatValue: formatValue)
BarTooltipCard(bar: hoveredBar, value: metric(hoveredBar), formatValue: formatValue)
.padding(.top, 6)
.offset(y: 92)
.transition(.opacity)
@ -260,6 +260,12 @@ private struct BarColumn: View {
private struct BarTooltipCard: View {
let bar: TrendBar
/// Value to display in the tooltip header. Matches the metric the trend chart
/// is currently using (tokens when the .all-providers view has token data,
/// cost when provider-filtered views force a $ fallback). Passing this in keeps
/// the tooltip in sync with the chart instead of always reading bar.tokens,
/// which is zero for provider-filtered days.
let value: Double
let formatValue: (Double) -> String
@Environment(\.colorScheme) private var colorScheme
@ -290,7 +296,7 @@ private struct BarTooltipCard: View {
.font(.system(size: 11, weight: .semibold))
.foregroundStyle(primaryText)
Spacer()
Text("\(formatValue(bar.tokens))")
Text("\(formatValue(value))")
.font(.codeMono(size: 10.5, weight: .semibold))
.foregroundStyle(Theme.brandAccent)
}
@ -313,10 +319,6 @@ private struct BarTooltipCard: View {
}
}
}
} else {
Text("No model breakdown available")
.font(.system(size: 10))
.foregroundStyle(tertiaryText)
}
}
.padding(11)