search

cBioPortal / cbioportal

cBioPortal for Cancer Genomics
https://cbioportal.org
GNU Affero General Public License v3.0
581 stars 448 forks source link

hide timeline events availability chart by default #10733

Open inodb opened 3 months ago

inodb commented 3 months ago

Hide this chart by default:

image

See this study:

https://www.cbioportal.org/study/summary?id=lgg_ucsf_2014

Requested by GENIE

Nithin9585 commented 3 months ago

Hiding the chart by default might be easy, but it could also make it less discoverable for users who need it. Here are some alternative ideas: WhatsApp Image 2024-04-06 at 23 06 08_6df58e96

Collapsible chart: Allow users to hide/expand the chart. Toggle button: Provide a quick way to show/hide the chart. Settings menu: Give users control over what elements they see. I have experience with web development (HTML, CSS, JS) and might be able to help implement these if needed.

mhsh312 commented 3 months ago

Hi @alisman are you working on this?

mhsh312 commented 3 months ago

Hi @inodb I have a question regarding this issue. The way I understand this is there should be a list of sorts that records which charts to show by default. I just need to remove Timeline Events Availability(or CLINICAL_EVENT_TYPE_COUNTS_TABLE) from that list. The problem is I am having trouble finding that list. I did find where the priorities are mentioned in StudyViewConfig.ts:

https://github.com/cBioPortal/cbioportal-frontend/blob/ff03554180e62481bdc240acd5521601b07d124a/src/pages/studyView/StudyViewConfig.ts#L141-L143

But how does the system decide which charts to show by default?