Versions of Angular Tree Component, Angular, Node, affected browser(s) and operating system(s):
10.0.4
Other information:
In tree-node.model.ts the method _calculateExpandedNodes uses Object.assign for every node that should be expanded. Doing it on every call is quite expensive. My suggestion would be to use extendObservable as the mobx documentation recommends. Then when all expandedNodeIds are known we can apply the expansive Object.assign once to update the reference so that the mobx can detect the change and react to it accordingly.
Minimal reproduction of the bug/regression with instructions:
Stackblitz Reproduction
If you only go 3 levels deep the tree can load.
Versions of Angular Tree Component, Angular, Node, affected browser(s) and operating system(s):
10.0.4
Other information:
In tree-node.model.ts the method _calculateExpandedNodes uses Object.assign for every node that should be expanded. Doing it on every call is quite expensive. My suggestion would be to use
extendObservable
as the mobx documentation recommends. Then when all expandedNodeIds are known we can apply the expansive Object.assign once to update the reference so that the mobx can detect the change and react to it accordingly.I would be willing to submit a PR to fix this issue