mapshaper 0.6.85 → 0.6.87

package/mapshaper.js

@@ -4236,12 +4236,16 @@
   function pathIsRectangle(ids, arcs) {
     var bbox = arcs.getSimpleShapeBounds(ids).toArray();
     var iter = arcs.getShapeIter(ids);
+    var count = 0;
     while (iter.hasNext()) {
       if (iter.x != bbox[0] && iter.x != bbox[2] ||
           iter.y != bbox[1] && iter.y != bbox[3]) {
         return false;
       }
+      count++;
     }
+    if (count < 5) return false;
+    if (bbox[2] > bbox[0] === false || bbox[3] > bbox[1] === false) return false;
     return true;
   }
 
@@ -45574,7 +45578,7 @@ ${svg}
     });
   }
 
-  var version = "0.6.85";
+  var version = "0.6.87";
 
   // Parse command line args into commands and run them
   // Function takes an optional Node-style callback. A Promise is returned if no callback is given.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mapshaper",
-  "version": "0.6.85",
+  "version": "0.6.87",
   "description": "A tool for editing vector datasets for mapping and GIS.",
   "keywords": [
     "shapefile",

package/www/mapshaper-gui.js

@@ -2428,7 +2428,7 @@
     function error() {
       var msg = GUI.formatMessageArgs(arguments);
       console.error(msg);
-      gui.alert('An unkown error occured');
+      gui.alert('An unknown error occured');
       throw new Error(msg);
     }
 
@@ -5811,11 +5811,14 @@
     var altDown = false;
     var spaceDown = false;
 
-    function updateControlKeys(e) {
+    function updateControlKeys(e, evtName) {
       shiftDown = e.shiftKey;
       ctrlDown = e.ctrlKey;
       metaDown = e.metaKey;
       altDown = e.altKey;
+      if (e.keyCode == 32) {
+        spaceDown = evtName == 'keydown';
+      }
     }
 
     function mouseIsPressed() {
@@ -5823,17 +5826,14 @@
     }
 
     document.addEventListener('keyup', function(e) {
-      if (!GUI.isActiveInstance(gui) || e.repeat) return;
-      // this can fail to fire if keyup occurs over a context menu
-      if (e.keyCode == 32) spaceDown = false;
-      updateControlKeys(e);
+      if (!GUI.isActiveInstance(gui) || e.repeat && e.keyCode == 32) return;
+      updateControlKeys(e, 'keyup');
       self.dispatchEvent('keyup', getEventData(e));
     });
 
     document.addEventListener('keydown', function(e) {
-      if (!GUI.isActiveInstance(gui) || e.repeat) return;
-      if (e.keyCode == 32) spaceDown = true;
-      updateControlKeys(e);
+      if (!GUI.isActiveInstance(gui) || e.repeat && e.keyCode == 32) return;
+      updateControlKeys(e, 'keyup');
       self.dispatchEvent('keydown', getEventData(e));
     });
 
@@ -5886,7 +5886,7 @@
       standard: ['info', 'selection', 'box'],
       empty: ['edit_points', 'edit_lines', 'edit_polygons', 'box'],
       polygons: ['info', 'selection', 'box', 'edit_polygons'],
-      rectangles: ['info', 'selection', 'box', 'rectangles'],
+      rectangles: ['info', 'selection', 'box', 'rectangles', 'edit_polygons'],
       lines: ['info', 'selection', 'box' , 'edit_lines'],
       table: ['info', 'selection'],
       labels: ['info', 'selection', 'box', 'labels', 'edit_points'],
@@ -8817,7 +8817,9 @@
       }
     });
     element.addEventListener('contextmenu', function(e) {
-      _self.dispatchEvent('contextmenu', procMouseEvent(e));
+      if (!e.ctrlKey) {
+        _self.dispatchEvent('contextmenu', procMouseEvent(e));
+      }
     });
 
     this.enable = function() {
@@ -10385,6 +10387,7 @@
 
     gui.on('redo_path_extend', function(e) {
       var target = hit.getHitTarget();
+
       if (pathDrawing() && prevHoverEvent) {
         updatePathEndpoint(e.p);
         appendVertex$1(target, pixToDataCoords(prevHoverEvent.x, prevHoverEvent.y));
@@ -10404,12 +10407,12 @@
       } else {
         deleteLastVertex(target);
       }
-      if (getLastArcLength(target) < 2) {
-        gui.undo.undo(); // remove the path
-      }
       if (e.shapes) {
         replaceDrawnShapes(e.shapes);
       }
+      if (getLastArcLength(target) < 2) {
+        gui.undo.undo(); // remove the path
+      }
     });
 
     function turnOn() {
@@ -10649,8 +10652,6 @@
       var p = pixToDataCoords(e.x, e.y);
       if (pathDrawing()) {
         extendCurrentPath(hoverVertexInfo?.point || p);
-      } else if (gui.keyboard.shiftIsPressed()) {
-        deleteActiveVertex(e);
       } else if (hoverVertexInfo?.type == 'interpolated') {
         // don't start new path if hovering along a segment -- this is
         // likely to be an attempt to add a new vertex, not start a new path
@@ -10752,7 +10753,8 @@
       if (!pathDrawing()) return;
       var target = hit.getHitTarget();
       if (getLastArcLength(target) <= 2) { // includes hover point
-        deleteLastPath(target);
+        // deleteLastPath(target);
+        gui.undo.undo(); // assume previous undo event was path_add
       } else {
         deleteLastVertex(target);
       }

package/www/mapshaper.js

@@ -4236,12 +4236,16 @@
   function pathIsRectangle(ids, arcs) {
     var bbox = arcs.getSimpleShapeBounds(ids).toArray();
     var iter = arcs.getShapeIter(ids);
+    var count = 0;
     while (iter.hasNext()) {
       if (iter.x != bbox[0] && iter.x != bbox[2] ||
           iter.y != bbox[1] && iter.y != bbox[3]) {
         return false;
       }
+      count++;
     }
+    if (count < 5) return false;
+    if (bbox[2] > bbox[0] === false || bbox[3] > bbox[1] === false) return false;
     return true;
   }
 
@@ -45574,7 +45578,7 @@ ${svg}
     });
   }
 
-  var version = "0.6.85";
+  var version = "0.6.87";
 
   // Parse command line args into commands and run them
   // Function takes an optional Node-style callback. A Promise is returned if no callback is given.

package/www/page.css

@@ -1193,8 +1193,8 @@ div.basemap-style-btn.active {
 }
 
 .basemap-style-btn.basemap-overlay-btn {
-	width: 35px;
-	height: 35px;
+	width: 40px;
+	height: 40px;
 	margin: 3px;
 	overflow: hidden;
 	display: inline-block;