Viewers icon indicating copy to clipboard operation
Viewers copied to clipboard

[Bug] Incorrect crosshairs when 2nd series is loaded in MPR mode

Open 03shek opened this issue 1 year ago • 3 comments
trafficstars

Describe the Bug

MPR crosshairs is not resetting when 2nd series is loaded in MPR mode, leading to incorrect crosshairs across axial/sagittal/coronal views

Steps to Reproduce

  1. Load a PET-CT study https://viewer-dev.ohif.org/viewer?StudyInstanceUIDs=1.3.6.1.4.1.14519.5.2.1.7009.2403.334240657131972136850343327463

  2. Load CT series into viewport

  3. Load MPR mode

  4. Enable crosshairs

  5. Move crosshairs to Right shoulder in coronal view crosshairs_1_loadedfirstseriesMPR

  6. Do not disable crosshairs. Exit out of MPR mode.

  7. Load PET series into viewport

  8. Load MPR mode. Crosshairs is already visible, but button is not enabled.

  9. Crosshairs is also not centered on the FOV of this new series; but still on Right shoulder in coronal panel, and now misaligned in axial and sagittal panel crosshairs_1_loadedsecondseriesMPR crosshairs_1_loadedsecondseriesMPRnowwrong

The current behavior

Crosshairs is persisting from last MPR mode (different series) and 1) not disabled, and 2) not centered on the current series's FOV.

The expected behavior

Like in v3.7.0, if a series different from the last is opened in MPR mode, no crosshairs should be visible immediately after opening MPR mode. And crosshairs should need manual enabling to become visible. Once enabled, it should default to the center of the FOV.

OS

Windows 10

Node version

3.8.0-beta.51

Browser

Chrome 120.0.0

03shek avatar Jan 26 '24 00:01 03shek

@dxlin , please look into this.

jenny-hm-lee avatar Feb 12 '24 16:02 jenny-hm-lee

i think this is fixed in master, re open if still an issue

sedghi avatar May 04 '24 17:05 sedghi

@sedghi , please re-open this issue. See #4088

jenny-hm-lee avatar May 09 '24 20:05 jenny-hm-lee

This has been fixed by the crosshair changes in https://github.com/OHIF/Viewers/pull/4113

IbrahimCSAE avatar May 29 '24 12:05 IbrahimCSAE