Skip to content
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

[Bug] Load Segmentation Dialog #4612

Open
xbankov opened this issue Dec 18, 2024 · 2 comments
Open

[Bug] Load Segmentation Dialog #4612

xbankov opened this issue Dec 18, 2024 · 2 comments
Assignees

Comments

@xbankov
Copy link

xbankov commented Dec 18, 2024

Describe the Bug

Answering Yes in Load Segmentation Dialog removes arrows to loop through Segments while No does not.

Steps to Reproduce

  1. Go to https://viewer.ohif.org/segmentation?StudyInstanceUIDs=61.7.110976287009623783394893059131426578073
  2. Manually open the segmentation
  3. Click Yes on Load segmentation Dialog

Same with RTSTRUCT

The current behavior

Arrows to cycle through segments dissappears. I would expect behaviour of Yes and No to be the opposite? Maybe I am wrong of what was intended.

The expected behavior

The arrows don't dissappear and the segmentation opens.

OS

not applicable

Node version

not applicable

Browser

Mozilla Firefox 133.0, Google Chrome 131.0.6778.139

@xbankov xbankov added the Awaiting Reproduction Can we reproduce the reported bug? label Dec 18, 2024
@xbankov xbankov changed the title [Bug] Segmentation Arrows [Bug] Load Segmentation Dialog Dec 18, 2024
@sedghi sedghi removed the Awaiting Reproduction Can we reproduce the reported bug? label Jan 16, 2025
@sedghi sedghi added the Feature Request label Jan 16, 2025 — with Linear
@sedghi sedghi self-assigned this Feb 19, 2025
@YiannisTheocharakis
Copy link
Contributor

@sedghi Hi Alireza. Have you started working on this issue or you want me to take a look? If so what is the preferrable solution? From my point of view I think we should enable arrows even if SEG or RTS is loaded. So to be visible all the time. What is your opinion on that?

@sedghi
Copy link
Member

sedghi commented Feb 27, 2025

I believe that's intentional, not a bug, but I get how it could be helpful. I'll discuss it with the team to see if we want to change it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants