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

Update Edit.Circle.js #1053

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

poet-of-the-fall
Copy link

fixed circle edit bug

fixed circle edit bug
This was referenced Nov 11, 2022
@ferily7
Copy link

ferily7 commented Apr 19, 2023

Is this PR going to be merged in? It looks like it was approved -- I (and other people) am having issues with trying to edit the size of the circle and I think this should fix the bug..

@poet-of-the-fall
Copy link
Author

Sorry, did not check on this one lately, but can’t merge as review is from a person with write access is missing. I switched to geoman-io lately…

@mdanko990
Copy link

could it be fixed soon? it still doesn't work

@AKopytenko
Copy link

Tell me, is there any progress on this task?
Since 2019, editing a circle on the map does not work. It is very strange.
Even stranger is that leaflet-draw-sector, which, if I’m not confusing anything, is based on Leaflet.circle, works without problems.

@vstoykov
Copy link

vstoykov commented Aug 2, 2024

Actually the same fix plus one extra is present in #977

@DanielKucal
Copy link

As the PR doesn't seem to be merged soon, I'll share simple workaround for this problem. The solution is basically the same as for #1005:
TypeScript:

(window as any).radius = undefined;

JavaScript:

window.radius = undefined;

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

Successfully merging this pull request may close these issues.

9 participants