replikation / poreCov

SARS-CoV-2 workflow for nanopore sequence data
https://case-group.github.io/
GNU General Public License v3.0
39 stars 16 forks source link

improve table style #207

Closed replikation closed 2 years ago

replikation commented 2 years ago

with all the features and columns the table stretches too much downwards (e.g. it's one line for each mutation currently). makes it a slight scrolling nightmare for > 40 samples

Proposals:

replikation commented 2 years ago
hoelzer commented 2 years ago
  • and yes i know we have tons of omicron spike mutations in the tables currently ;)

:) But yes, fully agree. It's nice to have all this information but the table gets super long.

RaverJay commented 2 years ago

imho you just need wider screens ;)

reducing font size is most feasible I guess?

replikation commented 2 years ago

but whats the maximum scale? its not automatically scaling on the x axis e.g. here

a good way would be auto-scaling and forced 10% space on left and right

hoelzer commented 2 years ago

Can we make the column with the mutations extendable? The table gets so long because we have especially with Omicron now so many mutations. Not sure if this is somehow possible with the python backend of the HTML report ;)

replikation commented 2 years ago

we might want to add a column "hide" feature like here: https://datatables.net/examples/api/show_hide.html

MarieLataretu commented 2 years ago

Hmm, maybe this is also a helpful element:

click here to expand Hello World!

(<details><summary>click</summary>content</details>)

replikation commented 2 years ago

yeah. keep an overview but "hide" / "unhide" mutations

RaverJay commented 2 years ago

Hmm, maybe this is also a helpful element: click here to expand

(<details><summary>click</summary>content</details>)

Was thinking about this one too

hoelzer commented 2 years ago

Can we then add a single button to hide/unhide all columns? Sometimes it's neat to scroll over the HTML even if it's long (e.g. so I discovered the BA.1 weirdos w/th the 5-vs-6-nt deletion).

Also would be nice if one can still search via crtl-F e.g. "E484A" but I think this is also possible w/ the hiddn content

RaverJay commented 2 years ago

image

how about this?

replikation commented 2 years ago

uh nice. really like it (the barcode 16 style, because table has more columns when using poreCov from reads)

would extend it to the deletions, insertions, and frameshifts too. So it's directly "future" proof.