You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is super minor, but the header looks very squat
compared to OncoPrint
or study view
Can you add a bit more space above/below 'Clinical' and 'Mutations'?
In the Clinical Data tab in Study View, there's a tooltip with the description of the clinical data element:
Could we do something similar here?
And just to put all the feedback in one place, here are @jjgao's comments from slack:
Do we need a “reset to default” button? (right now we can refresh the page to reset)
It would be good to remember the columns when switching genes, but that could be a separate effort as discussed on Monday.
I agree with both of those suggestions, and will just add that 3 becomes essential once we do 4. I'd be ok with 3 as a follow-up task if that makes more sense, but we should definitely do both.
The text was updated successfully, but these errors were encountered:
FixescBioPortal/cbioportal#8521, 3.1 Store user settings and 3.2 Add "reset to default" button based on app configuration in cBioPortal/cbioportal#8711, points 3 & 4 in cBioPortal/cbioportal#8716.
Changes:
Remembers active column selection when switching genes in Results View Mutation Table
Adds "Reset Columns" button which resets the visible columns back to their initial configuration (i.e., according to the visible attribute in the column definition). Note, this button is only shown when the current column selection is different from the default configuration. (Feedback on its design is appreciated.)
Fixes bug in cbioportal-frontend-commons where user-selection was prevented when passing columnVisibility prop to LazyMobXTable
This is awesome @Rajat-Sirohi !
I have just a few comments:
compared to OncoPrint
or study view
Can you add a bit more space above/below 'Clinical' and 'Mutations'?
Could we do something similar here?
And just to put all the feedback in one place, here are @jjgao's comments from slack:
Do we need a “reset to default” button? (right now we can refresh the page to reset)
It would be good to remember the columns when switching genes, but that could be a separate effort as discussed on Monday.
I agree with both of those suggestions, and will just add that 3 becomes essential once we do 4. I'd be ok with 3 as a follow-up task if that makes more sense, but we should definitely do both.
The text was updated successfully, but these errors were encountered: