sfu-dhil / wilde

eXist/XQuery app for detecting copying in a collection of XHTML documents.
GNU General Public License v3.0
2 stars 9 forks source link

Report graph is broken #47

Closed joeytakeda closed 4 years ago

joeytakeda commented 4 years ago

Describe the bug The graph on the reports page doesn't show the lines, only the legend.

To Reproduce Steps to reproduce the behavior:

  1. Go to the reports page
  2. Wait for the graph to load
  3. The lines never load

Expected behavior The graph should show the lines.

Screenshots (Firefox on left; Chrome on right)

Screen Shot 2020-04-29 at 11 51 03 AM

The error from the console is:

TypeError: d.values[lastHoveredBarIndex] is undefined d3.stacked.bar.js:458:38
    tooltipContent https://dhil.lib.sfu.ca/wilde/resources/js/d3.stacked.bar.js:458
    each https://dhil.lib.sfu.ca/wilde/resources/lib/d3/d3.min.js:3
    Y https://dhil.lib.sfu.ca/wilde/resources/lib/d3/d3.min.js:1
    each https://dhil.lib.sfu.ca/wilde/resources/lib/d3/d3.min.js:3
    tooltipContent https://dhil.lib.sfu.ca/wilde/resources/js/d3.stacked.bar.js:457
    showTooltip https://dhil.lib.sfu.ca/wilde/resources/js/d3.stacked.bar.js:444
    $/< https://dhil.lib.sfu.ca/wilde/resources/lib/d3/d3.min.js:1

Desktop (please complete the following information): FF 74; Chrome 80.0.3987.163 on Mojave 10.14.6.

Additional context

If page load is a concern, I think all of the Javascript on the page should be removed, including the filtering features on the page to make the page as simple as possible. It will still take time to render and load, but it should cut it down.

ubermichael commented 4 years ago

This page needs to be completely removed. The other browsing options are in place to take its place.