Skip to content
This repository was archived by the owner on Dec 27, 2022. It is now read-only.

Padding graphs broken? #80

@Mmmkayness

Description

@Mmmkayness

Hi,

I'm currently observing an issue when placing a padding ("target": false) element between graphs in Tasseo. When I add these padding graphs, the dashboard fails to load completely for me. From my observations this looks to be because the buildContainers process accounts for the missing index and builds an array smaller than the number of divs on the page (var j = i - falseTargets). But the createsGraph process just iterates over all and skips the ones with a false target. At least, I think that's what the cause was. Either way, my somewhat hacky temp fix locally was to remove the "j" variable and use the "i" index from the for loop for each element in the array.

There might be a more elegant solution to this!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions