Fix resizing circle when using L.CRS.Simple - Githubissues

Leaflet / Leaflet.Editable

Make geometries editable in Leaflet.

http://leaflet.github.io/Leaflet.Editable/doc/api.html

559 stars 197 forks source link

Fix resizing circle when using L.CRS.Simple #194

Closed Socolin closed 1 month ago

Socolin commented 5 years ago

When resizing circle on a map with the option: crs: L.CRS.Simple the circle radius is invalid

Repro link: https://codepen.io/socolin/pen/qBBrzMO
Fixed version: https://codepen.io/socolin/pen/xxxqoyb

The problem seems to be from this line: https://github.com/Leaflet/Leaflet.Editable/blob/master/src/Leaflet.Editable.js#L1712 When it compute the new radius size, it ignore the map crs option and use Earth.distance(this, toLatLng(other));

https://github.com/Leaflet/Leaflet/blob/37d2fd15ad6518c254fae3e033177e96c48b5012/src/geo/LatLng.js#L75

faizalami commented 2 years ago

I need this

yohanboniface commented 1 month ago

:+1: and sorry for that long delay!