search

jieter / Leaflet-semicircle

Extend Leaflet's circle class to display semicircles.
http://jieter.github.io/Leaflet-semicircle/examples/semicircle.html
MIT License
121 stars 52 forks source link

Make Semicircle.js compatible with Leaflet 1.4.0 #31

Closed austil closed 5 years ago

austil commented 5 years ago

Hi, I've been using your handy module at work and came across the following error :

TypeError: this._drawnLayers is undefined

It appear that _drawnLayers has been removed from Leaflet in 1.4.0 (Changelog, Pull request) because it has become useless following a canvas refactor from nov 2016.

Removing the single and only affectation to this variable in semicircle.js was enough to make it work again.

Regards.

Augustin :fr:

jieter commented 5 years ago

Thanks!

austil commented 5 years ago

Ty, that was fast ! Do you plan on publishing a "patch" version to npm anytime soon ?

jieter commented 5 years ago

published version 2.0.4