nih-cfde / dashboard

Files and documentation related to the CFDE project dashboards
Other
0 stars 0 forks source link

Epic 1 testing findings #16

Closed RLC-DCPPC closed 3 years ago

RLC-DCPPC commented 3 years ago

From @carlkesselman:

jonathancrabtree commented 3 years ago

The last item above is truncated, but the issue is that the lines between the labels and the corresponding donut slices can cross one another (usually in cases where there are multiple adjacent small slices):

donut-overlaps

There are also two issues with the mouse hover popups. The first popup issue is that the popup box is a fixed size and the label doesn't always fit:

popup-issue-1

The second popup issue is that the relative position of the popup div doesn't change when you're near the top and/or left edge of the graph and so part of the popup can be missing. In the following example (which actually illustrates both issues) the label should say "16s ribosomal..." not just "ribosomal":

popup-issue-2

jonathancrabtree commented 3 years ago

Committed some initial steps to address the label overflows:

Screen Shot 2020-12-09 at 3 43 07 PM

Screen Shot 2020-12-09 at 3 23 17 PM

jonathancrabtree commented 3 years ago

As the preceding screenshots show, there's quite a bit of variability in the amount of vertical whitespace between the graph and the form controls below. This should be standardized and then the labels allowed to extend as far as possible into that space.

victor73 commented 3 years ago

Made some commits to address issues with the tooltips going out-of-bounds in some cases and dynamically truncating/ellipsizing long category names to fit inside the tooltip borders.

jonathancrabtree commented 3 years ago

Switched donut charts to use a color legend instead of attaching labels. Also fixed problems with the donut chart color assignments and tooltip label overruns:

Screen Shot 2020-12-11 at 12 18 56 AM

jonathancrabtree commented 3 years ago

One more addition to the donut chart, which is to enable mouse hover on the legend as well as the chart itself:

Screen Shot 2020-12-15 at 12 20 48 PM

We can revisit the use of direct labels with connecting lines if this approach proves inadequate.