Report overview link on case

Clinical-Genomics / scout

VCF visualization interface

https://clinical-genomics.github.io/scout

BSD 3-Clause "New" or "Revised" License

152 stars 46 forks source link

Report overview link on case #4893

Open Jakob37 opened 1 month ago

Jakob37 commented 1 month ago

Describe the solution you'd like

Our geneticists found it hard to navigate to the overview report through panel -> overview report (you lose the case breadcrumb at the top for the case).

Would it be possible to put a link to the overview next to the report in the top left?

top_left

northwestwitch commented 1 month ago

I see, but what happens when you have many panels selected for a case? We have cases with >25 panels, and if you add a link for each one of them in the case sidebar then you need a very long monitor :) Moving this to Scout anyway, so we can have a discussion there!

Jakob37 commented 1 month ago

I see, but what happens when you have many panels selected for a case? We have cases with >25 panels, and if you add a link for each one of them in the case sidebar then you need a very long monitor :) Moving this to Scout anyway, so we can have a discussion there!

Hmm, aha. We normally have a bunch of non-selected panels but just one panel selected. Maybe that comes down to differences in how Lund / Stockholm works again.

Would it be too messy to put a small icon to the right of the panel then?

top_left

I think it would also be OK to leave as is. I think this is more a "nice to have" user ergonomics, than something that would block them.

But maybe open the panel in a new tab instead of navigating away from the case page? So that they don't "get lost" and lose track of the case they were working with.

northwestwitch commented 1 month ago

Mm I wouldn't add it there because of the following reasons:

It is a report, so it should go under the Reports expandable thinghy, I think
If you have a default gene panel with a long name, then there won't be space for the link, example:
Unlike the coverage report on case page (that is created based on all genes from all selected panels), the genes overview report feels more connected to the specific panel in question. I don't know, it has always been like this with the reports from chanjo(1) and I guess our clinicians are used to it..

Of couse we could open the panel in a new tab, it would make it easier to understand which case is which!

Jakob37 commented 1 month ago

Mm I wouldn't add it there because of the following reasons:

OK! Yes, that makes sense. Let's drop that line of thought then.

Of couse we could open the panel in a new tab, it would make it easier to understand which case is which!

OK, nice! I think this would be totally good enough for the Lund geneticists. Let's stick with this then.