-
Notifications
You must be signed in to change notification settings - Fork 310
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
Out-of-bounds nodes can break FlowChart #256
Comments
@jakubfaliszewski seems like this is no longer maintained any libray out there you can suggest? |
Has any lib replaced this? |
Yeah, react flow |
Your have link repository ? |
Nop
…On Sun, Mar 3, 2024 at 9:39 PM Willian Kevenis ***@***.***> wrote:
Your have link repository ?
—
Reply to this email directly, view it on GitHub
<#256 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOYCTR4PRLDPZ5BPQV3XN23YWNU4XAVCNFSM6AAAAAARH4HMYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZVGI2TMMBZGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @MrBlenny and everyone!
While working with https://github.com/MrBlenny/react-flow-chart I encountered a problem with disappearing links between nodes. It occurs when nodes are far away from each other or when zoom factor is high enough so nodes are out of bounds. This causes
children
of InnerCanvas to not be rendered.It seems that
nodesInView
that contains of nodes that are only in view was designed to not render unnecessary nodes due to the performance issues, but it creates some problems. When nodes are removed links between them cannot exist which leads to hiding links that should be visible and to rendering errors.Problem
Links that should be visible can be hidden because nodes are removed when they are positioned out of bounds.
Demo - here you can see that links disappears after zooming in and everything goes wild after zooming out:
Solution
I propose two solutions here:
nodesInView
will be justObject.keys(nodes)
visibility: hidden
will do the job)The text was updated successfully, but these errors were encountered: