Skip to content

Conversation

@wayfarer3130
Copy link
Contributor

Context

Add a copy to clipboard and fix a couple of rendering issues with how the capture viewport is setup/managed.

Changes & Results

Click on capture on an MPR oblique viewport, the capture image should look like the source image.
Click on copy to clipboard, it should store the image to be pasteable to another application.

Testing

Checklist

PR

  • [] My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • [] My code has been well-documented (function documentation, inline comments,
    etc.)

Public Documentation Updates

  • [] The documentation page has been updated as necessary for any public API
    additions or removals.

Tested Environment

  • [] OS:
  • [] Node version:
  • [] Browser:

@wayfarer3130 wayfarer3130 requested a review from jbocce January 16, 2026 01:01
@netlify
Copy link

netlify bot commented Jan 16, 2026

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit cefb30d
🔍 Latest deploy log https://app.netlify.com/projects/ohif-dev/deploys/69698ddf4f972b00085bbb7c
😎 Deploy Preview https://deploy-preview-5720--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cypress
Copy link

cypress bot commented Jan 16, 2026

Viewers    Run #5866

Run Properties:  status check passed Passed #5866  •  git commit cefb30dcd7: Add copy to clipboard option for capture
Project Viewers
Branch Review fix/save-jpeg
Run status status check passed Passed #5866
Run duration 02m 15s
Commit git commit cefb30dcd7: Add copy to clipboard option for capture
Committer Bill Wallace
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 37
View all changes introduced in this branch ↗︎

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