Skip to content

Feature/query store timerange improvments#140

Merged
erikdarlingdata merged 3 commits intoerikdarlingdata:devfrom
rferraton:feature/query-store-timerange-improvments
Mar 25, 2026
Merged

Feature/query store timerange improvments#140
erikdarlingdata merged 3 commits intoerikdarlingdata:devfrom
rferraton:feature/query-store-timerange-improvments

Conversation

@rferraton
Copy link
Contributor

What does this PR do?

Improve TimeSlicer

  • box selector for time range selection : adapt the selected period
  • tooltips for data and time in RangeTimeSlicer
  • vertical bands to display days inside the RangeTimeSlicer (even with gaps)
  • add a "move bar" at the top of the selected period
  • allow to move RangeTimeSlicer period using keyboards arrows left and right

Which component(s) does this affect?

  • Desktop App (PlanViewer.App)
  • Core Library (PlanViewer.Core)
  • CLI Tool (PlanViewer.Cli)
  • SSMS Extension (PlanViewer.Ssms)
  • Tests
  • Documentation

How was this tested?

2026-03-25_00h11_46

Describe the testing you've done. Include:

  • Plan files tested : QueryStore
  • Platforms tested : Windows Only

Checklist

  • I have read the contributing guide
  • My code builds with zero warnings (dotnet build -c Debug)
  • All tests pass (dotnet test)
  • I have not introduced any hardcoded credentials or server names

- tooltips for data and time in RangeTimeSlicer
- vertical bands to display days (even with gaps)
- allow to move RangeTimeSlicer period using keyboards arrows left and right
Copy link
Owner

@erikdarlingdata erikdarlingdata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good — solid UX improvements to the time range slicer. A few minor nits (DispatcherTimer reuse, missing PointerExited for hover dot, Children[^1] pattern) but nothing blocking. Approving.

@erikdarlingdata erikdarlingdata merged commit 141febd into erikdarlingdata:dev Mar 25, 2026
2 checks passed
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.

2 participants