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

addVertexOnClick + limitMarkersToCount doesn't work in Geoman-Pro #1524

Open
natedx opened this issue Oct 14, 2024 · 0 comments
Open

addVertexOnClick + limitMarkersToCount doesn't work in Geoman-Pro #1524

natedx opened this issue Oct 14, 2024 · 0 comments
Labels
bug Pro ⭐ Related to Leaflet-Geoman Pro ⭐

Comments

@natedx
Copy link

natedx commented Oct 14, 2024

When combining addVertexOnClick with limitMarkersToCount in Geoman-Pro, the pm:vertexAdded event is triggered but with a seemingly undefined event when adding a vertex directly on the line.

I tried reproducing a working geoman-pro situation in this JSFiddle :
https://jsfiddle.net/2o6sm3dz/

Swapping geoman for geoman-pro should in theory demonstrate the bug.
The other performance optimizations (limitMarkersToClick, limitMarkersToViewport, limitMarkersToZoom) do not seem affected.

@Falke-Design Falke-Design added the Pro ⭐ Related to Leaflet-Geoman Pro ⭐ label Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Pro ⭐ Related to Leaflet-Geoman Pro ⭐
Projects
None yet
Development

No branches or pull requests

4 participants
@Falke-Design @natedx and others