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

Uncaught TypeError: Cannot read property 'baseVal' of undefined in SvgOverlay #7221

Closed
5 tasks done
filipjakov opened this issue Jul 24, 2020 · 2 comments
Closed
5 tasks done

Comments

@filipjakov
Copy link

filipjakov commented Jul 24, 2020

  • I've looked at the documentation to make sure the behaviour is documented and expected

  • I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)

  • I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

  • use svgOverlay with the first parameter set as an URL to the svg

Expected behavior

  • svg should load via url

Current behavior

  • error: "Uncaught TypeError: Cannot read property 'baseVal' of undefined"
  • ImageOverlay has the ability to provide the url to the image. Since SVGOverlay extends ImageOverlay, i expect the same can be done

Environment

  • Leaflet version: ^1.6.0,
  • Browser (with version): any
  • OS/Platform (with version): any

Minimal example reproducing the issue

L.svgOverlay(url, bounds).addTo(map)

  • this example is as simple as possible
  • this example does not rely on any third party code
@IvanSanchez
Copy link
Member

use svgOverlay with the first parameter set as an URL to the svg

Show some Javascript code used to do that, please. It's not easy to debug something when there's no straightforward way to reproduce (i.e. load an example webpage which triggers the bug)

@filipjakov
Copy link
Author

#6517 (comment)

Tried to improve it by loading an element with XHR, but i don't have much time now to wrap it all around.
I think it's better to keep it simple by now.

That solves it

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

No branches or pull requests

2 participants