ucscXena / ucsc-xena-client

Functional genomics browser
Apache License 2.0
57 stars 42 forks source link

Update Legend for Mutation Columns #259

Closed NoopDog closed 5 years ago

NoopDog commented 6 years ago
  1. Group types of the same color.
  2. Modify for boxes and text legend.
maryjgoldman commented 6 years ago

Before: http://dev.xenabrowser.net/heatmap/?bookmark=389029419ec7fd361d5177fbb4dfe523 mutations

maryjgoldman commented 6 years ago

After: https://docs.google.com/presentation/d/1eJ-hwlYI3EgB34iSM8fdp2MFeEtSLoS0XPnJ-d-lzc4/edit#slide=id.g3541a2a88e_0_173 --> slide 2 mutations2

maryjgoldman commented 6 years ago

Note, only show each of the mutation types in the legend if they are present in the plot. Each plot will vary by gene

jingchunzhu commented 6 years ago

Mary, do we have to "only show each of the mutation types in the legend if they are present in the plot. Each plot will vary by gene". How about just showing the same legend for every mutation column.

maryjgoldman commented 6 years ago

That could work. I just figured we were already only showing what is there so the code must already be in place. So hook up the new legend to the same code.

jingchunzhu commented 6 years ago

I vote for the easier one , same legend across all mutation columns.

maryjgoldman commented 6 years ago

At first I wanted the other one but after thinking about it more, you're right. It's easier, the old version doesn't add much, and if they want to use the legend in a real figure they'll probably need to edit it anyways so editing out colors that aren't there isn't too much more work.

And on the off chance their first plot has all blue mutations or something like that, they will realize that we don't just color all mutations blue. That the color means something and if they open another plot they'll see that.

jingchunzhu commented 5 years ago

https://github.com/ucscXena/ucsc-xena-client/commit/34babd88af12b8f42badbcaadbe1447129716e58

maryjgoldman commented 5 years ago

Looking good! It seems to me like the words 'Variant Impact' and the 'no data' bar should be inside the card outline. Is this possible?

jingchunzhu commented 5 years ago

on beta