-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Reformat in HP MPR #5717
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Reformat in HP MPR #5717
Conversation
❌ Deploy Preview for ohif-dev failed. Why did it fail? →
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only this file should be left as is on merge.
| viewportId: 'mpr-axial', | ||
| toolGroupId: 'mpr', | ||
| viewportType: 'volume', | ||
| orientation: 'axial', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should setup the appropriate reformat type to switch back and forth
| viewportStructure, | ||
| viewports: [ | ||
| { | ||
| viewportOptions: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is possible the viewportId needs to change when navigating - I hope not since it would also apply other changes.
Context
Adds a second MPR hanging protocol which reformats the page to the reformat orientation.
Changes & Results
Testing
Checklist
PR
semantic-release format and guidelines.
Code
etc.)
Public Documentation Updates
additions or removals.
Tested Environment