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

Moving and Zooming of the Graph View throws Errors #49

Closed
phibr0 opened this issue Feb 13, 2021 · 2 comments
Closed

Moving and Zooming of the Graph View throws Errors #49

phibr0 opened this issue Feb 13, 2021 · 2 comments

Comments

@phibr0
Copy link

phibr0 commented Feb 13, 2021

If you zoom in or out of the Graph View or just move it in any direction following Error gets thrown in the Console:

your-first-note:402 Uncaught TypeError: Cannot set property 'innerHTML' of null
    at SVGSVGElement.resize (your-first-note:402)
    at H.apply (d3.min.js:2)
    at kt (d3.min.js:2)
    at w.emit (d3.min.js:2)
    at w.zoom (d3.min.js:2)
    at d3.min.js:2
    at d3.min.js:2
resize @ your-first-note:402
apply @ d3.min.js:2
kt @ d3.min.js:2
emit @ d3.min.js:2
zoom @ d3.min.js:2
(anonymous) @ d3.min.js:2
(anonymous) @ d3.min.js:2

image

@Am3ra
Copy link
Contributor

Am3ra commented Mar 30, 2021

I've noticed this as well, but couldn't figure it out immidiately either.

On my site, I also noticed that the graph overflows to the right.

@maximevaillancourt , would you mind if we (or even me specifically) update the graph to what is currently supported by the Foam VSCode extension? Personally, I believe it looks better.

Alternatively, we could simply fix this.

@maximevaillancourt
Copy link
Owner

Thanks for the bug report! Fixed in e6e2ea2.

@maximevaillancourt , would you mind if we (or even me specifically) update the graph to what is currently supported by the Foam VSCode extension? Personally, I believe it looks better.

Pull requests are welcome! 🙂

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