supwisdom / bootstrap-treefy

A jQuery plugin to make a table have tree behavior
MIT License
10 stars 8 forks source link

Freezes browser on large trees, too much recursion? #5

Open mikegioia opened 8 years ago

mikegioia commented 8 years ago

We ran bootstrap-treefy on a tree of 280 nodes and the plugin locked the browser for 20 seconds (or longer).

I think the problem is the recursive table scans going on to set up the table. I wanted to submit a patch but I ended up just writing the little plugin for our use case, but the whole thing can be done in 2 table scans.

Here's the working version, with only very minimal API changes (basically just option names): https://github.com/TeachBoost/jquery-treetable/blob/master/src/jquery.treetable.js

chanjarster commented 8 years ago

Thank you very much. We will working on it. @sun6ke