Fix resizing circle when using L.CRS.Simple #194
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When resizing circle on a map with the option: crs: L.CRS.Simple the circle radius is invalid
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 useEarth.distance(this, toLatLng(other));
https://github.com/Leaflet/Leaflet/blob/37d2fd15ad6518c254fae3e033177e96c48b5012/src/geo/LatLng.js#L75