Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
sruthijavvaji opened this issue Nov 15, 2023 · 3 comments

Comments

@sruthijavvaji
Copy link

sruthijavvaji commented Nov 15, 2023

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Owner

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

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

No branches or pull requests

2 participants