Skip to content

Fix duration chart hover, Dashboard grid lines, time range button hover#132

Merged
erikdarlingdata merged 1 commit intodevfrom
fix/feedback-round1
Feb 19, 2026
Merged

Fix duration chart hover, Dashboard grid lines, time range button hover#132
erikdarlingdata merged 1 commit intodevfrom
fix/feedback-round1

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Lite duration chart hover: Wire ChartHoverHelper.Clear()/Add() into all 4 duration chart update methods (collector, query, proc, query store) — hover tooltips now show series name, value, and timestamp
  • Dashboard grid lines: Bump alpha from 40 to 50 (40 was still invisible on Dashboard)
  • Time range button hover: New TimeRangeButton style with accent blue border + text brighten on hover for the hour picker buttons

Fixes user feedback from testing. Partial fix for #110.

Test plan

  • Lite: Collection Health > Duration Trends — hover over chart lines, tooltip should appear with collector name + ms value + time
  • Lite: Query Stats > Duration Trend, Proc Duration Trend, Query Store Duration Trend — same hover behavior
  • Dashboard: Charts should have faintly visible grid lines
  • Dashboard: Hover over hour picker buttons (1 Hour, 4 Hours, etc.) — should show accent blue border + brighter text

🤖 Generated with Claude Code

…utton hover

- Wire ChartHoverHelper Clear/Add calls into all 4 Lite duration chart
  update methods (collector, query, proc, query store) — hover tooltips
  now show series name, value, and timestamp
- Bump Dashboard chart grid line alpha from 40 to 50 (was invisible)
- Add TimeRangeButton style with accent border + text brighten on hover
  for the hour picker buttons (1h/4h/8h/12h/24h/7d/30d)

Partial fix for #110, fixes Lite #121

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 08c22b7 into dev Feb 19, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the fix/feedback-round1 branch February 20, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant