Skip to content

Issue #284: Standardize chart save/export filenames#288

Merged
erikdarlingdata merged 1 commit intodevfrom
worktree-issue-284-chart-export
Feb 25, 2026
Merged

Issue #284: Standardize chart save/export filenames#288
erikdarlingdata merged 1 commit intodevfrom
worktree-issue-284-chart-export

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Add chart right-click context menu (Copy Image, Save Image As, Open in New Window, Revert, Export CSV) to all 21 Lite charts — previously Lite had no chart export and defaulted to ScottPlot's ScottPlot.png
  • Standardize on yyyy-MM-dd_HH-mm-ss timestamp format for all chart filenames across both apps
  • Consolidate Dashboard's duplicate SetupChartSaveMenu instance method (~200 lines) into the shared TabHelpers.SetupChartContextMenu
  • Add context menus to 2 Current Waits charts that had none

Closes #284

Test plan

  • Both apps build with 0 errors
  • Lite: right-click any chart shows 5-item context menu
  • Lite: Save Image As defaults to {ChartName}_{yyyy-MM-dd_HH-mm-ss}.png
  • Dashboard: all ServerTab charts use TabHelpers.SetupChartContextMenu
  • Dashboard: Current Waits charts now have context menu
  • Double-click any chart to autoscale/revert

🤖 Generated with Claude Code

… and Lite

Add chart right-click context menu to all 21 Lite charts (Copy Image, Save Image As,
Open in New Window, Revert, Export CSV) with descriptive filenames using yyyy-MM-dd_HH-mm-ss
timestamp format. Consolidate Dashboard's duplicate SetupChartSaveMenu into the shared
TabHelpers.SetupChartContextMenu, and add context menus to the 2 Current Waits charts
that had none.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 822eba3 into dev Feb 25, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the worktree-issue-284-chart-export branch February 25, 2026 17:37
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