timeglider / jquery_widget

Timeglider's JS Widget component. See README for details.
193 stars 41 forks source link

Issue overrunning levels during collision detection. #19

Closed randallknutson closed 12 years ago

randallknutson commented 12 years ago

We were running into an issue where the number of blocks was exceeding the 300 levels on the tree. This would cause the timeline not to render. I've added a patch that bumps the number to 500 and adds error checking to make sure that it will at least render of overrun.

timeglider commented 12 years ago

Integrated this check on the level — using a numerical check agains the established tree_levels variable (which we can assume is the size of the levels array) instead of type checking, which is likely a bit slower.