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

[RFC 76] Redesign oncoprint annotation menu #4709

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

BasLee
Copy link

@BasLee BasLee commented Aug 24, 2023

Redesign the oncoprint annotation menu as described in RFC 76.

Changes

  • Remove the top 'Putative Drivers vs VUS' toggle
  • Move custom driver annotations to its own section
  • Reword filters to clarify meaning
  • Add info tooltips to add some additional information to the checkboxes
  • For automatic disabling of the 'exclude VUS' filter, the custom driver tiers now also need to be disabled

image
Screenshot of restyled menu

@BasLee BasLee changed the title [RFC 76] Restyle oncoprint annotation menu [RFC 76] Redesign oncoprint annotation menu Aug 24, 2023
@inodb
Copy link
Member

inodb commented Aug 24, 2023

@BasLee could you move the custom driver tiers to be indented below the custom driver annotations?

@SRodenburg
Copy link

SRodenburg commented Aug 25, 2023

@BasLee could you move the custom driver tiers to be indented below the custom driver annotations?

One of the points of this redesign was that many users did not understand that the Driver tiers mechanism is independent from the binary Custom Driver annotations mechanism.
Thus I would oppose making the indent :)

@BasLee
Copy link
Author

BasLee commented Aug 28, 2023

✔️ Align bottom 'only profiled' checkbox in 0023fa3

@BasLee BasLee force-pushed the restyle-oncoprint-annotation-menu branch 2 times, most recently from 0023fa3 to 3665e01 Compare August 30, 2023 17:09
@BasLee BasLee force-pushed the restyle-oncoprint-annotation-menu branch from b05bf02 to 0f5d99f Compare August 31, 2023 10:30
oncoprint.custom_driver_annotation.binary.menu_label=Custom driver annotation
oncoprint.custom_driver_annotation.tiers.menu_label=Custom driver tiers
oncoprint.custom_driver_annotation.binary.menu_label=Custom driver annotations
oncoprint.custom_driver_annotation.tiers.menu_label=Driver tiers

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
oncoprint.custom_driver_annotation.tiers.menu_label=Driver tiers
oncoprint.custom_driver_annotation.tiers.menu_label=Custom driver tiers

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pvannierop after our discussion with @inodb
Please follow up now that Bas is gone.

@pvannierop pvannierop force-pushed the restyle-oncoprint-annotation-menu branch 2 times, most recently from 05c155b to b8995a5 Compare September 14, 2023 14:42
@pvannierop pvannierop force-pushed the restyle-oncoprint-annotation-menu branch from b8995a5 to d4ad3c7 Compare September 19, 2023 13:32
@alisman alisman merged commit 9122026 into cBioPortal:master Sep 25, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants