-
Notifications
You must be signed in to change notification settings - Fork 223
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 with release 2.0.0 in combination with MarkerCluster #312
Comments
I've got the same error and I'm not using the marker clusterer, but I am using L.shapeMarker. So maybe it's paths or divIcons it has issues with?
|
Same here, I first thought the problem was with Markercluster or leaflet, but it starts when geocoder is initiated.. I made a test also: Thank you for mentioning the old version does not have the problem, will use that one for now.. |
Eugh, I just spent nearly 2 days trying to figure this. Such a N00b at JS but eventually...got here. Will make sure I version tie from here on in! |
I've just added the version to all my portals, and it's working again, so it's a lesson for me too. It didn't even cross my mind that a new version would be deployed at the same location. |
Sorry for breaking the event handling with the 2.0.0 release. Bringing together leaflet mixins and TypeScript wasn't easy and my approach for the 2.0.0 release turned out erroneous. A few minutes ago, I've released 2.0.1 which should address this issue. |
Thanks for the fast fix! |
thanks, that seems to have worked. 👍 |
I get an
Uncaught TypeError
in the browser console when using the new version 2.0.0.In Chrome the error message is:
In Firefox the error message is:
This looks like there is a problem in MarkerCluster, but when I use the old version of leaflet-control-geocoder this problem does not occur.
I've created a rudimentary example to comprehend the issue. The only difference is the version number of leaflet-control-geocoder.
v1.13.0.html - old version without error
v2.0.0.html - new version with error
The text was updated successfully, but these errors were encountered: