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

Padding graphs broken? #80

Open
Mmmkayness opened this issue Jan 27, 2014 · 0 comments
Open

Padding graphs broken? #80

Mmmkayness opened this issue Jan 27, 2014 · 0 comments

Comments

@Mmmkayness
Copy link

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!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant