How to make layers with {pmIgnore: true} snappable?

[aitanserikbayev]

I used the older version of leaflet-geoman in past and pmIgnore layers were snappable by default. Now they aren't. Is it possible to do them snappable but not editable and removable?

[Falke-Design]

Currently it is not really easy to solve, I think we should make this easier.

You have to add the layers they should snap manually on every drawstart or enabling:

var rect = L.rectangle(map.getBounds().pad(-0.3),{pmIgnore: true}).addTo(map);
var rect2 = L.rectangle(map.getBounds().pad(-0.6),{pmIgnore: true, color: 'red'}).addTo(map);


map.on('pm:drawstart',(e)=>{
  console.log(e);
  var pmDraw = map.pm.Draw[e.shape];
  // pmDraw._otherSnapLayers is cleared every time drawing is started 
  pmDraw._otherSnapLayers.push(rect); // Add all layers they should be snappable
  pmDraw._otherSnapLayers.push(rect2); 
  pmDraw._createSnapList();
});

map.on('pm:create',(e)=>{
  e.layer.on('pm:enable',(x)=>{
    console.log(x);
    var pm = x.layer.pm;
    // pm._otherSnapLayers is NEVER cleared, so we need to check if the layers to snap are already in the array
    pm._otherSnapLayers = pm._otherSnapLayers || []; // create array if it not exists
    if(pm._otherSnapLayers.indexOf(rect) === -1) { // check if layer is already in the array
      pm._otherSnapLayers.push(rect); // Add all layers they should be snappable
    }
    if(pm._otherSnapLayers.indexOf(rect2) === -1) {
      pm._otherSnapLayers.push(rect2); // Add all layers they should be snappable
    }
    console.log(pm._otherSnapLayers);
    pm._createSnapList();
  });
});

---

{pmIgnore: true, snapIgnore: false} should work

In the latest release we "fixed" that when a layer has pmIgnore: true that it is completly ignored from Geoman, so the option snapIgnore: false change nothing.

leaflet-geoman/src/js/Mixins/Snapping.js

Lines 171 to 184 in 2591108

	// find all layers that are or inherit from Polylines... and markers that are not
	// temporary markers of polygon-edits
	map.eachLayer(layer => {
	if (
	(layer instanceof L.Polyline ||
	layer instanceof L.Marker ||
	layer instanceof L.CircleMarker ||
	layer instanceof L.ImageOverlay) &&
	layer.options.snapIgnore !== true &&
	(
	(!L.PM.optIn && !layer.options.pmIgnore) || // if optIn is not set / true and pmIgnore is not set / true (default)
	(L.PM.optIn && layer.options.pmIgnore === false) // if optIn is true and pmIgnore is false
	)
	) {

[henrikmaa]

I think if pmIgnore: true it should still not snap by default, but snapIgnore: false or snappable: true, should allow.

This became an issue since #677 (before you had too { pmIgnore: true, snapIgnore:true }) to ignore snap)

[Falke-Design]

Yes I see it like you. I will talk with @codeofsumit and then implement it.