bumbeishvili / org-chart

Highly customizable org chart. Integrations available for Angular, React, Vue
https://stackblitz.com/edit/web-platform-o5t1ha
MIT License
926 stars 329 forks source link

Application is crashing when expanding/collapsing nodes with more than 25 levels or adding new nodes after 20th level #328

Open sruthijavvaji opened 1 year ago

sruthijavvaji commented 1 year ago

Describe the bug Application crashes when expanding/collapsing nodes with more than 25 levels or adding new nodes after 20th level

To Reproduce Steps to reproduce the behavior:

  1. Create nodes with more than 25 levels and try to expand by default or try to add the new node after the 25th level using the chart.addNode method

Expected behavior It should expand/collapse or add a new node without any performance issue

bumbeishvili commented 1 year ago

Can you provide an online sample where the issue is reproduced?

I've tested the new 3.x version for 500 level tree and it was working as expected, previous version was inefficient for large level trees.

You can fork this example - https://stackblitz.com/edit/js-mfzkbs?file=index.html

sruthijavvaji commented 1 year ago

Appreciate the prompt response!

I have tested with the latest 3.x version(3.1.1) and it is working as expected but observed that Pagination is not working with the 3.1.1 version with angular15.

The issue is reproducible in the below link: https://stackblitz.com/edit/angular-ivy-x5qr97?file=package.json,src%2Fapp%2Fapp.component.ts,src%2Fapp%2Fcomponents%2Fd3-org-chart%2Fd3-org-chart.component.html,src%2Fapp%2Fcomponents%2Fd3-org-chart%2Fd3-org-chart.component.ts,src%2Fapp%2Fapp.component.html,src%2Fapp%2Fapp.module.ts,src%2Fapp%2Fcomponents%2Fphoto-gallery%2Fphoto-gallery.component.ts

bumbeishvili commented 1 year ago

Yes, I know about paging - #316 , will get to that eventually