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

[Feature Request] When having multiple studies have a different color outline on viewports #4031

Open
NurGVaras opened this issue Apr 11, 2024 · 5 comments

Comments

@NurGVaras
Copy link

What feature or change would you like to see made?

We are using the functionality to show multiple studies at the view giving different StudyInstancesUIds, the problem is that the professionals use the viewer and put the images in the same layout, and they want to distinguish between studies to minimize errors.

Any chance that in the viewports use different colored outlines for different StudyInstancesUIds?

Why should we prioritize this feature?

This would be helpful to reduce cross examination errors

@sedghi
Copy link
Member

sedghi commented May 4, 2024

I think it is useful feature, and i have seen it in other viewers too. Although i have seen they have different color for overlay text and not bprder

@NurGVaras NurGVaras reopened this May 4, 2024
@NurGVaras
Copy link
Author

yeah the idea is to show that are from different studies to avoid cross examinations, that could work too, sorry for the miss click

@sedghi
Copy link
Member

sedghi commented May 6, 2024

I think it is valuable. We will triage and see when we can do it

@sedghi
Copy link
Member

sedghi commented May 6, 2024

@salimkanoun Could you help us gather requirements for this project? What insights have you observed from viewers?

@salimkanoun
Copy link
Contributor

We did this in our distribution, it can be donne by adding a custom attribute to the Hanging protocol for instance "borderColor" and make a custom viewport that will inject this property in the rendered viewport.

So basically in OHIF you can just add a default "borderColor" porperty to the defautl viewport and add the tailwind classname according to the predefined list of colors.

Then each user can use or not this attribute to make this color border and chose it's own color.

I don't think viewport should have this by default, colorborder can also overload display not everyone will like it.

Other way could be to change color of the seriesDescription, or seriesDate to differentiate viewport.

The point is, OHIF is flexible enough to do all of this by customizing viewport and using custom HP properties.

Eventually add an optional borderColor properties can make it easier to implement

@sedghi sedghi added the Feature Request label Mar 28, 2025 — with Linear
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