-
Notifications
You must be signed in to change notification settings - Fork 233
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
GC - use new group label components for overlap tab and survival tab #2526
GC - use new group label components for overlap tab and survival tab #2526
Conversation
25f8577
to
df4695e
Compare
@adamabeshouse looks good overall. For UpSet plot, I am wondering if it's possible (and looks good) to just add a colored square before the group name? |
@jjgao how does this look? If you like it, then should we use this new style also for the clinical tab ticks, and for anywhere else where we show color? Or go with the color on the ordinal? |
@adamabeshouse I like the square version. Maybe don't worry about the ticks in clinical tab and I think other tabs look good. |
df4695e
to
13ee628
Compare
13ee628
to
2ca22e3
Compare
2ca22e3
to
9515ffe
Compare
9515ffe
to
6be5f8a
Compare
6be5f8a
to
f2501cf
Compare
@@ -75,7 +79,7 @@ export default class Survival extends React.Component<ISurvivalProps, {}> { | |||
name, | |||
color: blendColors(partitionGroupUids.map(uid => uidToGroup[uid].color)), | |||
value, | |||
legendText: name | |||
legendText: JSON.stringify(partitionGroupUids), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just want to confirm this is intended
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes its a bit weird but that's how I'm passing it through the system I have rigged up
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
f2501cf
to
ff56fcd
Compare
Addresses cBioPortal/cbioportal#6218