mapshaper 0.7.52 → 0.7.53

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mapshaper",
-  "version": "0.7.52",
+  "version": "0.7.53",
   "description": "A tool for editing geospatial data for mapping and GIS.",
   "keywords": [
     "shapefile",
@@ -32,7 +32,6 @@
     "postpublish": "./release_web_ui; ./release_github_version",
     "dev": "rollup --config --watch",
     "test:browser": "playwright test --fully-parallel --workers=6",
-    "install:browsers": "./install-playwright-browsers",
     "benchmark:raster": "MAPSHAPER_RUN_BENCHMARKS=1 playwright test browser-tests/raster-resampling-benchmark.spec.mjs --project=chromium --workers=1"
   },
   "main": "./mapshaper.js",

package/www/mapshaper-gui.js

@@ -12786,8 +12786,10 @@
     });
 
     gui.on('snip', function(e) {
-      var target = e.target;
-      var result = e.result;
+      // read from e.data: the event dispatcher sets e.target to itself, so a
+      // 'target' property in the payload never reaches the handler
+      var target = e.data.target;
+      var result = e.data.result;
       var undo = function() {
         undoSnip(target, result);
         gui.model.updated({arc_count: true});
@@ -13306,9 +13308,9 @@
       polygon_style: 'style polygons',
       labels: 'position labels',
       edit_points: 'add/drag points',
-      edit_lines: 'draw/edit polylines',
+      edit_lines: 'draw/edit lines',
       edit_polygons: 'draw/edit polygons',
-      snip_lines: 'snip polylines',
+      snip_lines: 'snip lines',
       vertices: 'edit vertices',
       selection: 'selection tool',
       ruler: 'measure distance',
@@ -23221,9 +23223,11 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     }
   }
 
-  function getDistanceDisplay(distanceMeters, unitHint) {
-    var unit = unitHint || (distanceMeters >= 1000 ? 'km' : 'm');
-    var value = unit == 'km' ? distanceMeters / 1000 : distanceMeters;
+  function getDistanceDisplay(distanceMeters) {
+    var unit = getDistanceUnit(distanceMeters);
+    var value = unit == 'km' && distanceMeters / 1000 ||
+      unit == 'm' && distanceMeters || unit == 'cm' && distanceMeters * 100 ||
+      unit == 'mm' && distanceMeters * 1000 || 0;
     return {
       value,
       unit,
@@ -23237,7 +23241,9 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
   }
 
   function getDistanceUnit(distanceMeters) {
-    return distanceMeters >= 1000 ? 'km' : 'm';
+    return distanceMeters >= 1000 && 'km' ||
+      distanceMeters >= 0.1 && 'm' || distanceMeters >= 0.01 && 'cm' ||
+      distanceMeters > 0 && 'mm' || 'm';
   }
 
   function pointIsInLngLatRange(p) {
@@ -23281,7 +23287,8 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     if (value >= 100) return 0;
     if (value >= 10) return 1;
     if (value >= 1) return 2;
-    return 3;
+    if (value > 0) return 3;
+    return 1;
   }
 
   function lngLatToUnitVector(lng, lat) {
@@ -23759,11 +23766,10 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var projectedMeters = isLatLng ? null : getProjectedPathDistanceMeters(path);
       var geographicMeters = getGeographicPathDistanceMeters(path);
       var displayedMeters = geographicMeters !== null ? geographicMeters : projectedMeters;
-      var unit = getDistanceUnit(displayedMeters || 0);
       return {
         projectedMeters,
         geographicMeters,
-        distanceLabel: displayedMeters === null ? null : getDistanceDisplay(displayedMeters, unit).label
+        distanceLabel: displayedMeters === null ? null : getDistanceDisplay(displayedMeters).label
       };
     }
 
@@ -25693,6 +25699,23 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
         appendNewPointForTest(target.layer, p);
         gui.dispatchEvent('point_add', {target: target.layer, p: p});
         gui.dispatchEvent('map-needs-refresh');
+      },
+      // Perform the same edit as the snip tool, without simulating the pointer
+      // events that choose the cut locations.
+      snipActiveLayerPath: function(fid, partId, cuts) {
+        var target = gui.model.getActiveLayer();
+        var lyr = target && target.layer;
+        var result;
+        if (!lyr || lyr.geometry_type != 'polyline') {
+          throw new Error('Active layer is not a polyline layer');
+        }
+        result = snipPath(lyr, fid, partId, cuts);
+        if (!result) {
+          throw new Error('Snip was rejected');
+        }
+        gui.dispatchEvent('snip', {target: lyr, result: result});
+        gui.model.updated({arc_count: true});
+        return result;
       }
     };
   }