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

Potentially unnecessary rendering #132

Open
zamplyy opened this issue Sep 21, 2023 · 2 comments
Open

Potentially unnecessary rendering #132

zamplyy opened this issue Sep 21, 2023 · 2 comments

Comments

@zamplyy
Copy link

zamplyy commented Sep 21, 2023

Hi!
Thanks for a great library. Really shows you've put in alot of work!

I am trying to get more performance for my app when using force-layout, and trying a few ways of doing that.
I found that we're always rendering the VEdgeBackgrounds and wondering if we can somehow conditionally render them instead.
And what do is the functionality provided by them, i couldn't find a example.

@dash14
Copy link
Owner

dash14 commented Sep 30, 2023

Hi @zamplyy,
Thank you so much for your consideration of performance improvement!

The purpose of VEdgeBackgrounds is to address the fact that when edges are drawn as dashed lines, browser mouse events only respond to the drawn part and not to the non-drawn part. VEdgeBackgrounds are transparent solid lines that are responsible for catching mouse events.
Therefore, VEdgeBackgrounds need not be present on the DOM, at least if the line style of the edge is solid....

@kogratte
Copy link

kogratte commented Dec 7, 2023

Clear and concise answer. A pleasure to see such support :)

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

3 participants