Skip to content

Graph Panel widget: add a context menu#395

Merged
agarny merged 5 commits intoopencor:mainfrom
agarny:issue386
Feb 7, 2026
Merged

Graph Panel widget: add a context menu#395
agarny merged 5 commits intoopencor:mainfrom
agarny:issue386

Conversation

@agarny
Copy link
Contributor

@agarny agarny commented Feb 7, 2026

Fixes #386.

Copilot AI review requested due to automatic review settings February 7, 2026 04:56
@agarny agarny changed the title Save figure feature Graph Panel widget: add a context menu Feb 7, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a “save figure / export” feature to GraphPanel plots (Fixes #386) by introducing a right-click context menu for image export, clipboard copy, and CSV data export, plus UI support for showing export progress.

Changes:

  • Add Plotly graph context menu actions (zoom/reset, copy to clipboard, export image formats, export data to CSV).
  • Add a blocking progress message UI and provide it to child components for long-running operations (CSV export).
  • Extend plot trace metadata (xValue/yValue) to support correct CSV headers and update CSP to allow blob-backed images.

Reviewed changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/renderer/src/components/widgets/GraphPanelWidget.vue Adds context menu + image/clipboard/CSV export logic for Plotly graphs.
src/renderer/src/components/views/SimulationExperimentView.vue Populates new trace metadata fields needed for CSV export; tweaks COMBINE archive error message.
src/renderer/src/components/OpenCOR.vue Introduces provided progress-message API and renders a blocking progress overlay.
src/renderer/src/components/BlockingMessageComponent.vue Adds optional determinate progress bar support.
src/renderer/src/common/constants.ts Adds NO_DELAY constant used to yield during CSV export.
src/renderer/src/common/common.ts Adds sleep(ms) helper used by CSV export and progress UI pacing.
src/renderer/index.html Updates CSP to allow blob: images (needed for exported image flows).
src/renderer/package.json Version bump + dependency patch updates.
src/renderer/bun.lock Lockfile updates for renderer deps.
package.json Version bump + dependency patch updates.
bun.lock Lockfile updates for root deps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 11 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Apparently, this is nothing to be concerned with, but at the same time I am not keen on warnings, so...
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 11 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@agarny agarny requested a review from Copilot February 7, 2026 07:15
@agarny agarny merged commit ba9285f into opencor:main Feb 7, 2026
14 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 11 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Graph Panel widget: add a context menu

1 participant