mapshaper 0.6.74 → 0.6.76

package/www/mapshaper-gui.js

@@ -760,10 +760,7 @@
   //
   //El.prototype.__domevents = utils.arrayToIndex("click,mousedown,mousemove,mouseup".split(','));
   El.prototype.__on = El.prototype.on;
-  El.prototype.on = function(type, func, ctx) {
-    if (ctx) {
-      error$1("[El#on()] Third argument no longer supported.");
-    }
+  El.prototype.on = function(type, func) {
     if (this.constructor == El) {
       this.el.addEventListener(type, func);
     } else {
@@ -1144,10 +1141,18 @@
       return self;
     };
 
-    self.close = function() {
-      if (el) el.remove();
+    self.close = function(action) {
+      var ms = 0;
+      var _el = el;
+      if (action == 'fade' && _el) {
+        ms = 1000;
+        _el.addClass('fade-out');
+      }
       if (_close) _close();
       el = _cancel = _close = null;
+      setTimeout(function() {
+        if (_el) _el.remove();
+      }, ms);
     };
     return self;
   }
@@ -1361,10 +1366,12 @@
     init();
 
     async function init() {
-      var enabled = await isStorageEnabled();
-      if (!enabled) return;
-
       btn = gui.buttons.addButton('#ribbon-icon').addClass('menu-btn save-btn');
+      var enabled = await isStorageEnabled();
+      if (!enabled) {
+        btn.remove();
+        return;
+      }
       menu = El('div').addClass('nav-sub-menu save-menu').appendTo(btn.node());
       await initialCleanup();
 
@@ -1436,14 +1443,6 @@
           renderMenu();
         }).text('remove');
       });
-
-      // if (snapshots.length >= 1) {
-      //   addMenuLink({
-      //     slug: 'clear',
-      //     label: 'remove all',
-      //     action: clearData
-      //   });
-      // }
     }
 
     function addMenuLink(item) {
@@ -1753,6 +1752,9 @@
       }],
       info: {}
     };
+    if (type == 'polygon' || type == 'point') {
+      dataset.arcs = new internal.ArcCollection();
+    }
     if (crsInfo) {
       internal.setDatasetCrsInfo(dataset, crsInfo);
     }
@@ -2426,7 +2428,10 @@
     }
 
     function error() {
-      stop.apply(null, utils$1.toArray(arguments));
+      var msg = GUI.formatMessageArgs(arguments);
+      console.error(msg);
+      gui.alert('An unkown error occured');
+      throw new Error(msg);
     }
 
     function message() {
@@ -2540,7 +2545,7 @@
   // Convert a point from display CRS coordinates to data coordinates.
   // These are only different when using dynamic reprojection (basemap view).
   function translateDisplayPoint(lyr, p) {
-    return isProjectedLayer(lyr) ? lyr.invertPoint(p[0], p[1]) : p;
+    return isProjectedLayer(lyr) ? lyr.gui.invertPoint(p[0], p[1]) : p;
   }
 
   // bbox: display coords
@@ -2557,7 +2562,7 @@
 
   function isProjectedLayer(lyr) {
     // TODO: could do some validation on the layer's contents
-    return !!(lyr.source && lyr.invertPoint);
+    return !!lyr.gui.invertPoint;
   }
 
   var darkStroke = "#334",
@@ -2807,7 +2812,7 @@
       strokeWidth: 1.2,
       vertices: true,
       vertex_overlay_color: o.hit_type == 'vertex' ? violet : black,
-      vertex_overlay_scale: o.hit_type == 'vertex' ? 2.4 : 1.7,
+      vertex_overlay_scale: o.hit_type == 'vertex' ? 2.5 : 2,
       vertex_overlay: o.hit_coordinates || null,
       selected_points: o.selected_points || null,
       fillColor: null
@@ -2919,22 +2924,22 @@
   }
 
   function flattenArcs(lyr) {
-    lyr.source.dataset.arcs.flatten();
+    lyr.gui.source.dataset.arcs.flatten();
     if (isProjectedLayer(lyr)) {
-      lyr.arcs.flatten();
+      lyr.gui.displayArcs.flatten();
     }
   }
 
   function setZ(lyr, z) {
-    lyr.source.dataset.arcs.setRetainedInterval(z);
+    lyr.gui.source.dataset.arcs.setRetainedInterval(z);
     if (isProjectedLayer(lyr)) {
-      lyr.arcs.setRetainedInterval(z);
+      lyr.gui.displayArcs.setRetainedInterval(z);
     }
   }
 
   function updateZ(lyr) {
-    if (isProjectedLayer(lyr) && !lyr.source.dataset.arcs.isFlat()) {
-      lyr.arcs.setThresholds(lyr.source.dataset.arcs.getVertexData().zz);
+    if (isProjectedLayer(lyr) && !lyr.gui.source.dataset.arcs.isFlat()) {
+      lyr.gui.displayArcs.setThresholds(lyr.gui.source.dataset.arcs.getVertexData().zz);
     }
   }
 
@@ -2970,103 +2975,107 @@
   }
 
   function deleteLastPath(lyr) {
-    var arcId = lyr.arcs.size() - 1;
-    if (lyr.layer.data) {
-      lyr.layer.data.getRecords().pop();
+    var arcId = lyr.gui.displayArcs.size() - 1;
+    if (lyr.data) {
+      lyr.data.getRecords().pop();
     }
-    var shp = lyr.layer.shapes.pop();
-    internal.deleteLastArc(lyr.arcs);
+    var shp = lyr.shapes.pop();
+    internal.deleteLastArc(lyr.gui.displayArcs);
     if (isProjectedLayer(lyr)) {
-      internal.deleteLastArc(lyr.source.dataset.arcs);
+      internal.deleteLastArc(lyr.gui.source.dataset.arcs);
     }
   }
 
   // p1, p2: two points in source data CRS coords.
   function appendNewPath(lyr, p1, p2) {
-    var arcId = lyr.arcs.size();
-    internal.appendEmptyArc(lyr.arcs);
-    lyr.layer.shapes.push([[arcId]]);
+    var arcId = lyr.gui.displayArcs.size();
+    internal.appendEmptyArc(lyr.gui.displayArcs);
+    lyr.shapes.push([[arcId]]);
     if (isProjectedLayer(lyr)) {
-      // lyr.source.layer.shapes.push([[arcId]]);
-      internal.appendEmptyArc(lyr.source.dataset.arcs);
+      internal.appendEmptyArc(lyr.gui.source.dataset.arcs);
     }
     appendVertex$1(lyr, p1);
     appendVertex$1(lyr, p2);
-    appendNewDataRecord(lyr.layer);
+    appendNewDataRecord(lyr);
   }
 
   // p: point in source data CRS coords.
   function insertVertex$1(lyr, id, p) {
-    internal.insertVertex(lyr.source.dataset.arcs, id, p);
+    internal.insertVertex(lyr.gui.source.dataset.arcs, id, p);
     if (isProjectedLayer(lyr)) {
-      internal.insertVertex(lyr.arcs, id, lyr.projectPoint(p[0], p[1]));
+      internal.insertVertex(lyr.gui.displayArcs, id, lyr.gui.projectPoint(p[0], p[1]));
     }
   }
 
   function appendVertex$1(lyr, p) {
-    var n = lyr.source.dataset.arcs.getPointCount();
+    var n = lyr.gui.source.dataset.arcs.getPointCount();
     insertVertex$1(lyr, n, p);
   }
 
   // TODO: make sure we're not also removing an entire arc
   function deleteLastVertex(lyr) {
-    deleteVertex$1(lyr, lyr.arcs.getPointCount() - 1);
+    deleteVertex$1(lyr, lyr.gui.displayArcs.getPointCount() - 1);
   }
 
   function deleteVertex$1(lyr, id) {
-    internal.deleteVertex(lyr.arcs, id);
+    internal.deleteVertex(lyr.gui.displayArcs, id);
     if (isProjectedLayer(lyr)) {
-      internal.deleteVertex(lyr.source.dataset.arcs, id);
+      internal.deleteVertex(lyr.gui.source.dataset.arcs, id);
     }
   }
 
   function getLastArcCoords(target) {
-    var arcId = target.source.dataset.arcs.size() - 1;
-    return internal.getUnfilteredArcCoords(arcId, target.source.dataset.arcs);
+    var arcId = target.gui.source.dataset.arcs.size() - 1;
+    return internal.getUnfilteredArcCoords(arcId, target.gui.source.dataset.arcs);
+  }
+
+  function getLastVertexCoords(target) {
+    var arcs = target.gui.source.dataset.arcs;
+    return internal.getVertexCoords(arcs.getPointCount() - 1, arcs);
   }
 
   function getLastArcLength(target) {
-    var arcId = target.source.dataset.arcs.size() - 1;
-    return internal.getUnfilteredArcLength(arcId, target.source.dataset.arcs);
+    var arcId = target.gui.source.dataset.arcs.size() - 1;
+    return internal.getUnfilteredArcLength(arcId, target.gui.source.dataset.arcs);
   }
 
   function getPointCoords(lyr, fid) {
-    return internal.cloneShape(lyr.source.layer.shapes[fid]);
+    return internal.cloneShape(lyr.shapes[fid]);
   }
 
   function getVertexCoords(lyr, id) {
-    return lyr.source.dataset.arcs.getVertex2(id);
+    return lyr.gui.source.dataset.arcs.getVertex2(id);
   }
 
   // set data coords (not display coords) of one or more vertices.
   function setVertexCoords(lyr, ids, dataPoint) {
-    internal.snapVerticesToPoint(ids, dataPoint, lyr.source.dataset.arcs, true);
+    internal.snapVerticesToPoint(ids, dataPoint, lyr.gui.source.dataset.arcs);
     if (isProjectedLayer(lyr)) {
-      var p = lyr.projectPoint(dataPoint[0], dataPoint[1]);
-      internal.snapVerticesToPoint(ids, p, lyr.arcs, true);
+      var p = lyr.gui.projectPoint(dataPoint[0], dataPoint[1]);
+      internal.snapVerticesToPoint(ids, p, lyr.gui.displayArcs);
     }
   }
 
   // coords: [x, y] point in data CRS (not display CRS)
   function setPointCoords(lyr, fid, coords) {
-    lyr.source.layer.shapes[fid] = coords;
+    lyr.shapes[fid] = coords;
     if (isProjectedLayer(lyr)) {
-      lyr.layer.shapes[fid] = projectPointCoords(coords, lyr.projectPoint);
+      lyr.shapes[fid] = projectPointCoords(coords, lyr.gui.projectPoint);
     }
   }
 
   function updateVertexCoords(lyr, ids) {
     if (!isProjectedLayer(lyr)) return;
-    var p = lyr.arcs.getVertex2(ids[0]);
-    internal.snapVerticesToPoint(ids, lyr.invertPoint(p[0], p[1]), lyr.source.dataset.arcs, true);
+    var p = lyr.gui.displayArcs.getVertex2(ids[0]);
+    internal.snapVerticesToPoint(ids, lyr.gui.invertPoint(p[0], p[1]), lyr.gui.source.dataset.arcs);
   }
 
   function setRectangleCoords(lyr, ids, coords) {
     ids.forEach(function(id, i) {
       var p = coords[i];
-      internal.snapVerticesToPoint([id], p, lyr.source.dataset.arcs, true);
+      internal.snapVerticesToPoint([id], p, lyr.gui.source.dataset.arcs);
       if (isProjectedLayer(lyr)) {
-        internal.snapVerticesToPoint([id], lyr.projectPoint(p[0], p[1]), lyr.arcs, true);
+        internal.snapVerticesToPoint([id], lyr.gui.projectPoint(p[0], p[1]), lyr.gui.displayArcs);
       }
     });
   }
@@ -3074,8 +3083,8 @@
   // Update source data coordinates by projecting display coordinates
   function updatePointCoords(lyr, fid) {
     if (!isProjectedLayer(lyr)) return;
-    var displayShp = lyr.layer.shapes[fid];
-    lyr.source.layer.shapes[fid] = projectPointCoords(displayShp, lyr.invertPoint);
+    var displayShp = lyr.shapes[fid];
+    lyr.shapes[fid] = projectPointCoords(displayShp, lyr.gui.invertPoint);
   }
 
   function projectPointCoords(src, proj) {
@@ -4212,6 +4221,11 @@
     return layers.reverse();
   }
 
+  async function saveFileContentToClipboard(content) {
+    var str = utils$1.isString(content) ? content : content.toString();
+    await navigator.clipboard.writeText(str);
+  }
+
   // Export buttons and their behavior
   var ExportControl = function(gui) {
     var model = gui.model;
@@ -4355,12 +4369,6 @@
       } else {
         await utils$1.promisify(internal.writeFiles)(files, opts);
       }
-
-    }
-
-    async function saveFileContentToClipboard(content) {
-      var str = utils$1.isString(content) ? content : content.toString();
-      await navigator.clipboard.writeText(str);
     }
 
     function initLayerItem(o, i) {
@@ -4920,13 +4928,10 @@
       return internal.getLayerSourceFile(lyr, dataset);
     }
 
-
     function isPinnable(lyr) {
-      return internal.layerHasGeometry(lyr) || internal.layerHasFurniture(lyr);
+      return internal.layerIsGeometric(lyr) || internal.layerHasFurniture(lyr);
     }
 
-
-
     function rowHTML(c1, c2, cname) {
       return utils$1.format('<div class="row%s"><div class="col1">%s</div>' +
         '<div class="col2">%s</div></div>', cname ? ' ' + cname : '', c1, c2);
@@ -5205,10 +5210,10 @@
 
     gui.on('path_extend', function(e) {
       var redo = function() {
-        gui.dispatchEvent('redo_path_extend', {p: e.p});
+        gui.dispatchEvent('redo_path_extend', {p: e.p, shapes: e.shapes2});
       };
       var undo = function() {
-        gui.dispatchEvent('undo_path_extend');
+        gui.dispatchEvent('undo_path_extend', {shapes: e.shapes1});
       };
       addHistoryState(undo, redo);
     });
@@ -5381,19 +5386,23 @@
   function KeyboardEvents(gui) {
     var self = this;
     var shiftDown = false;
+    var ctrlDown = false;
     document.addEventListener('keyup', function(e) {
       if (!GUI.isActiveInstance(gui)) return;
       if (e.keyCode == 16) shiftDown = false;
+      if (e.keyCode == 17) ctrlDown = false;
       self.dispatchEvent('keyup', getEventData(e));
     });
 
     document.addEventListener('keydown', function(e) {
       if (!GUI.isActiveInstance(gui)) return;
       if (e.keyCode == 16) shiftDown = true;
+      if (e.keyCode == 17) ctrlDown = true;
       self.dispatchEvent('keydown', getEventData(e));
     });
 
     this.shiftIsPressed = function() { return shiftDown; };
+    this.ctrlIsPressed = function() { return ctrlDown; };
 
     this.onMenuSubmit = function(menuEl, cb) {
       gui.on('enter_key', function(e) {
@@ -5410,6 +5419,7 @@
     9: 'tab',
     13: 'enter',
     16: 'shift',
+    17: 'ctrl',
     27: 'esc',
     32: 'space',
     37: 'left',
@@ -5431,7 +5441,7 @@
     var menus = {
       standard: ['info', 'selection', 'box'],
       polygons: ['info', 'selection', 'box', 'drawing'],
-      rectangles: ['info', 'selection', 'box', 'rectangles'], // 'vertices'
+      rectangles: ['info', 'selection', 'box', 'rectangles'],
       lines: ['info', 'selection', 'box' , 'drawing'],
       table: ['info', 'selection'],
       labels: ['info', 'selection', 'box', 'labels', 'location', 'add-points'],
@@ -5452,7 +5462,7 @@
       labels: 'position labels',
       location: 'drag points',
       vertices: 'edit vertices',
-      selection: 'select features',
+      selection: 'selection tool',
       'add-points': 'add points',
       drawing: 'draw/reshape tool',
       rectangles: 'drag-to-resize',
@@ -5543,22 +5553,23 @@
       if (!o || !o.layer) {
         return menus.standard; // TODO: more sensible handling of missing layer
       }
-      if (!internal.layerHasGeometry(o.layer)) {
+      if (!o.layer.geometry_type) {
         return menus.table;
       }
       if (internal.layerHasLabels(o.layer)) {
         return menus.labels;
       }
-      if (internal.layerHasPoints(o.layer)) {
+      if (o.layer.geometry_type == 'point') {
         return menus.points;
       }
-      if (internal.layerHasPaths(o.layer) && o.layer.geometry_type == 'polyline') {
+      if (o.layer.geometry_type == 'polyline') {
         return menus.lines;
       }
-      if (internal.layerHasPaths(o.layer) && o.layer.geometry_type == 'polygon') {
+      if (o.layer.geometry_type == 'polygon') {
         return internal.layerOnlyHasRectangles(o.layer, o.dataset.arcs) ?
           menus.rectangles : menus.polygons;
       }
+
       return menus.standard;
     }
 
@@ -5908,13 +5919,13 @@
   }
 
   // Format an array of (preferably short) strings in columns for console logging.
-  function formatStringsAsGrid(arr) {
+  function formatStringsAsGrid(arr, width) {
     // TODO: variable column width
     var longest = arr.reduce(function(len, str) {
           return Math.max(len, str.length);
         }, 0),
         colWidth = longest + 2,
-        perLine = Math.floor(80 / colWidth) || 1;
+        perLine = Math.floor((width || 80) / colWidth) || 1;
     return arr.reduce(function(memo, name, i) {
       var col = i % perLine;
       if (i > 0 && col === 0) memo += '\n';
@@ -6575,7 +6586,7 @@
   }
 
   function getGenericComparator(asc) {
-    asc = asc !== false;
+    asc = asc !== false && asc != 'descending'; // ascending is the default
     return function(a, b) {
       var retn = 0;
       if (b == null) {
@@ -7244,11 +7255,11 @@
 
   // featureFilter: optional test function, accepts feature id
   //
-  function getShapeHitTest(displayLayer, ext, interactionMode, featureFilter) {
-    var geoType = displayLayer.layer.geometry_type;
+  function getShapeHitTest(layer, ext, interactionMode, featureFilter) {
+    var geoType = layer.gui.displayLayer.geometry_type;
     var test;
-    if (geoType == 'point' && displayLayer.style.type == 'styled') {
-      test = getGraduatedCircleTest(getRadiusFunction(displayLayer.style));
+    if (geoType == 'point' && layer.gui.style.type == 'styled') {
+      test = getGraduatedCircleTest(getRadiusFunction(layer.gui.style));
     } else if (geoType == 'point') {
       test = pointTest;
     } else if (interactionMode == 'drawing' && geoType == 'polygon') {
@@ -7288,13 +7299,13 @@
           cand, hitId;
       for (var i=0; i<cands.length; i++) {
         cand = cands[i];
-        if (geom.testPointInPolygon(x, y, cand.shape, displayLayer.arcs)) {
+        if (geom.testPointInPolygon(x, y, cand.shape, layer.gui.displayArcs)) {
           hits.push(cand);
         }
       }
       if (cands.length > 0 && hits.length === 0) {
         // secondary detection: proximity, if not inside a polygon
-        sortByDistance(x, y, cands, displayLayer.arcs);
+        sortByDistance(x, y, cands, layer.gui.displayArcs);
         hits = pickNearestCandidates(cands, 0, maxDist);
       }
       return {
@@ -7319,7 +7330,7 @@
           bufPix = bufArg >= 0 ? bufArg : 0.05, // tiny threshold for hitting almost-identical lines
           bufDist = getZoomAdjustedHitBuffer(bufPix),
           cands = findHitCandidates(x, y, maxDist);
-      sortByDistance(x, y, cands, displayLayer.arcs);
+      sortByDistance(x, y, cands, layer.gui.displayArcs);
       cands = pickNearestCandidates(cands, bufDist, maxDist);
       return {
         ids: utils$1.pluck(cands, 'id')
@@ -7360,7 +7371,7 @@
           hits = [];
 
       // inlining forEachPoint() does not not appreciably speed this up
-      internal.forEachPoint(displayLayer.layer.shapes, function(p, id) {
+      internal.forEachPoint(layer.gui.displayLayer.shapes, function(p, id) {
         var dist = geom.distance2D(x, y, p[0], p[1]) * toPx;
         if (dist > hitThreshold) return;
         if (dist < hitThreshold && hitThreshold > bullseyeDist) {
@@ -7394,7 +7405,7 @@
             directHit = false,
             hitRadius = 0,
             hitDist;
-        internal.forEachPoint(displayLayer.layer.shapes, function(p, id) {
+        internal.forEachPoint(layer.gui.displayLayer.shapes, function(p, id) {
           var distSq = geom.distanceSq(x, y, p[0], p[1]);
           var isHit = false;
           var isOver, isNear, r, d, rpix;
@@ -7434,11 +7445,11 @@
 
     // Returns array of shape ids for shapes that pass a buffered bounding-box test
     function findHitCandidates(x, y, dist) {
-      var arcs = displayLayer.arcs,
+      var arcs = layer.gui.displayArcs,
           index = {},
           cands = [],
           bbox = [];
-      displayLayer.layer.shapes.forEach(function(shp, shpId) {
+      layer.gui.displayLayer.shapes.forEach(function(shp, shpId) {
         var cand;
         if (featureFilter && !featureFilter(shpId)) {
           return;
@@ -7570,7 +7581,7 @@
 
   function getPointerHitTest(mapLayer, ext, interactionMode, featureFilter) {
     var shapeTest, targetLayer;
-    if (!mapLayer || !internal.layerHasGeometry(mapLayer.layer)) {
+    if (!mapLayer || !internal.layerHasGeometry(mapLayer.gui?.displayLayer)) {
       return function() {return {ids: []};};
     }
     shapeTest = getShapeHitTest(mapLayer, ext, interactionMode, featureFilter);
@@ -7609,6 +7620,13 @@
     // pan navigation
     var priority = 2;
 
+    mouse.on('contextmenu', function(e) {
+      if (isOverMap(e)) {
+        e.originalEvent.preventDefault();
+      }
+      triggerHitEvent('contextmenu', e);
+    }, false);
+
     // init keyboard controls for pinned features
     gui.keyboard.on('keydown', function(evt) {
       var e = evt.originalEvent;
@@ -7636,7 +7654,7 @@
         // to help protect against inadvertent deletion, don't delete
         // when console is open or a popup menu is open
         if (!gui.getMode() && !gui.consoleIsOpen()) {
-          internal.deleteFeatureById(targetLayer.layer, pinnedId());
+          internal.deleteFeatureById(targetLayer, pinnedId());
           self.clearSelection();
           gui.model.updated({flags: 'filter'}); // signal map to update
         }
@@ -7679,6 +7697,7 @@
 
     function pinnable() {
       return clickable() && !selectable();
+      // return clickable();
     }
 
     function draggable() {
@@ -7773,7 +7792,7 @@
 
     self.getTargetDataTable = function() {
       var targ = self.getHitTarget();
-      return targ && targ.layer.data || null;
+      return targ?.data || null;
     };
 
     // get function for selecting next or prev feature within the current set of
@@ -7786,7 +7805,7 @@
 
     // diff: 1 or -1
     function advanceSelectedFeature(diff) {
-      var n = internal.getFeatureCount(targetLayer.layer);
+      var n = internal.getFeatureCount(targetLayer);
       if (n < 2 || pinnedId() == -1) return;
       storedData.id = (pinnedId() + n + diff) % n;
       storedData.ids = [storedData.id];
@@ -7831,12 +7850,15 @@
 
     mouse.on('click', function(e) {
       if (!hitTest || !active) return;
+      if (!eventIsEnabled('click')) return;
+
       e.stopPropagation();
 
       // TODO: move pinning to inspection control?
       if (clickable()) {
         updateSelectionState(convertClickDataToSelectionData(hitTest(e)));
       }
+
       triggerHitEvent('click', e);
     }, null, priority);
 
@@ -7855,6 +7877,7 @@
       }
     }, null, priority);
 
+
     function targetIsRollover(target) {
       while (target.parentNode && target != target.parentNode) {
         if (target.className && String(target.className).indexOf('rollover') > -1) {
@@ -7893,6 +7916,19 @@
       return hitData;
     }
 
+    function mergeSelectionModeHoverData(hitData) {
+        if (hitData.ids.length === 0 || selectionIds.includes(hitData.ids[0])) {
+          hitData.ids = selectionIds;
+          hitData.pinned = storedData.pinned;
+        } else {
+          //
+        }
+
+        // kludge to inhibit hover effect while dragging a box
+        if (gui.keydown) hitData.id = -1;
+        return hitData;
+    }
+
     function mergeHoverData(hitData) {
       if (storedData.pinned) {
         hitData.id = storedData.id;
@@ -7947,14 +7983,21 @@
         return true; // special case -- using hover for line drawing animation
       }
 
+      if (type == 'click' && gui.keyboard.ctrlIsPressed()) {
+        return false; // don't fire if context menu might open
+      }
+      if (type == 'click' && gui.contextMenu.isOpen()) {
+        return false;
+      }
+      if (type == 'click' && interactionMode == 'drawing') {
+        return true; // click events are triggered even if no shape is hit
+      }
+
       // ignore pointer events when no features are being hit
       // (don't block pan and other navigation when events aren't being used for editing)
       var hitId = self.getHitId();
       if (hitId == -1) return false;
 
-      if (type == 'click' && !clickable()) {
-        return false;
-      }
       if ((type == 'drag' || type == 'dragstart' || type == 'dragend') && !draggable()) {
         return false;
       }
@@ -7983,10 +8026,13 @@
     // evt: event data (may be a pointer event object, an ordinary object or null)
     function triggerHitEvent(type, evt) {
       var eventData = {
-        mode: interactionMode,
-        overMap: evt ? isOverMap(evt) : null,
-        originalEvent: evt ? evt : null
+        mode: interactionMode
       };
+      if (evt) {
+        eventData.coordinates = translateDisplayPoint(targetLayer, ext.translatePixelCoords(evt.x, evt.y));
+        eventData.originalEvent = evt;
+        eventData.overMap = isOverMap(evt);
+      }
       // Merge stored hit data into the event data
       utils$1.defaults(eventData, evt && evt.data || {}, storedData);
       // utils.extend(eventData, storedData);
@@ -8033,7 +8079,7 @@
 
     function enabled() {
       var lyr = gui.map.getActiveLayer();
-      return !!(lyr && lyr.layer.geometry_type);
+      return !!(lyr && lyr.gui.displayLayer.geometry_type);
     }
 
     gui.model.on('update', function(e) {
@@ -8078,7 +8124,7 @@
       var p2 = gui.map.translatePixelCoords(ext.width(), 0);
       var bbox = p1.concat(p2);
       var decimals = internal.getBoundsPrecisionForDisplay(bbox);
-       var str = internal.getRoundedCoordString(p, decimals);
+      var str = internal.getRoundedCoordString(p, decimals);
       readout.text(str).show();
     }
 
@@ -8359,6 +8405,9 @@
     element.addEventListener('mouseout', onAreaOut);
     element.addEventListener('mousedown', onAreaDown);
     element.addEventListener('dblclick', onAreaDblClick);
+    document.addEventListener('contextmenu', function(e) {
+      _self.dispatchEvent('contextmenu', procMouseEvent(e));
+    });
 
     this.enable = function() {
       if (!_disabled) return;
@@ -8926,8 +8975,8 @@
     gui.on('interaction_mode_change', function(e) {
       if (e.mode === 'selection') {
         gui.enterMode('selection_tool');
-      } else if (gui.getMode() == 'selection_tool') {
-        gui.clearMode();
+      } else if (_on) {
+        turnOff();
       }
     });
 
@@ -9393,8 +9442,17 @@
       }
     });
 
+    hit.on('contextmenu', function(e) {
+      var target = hit.getHitTarget();
+      if (!e.overMap || !target) return;
+
+      if (e.mode == 'drawing') return; // TODO: make special menu for this mode
+      gui.contextMenu.open(e, hit.getHitTarget());
+    });
+
     hit.on('change', function(e) {
       if (!inspecting()) return;
+      if (gui.keyboard.ctrlIsPressed()) return;
       var ids;
       if (e.mode == 'selection') {
         ids = e.pinned && e.ids || [];
@@ -9407,8 +9465,8 @@
     // id: Id of a feature in the active layer, or -1
     function inspect(id, ids, pin) {
       var target = hit.getHitTarget();
-      if ((id > -1 || ids && ids.length > 0) && inspecting() && target && target.layer) {
-        _popup.show(id, ids, target.layer, pin);
+      if ((id > -1 || ids && ids.length > 0) && inspecting() && target && target) {
+        _popup.show(id, ids, target, pin);
       } else {
         _popup.hide();
       }
@@ -9512,15 +9570,20 @@
 
   function initLabelDragging(gui, ext, hit) {
     var downEvt;
-    var activeId;
+    var activeId = -1;
+    var prevHitEvt;
     var activeRecord;
 
-    function active(e) {
-      return e.id > -1 && gui.interaction.getMode() == 'labels';
+    function active() {
+      return gui.interaction.getMode() == 'labels';
+    }
+
+    function labelSelected(e) {
+      return e.id > -1 && active();
     }
 
     hit.on('dragstart', function(e) {
-      if (!active(e)) return;
+      if (!labelSelected(e)) return;
       var symNode = getSymbolTarget(e);
       var table = hit.getTargetDataTable();
       if (!symNode || !table) {
@@ -9533,8 +9596,25 @@
       gui.dispatchEvent('label_dragstart', {FID: activeId});
     });
 
+    hit.on('change', function(e) {
+      if (!active()) return;
+      if (prevHitEvt) clearLabelHighlight(prevHitEvt);
+      showLabelHighlight(e);
+      prevHitEvt = e;
+    });
+
+    function clearLabelHighlight(e) {
+      var txt = getTextNode(getSymbolTarget(e));
+      if (txt) txt.classList.remove('active-label');
+    }
+
+    function showLabelHighlight(e) {
+      var txt = getTextNode(getSymbolTarget(e));
+      if (txt) txt.classList.add('active-label');
+    }
+
     hit.on('drag', function(e) {
-      if (!active(e) || activeId == -1) return;
+      if (!labelSelected(e) || activeId == -1) return;
       if (e.id != activeId) {
         error$1("Mismatched hit ids:", e.id, activeId);
       }
@@ -9548,7 +9628,7 @@
       // to anchor point, for better placement when eventual display font is
       // different from mapshaper's font.
       // if (!isMultilineLabel(textNode)) {
-      autoUpdateTextAnchor(textNode, activeRecord, getDisplayCoordsById(activeId, hit.getHitTarget().layer, ext));
+      autoUpdateTextAnchor(textNode, activeRecord, getDisplayCoordsById(activeId, hit.getHitTarget(), ext));
       // }
       updateNumber(activeRecord, 'dx', internal.roundToDigits(+activeRecord.dx, 3));
       updateNumber(activeRecord, 'dy', internal.roundToDigits(+activeRecord.dy, 3));
@@ -9558,7 +9638,7 @@
     });
 
     hit.on('dragend', function(e) {
-      if (!active(e) || activeId == -1) return;
+      if (!labelSelected(e) || activeId == -1) return;
       gui.dispatchEvent('label_dragend', {FID: e.id});
       activeId = -1;
       activeRecord = null;
@@ -9579,19 +9659,25 @@
     }
 
     function getSymbolTarget(e) {
-      if (e.id > -1 === false || !e.container) return null;
-      return getSymbolNodeById(e.id, e.container);
+      return e.id > -1 ? getSymbolNodeById(e.id) : null;
     }
 
     function getTextNode(symNode) {
+      if (!symNode) return null;
       if (symNode.tagName == 'text') return symNode;
       return symNode.querySelector('text');
     }
 
-    function getSymbolNodeById(id, parent) {
+    // function getSymbolNodeById_OLD(id, parent) {
+    //   var sel = '[data-id="' + id + '"]';
+    //   return parent.querySelector(sel);
+    // }
+
+    function getSymbolNodeById(id) {
       // TODO: optimize selector
       var sel = '[data-id="' + id + '"]';
-      return parent.querySelector(sel);
+      var activeLayer = hit.getHitTarget();
+      return activeLayer.gui.svg_container.querySelector(sel);
     }
 
     // function getTextTarget2(e) {
@@ -9638,7 +9724,7 @@
     // fails when symbol includes a dot (<g><circle/><text/></g> structure)
     function updateSymbolNode(node, d, id) {
       var o = internal.svg.renderStyledLabel(d); // TODO: symbol support
-      var activeLayer = hit.getHitTarget().layer;
+      var activeLayer = hit.getHitTarget();
       var xy = activeLayer.shapes[id][0];
       var g = document.createElementNS('http://www.w3.org/2000/svg', 'g');
       var node2;
@@ -9671,7 +9757,7 @@
     hit.on('drag', function(e) {
       if (!active(e)) return;
       // TODO: support multi points... get id of closest part to the pointer
-      var p = getPointCoordsById(e.id, symbolInfo.target.layer);
+      var p = getPointCoordsById(e.id, symbolInfo.target);
       if (!p) return;
       var diff = translateDeltaDisplayCoords(e.dx, e.dy, ext);
       p[0] += diff[0];
@@ -9702,20 +9788,22 @@
     }
   }
 
-  // pointer thresholds for hovering near a vertex or segment midpoint
-  var HOVER_THRESHOLD$1 = 10;
+  // pixel distance threshold for hovering near a vertex or segment midpoint
+  var HOVER_THRESHOLD = 10;
 
   function initLineEditing(gui, ext, hit) {
     var hoverVertexInfo;
     var prevClickEvent;
     var prevHoverEvent;
     var initialArcCount = -1;
+    var initialShapeCount = -1;
     var drawingId = -1; // feature id of path being drawn
+    var sessionCount = 0;
     var alert;
     var _dragging = false;
 
     function active() {
-      return !!alert;
+      return initialArcCount >= 0;
     }
 
     function dragging() {
@@ -9727,7 +9815,7 @@
     }
 
     function polygonMode() {
-      return active() && hit.getHitTarget().layer.geometry_type == 'polygon';
+      return active() && hit.getHitTarget().geometry_type == 'polygon';
     }
 
     function clearHoverVertex() {
@@ -9743,7 +9831,7 @@
       if (e.mode == 'drawing') {
         gui.enterMode('drawing_tool');
       } else if (active()) {
-        gui.clearMode();
+        turnOff();
       }
     }, null, 10); // higher priority than hit control, so turnOff() has correct hit target
 
@@ -9753,7 +9841,7 @@
       appendNewPath(target, e.p1, e.p2);
       deleteLastVertex(target); // second vertex is a placeholder
       gui.undo.redo(); // add next vertex in the path
-      gui.model.updated({arc_count: true});
+      fullRedraw();
     });
 
     gui.on('undo_path_add', function(e) {
@@ -9769,6 +9857,9 @@
       } else {
         appendVertex$1(target, e.p);
       }
+      if (e.shapes) {
+        replaceDrawnShapes(e.shapes);
+      }
     });
 
     gui.on('undo_path_extend', function(e) {
@@ -9782,75 +9873,74 @@
       if (getLastArcLength(target) < 2) {
         gui.undo.undo(); // remove the path
       }
+      if (e.shapes) {
+        replaceDrawnShapes(e.shapes);
+      }
     });
 
     function turnOn() {
       var target = hit.getHitTarget();
-      var pathStr = polygonMode() ? 'closed paths' : 'paths';
+      initialArcCount = target.gui.displayArcs.size();
+      initialShapeCount = target.shapes.length;
+      if (sessionCount === 0) {
+        showInstructions();
+      }
+      sessionCount++;
+    }
+
+    function showInstructions() {
       var isMac = navigator.userAgent.includes('Mac');
       var symbol = isMac ? '⌘' : '^';
+      var pathStr = polygonMode() ? 'closed paths' : 'paths';
       var msg = `Instructions: Click on the map to draw ${pathStr}. Drag vertices to reshape a path. Type ${symbol}Z/${symbol}Y to undo/redo.`;
-      initialArcCount = hit.getHitTarget().arcs.size();
-      alert = showPopupAlert(msg, null, {non_blocking: true, max_width: '360px'});
+        alert = showPopupAlert(msg, null, {
+          non_blocking: true, max_width: '360px'});
+    }
+
+    function hideInstructions() {
+      if (!alert) return;
+      alert.close('fade');
+      alert = null;
     }
 
     function turnOff() {
-      finishPath();
+      var removed = 0;
+      finishCurrentPath();
       if (polygonMode()) {
-        finishPolygons();
+        removed = removeOpenPolygons();
       }
       clearDrawingInfo();
-      alert.close();
-      alert = null;
+      hideInstructions();
       initialArcCount = -1;
+      initialShapeCount = -1;
       if (gui.interaction.getMode() == 'drawing') {
         // mode change was not initiated by interactive menu -- turn off interactivity
         gui.interaction.turnOff();
       }
-    }
-
-    function finish() {
-      if (polygonMode()) {
-        finishPolygons();
+      if (removed > 0) {
+        fullRedraw();
       }
     }
 
-    function finishPolygons() {
-      // step1: make a polyline layer containing just newly drawn paths
+    // returns number of removed shapes
+    function removeOpenPolygons() {
       var target = hit.getHitTarget();
-      var newShapeCount = target.arcs.size() - initialArcCount;
-      if (newShapeCount <= 0) return; // no paths added
-      var polygonLyr = target.source.layer;
-      var polygonRecords = polygonLyr.data ? polygonLyr.data.getRecords() : null;
-      var initialShapeCount = polygonLyr.shapes.length - newShapeCount;
-      var templateRecord;
-      if (polygonRecords) {
-        // use one of the newly created records as a template (they should all be the same)
-        templateRecord = polygonRecords.pop();
-        polygonRecords.splice(initialShapeCount); // remove new records
-      }
-      var polylineLyr = {
-        geometry_type: 'polyline',
-        shapes: polygonLyr.shapes.splice(initialShapeCount) // move new shapes to polyline layer
-      };
-
-      // step2: convert polylines to polygons
-      //
-      // create a temp dataset containing both layers (so original arcs are preserved)
-      var tmp = Object.assign({}, target.source.dataset);
-      tmp.layers = tmp.layers.concat(polylineLyr);
-      var outputLayers = mapshaper.cmd.polygons([polylineLyr], tmp, {});
-
-      // step3: add new polygons to the original polygons
-      outputLayers[0].shapes.forEach(function(shp) {
-        var rec = polygonRecords ? internal.copyRecord(templateRecord) : null;
-        polygonLyr.shapes.push(shp);
-        if (polygonRecords) {
-          polygonRecords.push(rec);
+      var arcs = target.gui.source.dataset.arcs;
+      var n = target.shapes.length;
+      // delete open paths
+      for (var i=initialShapeCount; i<n; i++) {
+        var shp = target.shapes[i];
+        if (!geom.pathIsClosed(shp[0], arcs)) { // assume open paths have one arc
+          target.shapes[i] = null;
         }
-      });
+      }
+      // removes features with wrong winding order or null geometry
+      mapshaper.cmd.filterFeatures(target, arcs, {remove_empty: true, quiet: true});
+      return n - target.shapes.length;
+    }
 
-      // step4: update map
+    // updates display arcs and redraws all layers
+    function fullRedraw() {
       gui.model.updated({arc_count: true});
     }
 
@@ -9861,10 +9951,25 @@
       prevClickEvent = prevHoverEvent = null;
     }
 
+    hit.on('contextmenu', function(e) {
+      if (!active() || drawing() || dragging()) return;
+      var target = hit.getHitTarget();
+      var vInfo = hoverVertexInfo;
+      if (hoverVertexInfo?.type == 'vertex' && !vertexIsEndpoint(vInfo, target)) {
+        e.deleteVertex = function() {
+          deleteActiveVertex(e, vInfo);
+        };
+      }
+      // don't allow copying of open paths as geojson in polygon mode
+      gui.contextMenu.open(e, target);
+    });
+
     hit.on('dragstart', function(e) {
       if (!active() || drawing() || !hoverVertexInfo) return;
+      hideInstructions();
       e.originalEvent.stopPropagation();
       _dragging = true;
+      updateCursor();
       if (hoverVertexInfo.type == 'interpolated') {
         insertVertex$1(hit.getHitTarget(), hoverVertexInfo.i, hoverVertexInfo.point);
         hoverVertexInfo.ids = [hoverVertexInfo.i];
@@ -9878,11 +9983,10 @@
       var target = hit.getHitTarget();
       var p = ext.translatePixelCoords(e.x, e.y);
       if (gui.keyboard.shiftIsPressed()) {
-        internal.snapPointToArcEndpoint(p, hoverVertexInfo.ids, target.arcs);
+        internal.snapPointToArcEndpoint(p, hoverVertexInfo.ids, target.gui.displayArcs);
       }
-      internal.snapVerticesToPoint(hoverVertexInfo.ids, p, target.arcs);
+      internal.snapVerticesToPoint(hoverVertexInfo.ids, p, target.gui.displayArcs);
       hit.setHoverVertex(p, '');
-
       // redrawing the whole map updates the data layer as well as the overlay layer
       // gui.dispatchEvent('map-needs-refresh');
     });
@@ -9890,9 +9994,9 @@
     hit.on('dragend', function(e) {
       if (!dragging()) return;
       _dragging = false;
-      // kludge to get dataset to recalculate internal bounding boxes
       var target = hit.getHitTarget();
-      target.arcs.transformPoints(function() {});
+      // kludge to get dataset to recalculate internal bounding boxes
+      target.gui.displayArcs.transformPoints(function() {});
       updateVertexCoords(target, hoverVertexInfo.ids);
       gui.dispatchEvent('vertex_dragend', hoverVertexInfo);
       gui.dispatchEvent('map-needs-refresh'); // redraw basemap
@@ -9903,12 +10007,10 @@
     // double-click finishes a path (when drawing)
     hit.on('dblclick', function(e) {
       if (!active()) return;
+      // double click finishes a path
+      // note: if the preceding 'click' finished the path, this does not fire
       if (drawing()) {
-        // double click finishes a path
-        // before: dblclick is preceded by two clicks, need another vertex delete
-        // now: second click is suppressed
-        // deleteLastVertex(hit.getHitTarget());
-        finishPath();
+        finishCurrentPath();
         e.originalEvent.stopPropagation(); // prevent dblclick zoom
         return;
       }
@@ -9918,9 +10020,10 @@
     // ... or the closest point along the segment (for adding a new vertex)
     hit.on('hover', function(e) {
       if (!active() || dragging()) return;
+
       if (drawing()) {
         if (!e.overMap) {
-          finishPath();
+          finishCurrentPath();
           return;
         }
         if (gui.keyboard.shiftIsPressed()) {
@@ -9940,6 +10043,7 @@
       } else {
         clearHoverVertex();
       }
+      updateCursor();
       prevHoverEvent = e;
     }, null, 100);
 
@@ -9948,27 +10052,26 @@
       if (!active()) return;
       if (detectDoubleClick(e)) return; // ignore second click of a dblclick
       var p = pixToDataCoords(e.x, e.y);
-      if (drawing() && hoverVertexInfo) {
-        // finish the path if a vertex is highlighted
-        p = hoverVertexInfo.point;
-        extendPath(p);
-        finishPath();
-      } else if (drawing()) {
-        extendPath(p);
+      if (drawing()) {
+        extendCurrentPath(hoverVertexInfo?.point || p);
       } else if (gui.keyboard.shiftIsPressed()) {
         deleteActiveVertex(e);
       } else {
-        startPath(p);
+        startNewPath(p);
+        hideInstructions();
+        updateCursor();
       }
       prevClickEvent = e;
     });
 
-    // esc key finishes a path
+    // esc or enter key finishes a path
     gui.keyboard.on('keydown', function(e) {
-      if (active() && e.keyName == 'esc') {
-        finishPath();
+      if (active() && (e.keyName == 'esc' || e.keyName == 'enter')) {
+        e.stopPropagation();
+        finishCurrentPath();
+        e.originalEvent.preventDefault(); // block console "enter"
       }
-    });
+    }, null, 10);
 
     // detect second 'click' event of a double-click action
     function detectDoubleClick(evt) {
@@ -9980,16 +10083,24 @@
       return dbl;
     }
 
-    function deleteActiveVertex(e) {
-      var info = findDraggableVertices(e);
-      if (!info) return;
+    function updateCursor() {
+      var useArrow = hoverVertexInfo && !hoverVertexInfo.extendable && !drawing();
+      gui.container.findChild('.map-layers').classed('dragging', useArrow);
+    }
+
+    function vertexIsEndpoint(info, target) {
+      var vId = info.ids[0];
+      return internal.vertexIsArcStart(vId, target.gui.displayArcs) ||
+        internal.vertexIsArcEnd(vId, target.gui.displayArcs);
+    }
+
+    // info: optional vertex info object
+    function deleteActiveVertex(e, infoArg) {
+      var info = infoArg || findDraggableVertices(e);
+      if (!info || info.type != 'vertex') return;
       var vId = info.ids[0];
       var target = hit.getHitTarget();
-      if (internal.vertexIsArcStart(vId, target.arcs) ||
-          internal.vertexIsArcEnd(vId, target.arcs)) {
-        // TODO: support removing arc endpoints
-        return;
-      }
+      if (vertexIsEndpoint(info, target)) return;
       gui.dispatchEvent('vertex_delete', {
         target: target,
         vertex_id: vId
@@ -10015,28 +10126,30 @@
       var dist2 = dist / Math.sqrt(2);
       var minDist = Infinity;
       var cands = [
-        {x: x0, y: y0 + dist},
-        {x: x0, y: y0 - dist},
-        {x: x0 + dist, y: y0},
-        {x: x0 - dist, y: y0},
-        {x: x0 + dist2, y: y0 + dist2},
-        {x: x0 + dist2, y: y0 - dist2},
-        {x: x0 - dist2, y: y0 + dist2},
-        {x: x0 - dist2, y: y0 - dist2}
+        {dx: 0, dy: dist},
+        {dx: 0, dy: -dist},
+        {dx: dist, dy: 0},
+        {dx: -dist, dy: 0},
+        {dx: dist2, dy: dist2},
+        {dx: dist2, dy: -dist2},
+        {dx: -dist2, dy: dist2},
+        {dx: -dist2, dy: -dist2}
       ];
       var snapped = cands.reduce(function(memo, cand) {
-        var dist = geom.distance2D(thisEvt.x, thisEvt.y, cand.x, cand.y);
+        var dist = geom.distance2D(thisEvt.x, thisEvt.y, x0 + cand.dx, y0 + cand.dy);
         if (dist < minDist) {
           minDist = dist;
           return cand;
         }
         return memo;
       }, null);
-      thisEvt.x = snapped.x;
-      thisEvt.y = snapped.y;
+      thisEvt.x = x0 + snapped.dx;
+      thisEvt.y = y0 + snapped.dy;
+
+      return null;
     }
 
-    function finishPath() {
+    function finishCurrentPath() {
       if (!drawing()) return;
       var target = hit.getHitTarget();
       if (getLastArcLength(target) <= 2) { // includes hover point
@@ -10045,33 +10158,64 @@
         deleteLastVertex(target);
       }
       clearDrawingInfo();
-      gui.model.updated({arc_count: true});
+      fullRedraw();
     }
 
     // p: [x, y] source data coordinates
-    function startPath(p2) {
+    function startNewPath(p2) {
       var target = hit.getHitTarget();
-      var p1 = hoverVertexInfo ? hoverVertexInfo.point : p2;
+      var p1 = hoverVertexInfo?.point || p2;
       appendNewPath(target, p1, p2);
       gui.dispatchEvent('path_add', {target, p1, p2});
-      drawingId = target.layer.shapes.length - 1;
+      drawingId = target.shapes.length - 1;
       hit.setDrawingId(drawingId);
     }
 
-    // p: [x, y] source data coordinates
-    function extendPath(p) {
+    // p: [x, y] source data coordinates of new point on path
+    function extendCurrentPath(p) {
       var target = hit.getHitTarget();
-      if (getLastArcLength(target) >= 2) {
+      var shapes1, shapes2;
+      // finish the path if a vertex is selected (but not an interpolated point)
+      var finish = hoverVertexInfo?.type == 'vertex';
+      if (getLastArcLength(target) < 2) {
+        error$1('Defective path');
+      }
+      if (finish && polygonMode()) {
+        shapes1 = target.shapes.slice(initialShapeCount);
+        shapes2 = convertClosedPaths(shapes1);
+      }
+      if (shapes2) {
+        replaceDrawnShapes(shapes2);
+        gui.dispatchEvent('path_extend', {target, p, shapes1, shapes2});
+        clearDrawingInfo();
+        fullRedraw();
+
+      } else {
+        appendVertex$1(target, p);
         gui.dispatchEvent('path_extend', {target, p});
+        hit.triggerChangeEvent(); // trigger overlay redraw
+      }
+    }
+
+    function replaceDrawnShapes(shapes) {
+      var target = hit.getHitTarget();
+      var records = target.data?.getRecords();
+      var prevLen = target.shapes.length;
+      var newLen = initialShapeCount + shapes.length;
+      var recordCount = records?.length || 0;
+      target.shapes = target.shapes.slice(0, initialShapeCount).concat(shapes);
+      while (records && records.length > newLen) {
+        records.pop();
+      }
+      while (records && records.length < newLen) {
+        appendNewDataRecord(target);
       }
-      appendVertex$1(target, p);
-      hit.triggerChangeEvent();
     }
 
     // p: [x, y] source data coordinates
     function updatePathEndpoint(p) {
       var target = hit.getHitTarget();
-      var i = target.arcs.getPointCount() - 1;
+      var i = target.gui.displayArcs.getPointCount() - 1;
       if (hoverVertexInfo) {
         p = hoverVertexInfo.point; // snap to selected point
       }
@@ -10081,8 +10225,8 @@
 
     function findPathStartInfo(e) {
       var target = hit.getHitTarget();
-      var arcId = target.arcs.size() - 1;
-      var data = target.arcs.getVertexData();
+      var arcId = target.gui.displayArcs.size() - 1;
+      var data = target.gui.displayArcs.getVertexData();
       var i = data.ii[arcId];
       var x = data.xx[i];
       var y = data.yy[i];
@@ -10090,7 +10234,7 @@
       var dist = geom.distance2D(p[0], p[1], x, y);
       var pathLen = data.nn[arcId];
       var pixelDist = dist / ext.getPixelSize();
-      if (pixelDist > HOVER_THRESHOLD$1 || pathLen < 4) {
+      if (pixelDist > HOVER_THRESHOLD || pathLen < 4) {
         return null;
       }
       var point = translateDisplayPoint(target, [x, y]);
@@ -10104,41 +10248,38 @@
     //
     function findDraggableVertices(e) {
       var target = hit.getHitTarget();
-      var shp = target.layer.shapes[e.id];
+      var shp = target.shapes[e.id];
       var p = ext.translatePixelCoords(e.x, e.y);
-      var ids = internal.findNearestVertices(p, shp, target.arcs);
-      var p2 = target.arcs.getVertex2(ids[0]);
+      var ids = internal.findNearestVertices(p, shp, target.gui.displayArcs);
+      var p2 = target.gui.displayArcs.getVertex2(ids[0]);
       var dist = geom.distance2D(p[0], p[1], p2[0], p2[1]);
       var pixelDist = dist / ext.getPixelSize();
-      if (pixelDist > HOVER_THRESHOLD$1) {
+      if (pixelDist > HOVER_THRESHOLD) {
         return null;
       }
       var point = getVertexCoords(target, ids[0]); // data coordinates
       // find out if the vertex is the endpoint of a single path
       // (which could be extended by a newly drawn path)
       var extendable = ids.length == 1 &&
-        internal.vertexIsArcEndpoint(ids[0], target.arcs);
-      var displayPoint = target.arcs.getVertex2(ids[0]);
+        internal.vertexIsArcEndpoint(ids[0], target.gui.displayArcs);
+      var displayPoint = target.gui.displayArcs.getVertex2(ids[0]);
       return {target, ids, extendable, point, displayPoint, type: 'vertex'};
     }
 
+
     function findInterpolatedPoint(e) {
       var target = hit.getHitTarget();
       //// vertex insertion not supported with simplification
       // if (!target.arcs.isFlat()) return null;
       var p = ext.translatePixelCoords(e.x, e.y);
       var minDist = Infinity;
-      var shp = target.layer.shapes[e.id];
+      var shp = target.shapes[e.id];
       var closest;
-      internal.forEachSegmentInShape(shp, target.arcs, function(i, j, xx, yy) {
+      internal.forEachSegmentInShape(shp, target.gui.displayArcs, function(i, j, xx, yy) {
         var x1 = xx[i],
             y1 = yy[i],
             x2 = xx[j],
             y2 = yy[j],
-            // switching from midpoint to nearest point to the mouse
-            // cx = (x1 + x2) / 2,
-            // cy = (y1 + y2) / 2,
-            // p2 = [cx, cy],
             p2 = internal.findClosestPointOnSeg(p[0], p[1], x1, y1, x2, y2, 0),
             dist = geom.distance2D(p2[0], p2[1], p[0], p[1]);
         if (dist < minDist) {
@@ -10151,7 +10292,7 @@
         }
       });
 
-      if (closest.distance / ext.getPixelSize() > HOVER_THRESHOLD$1) {
+      if (closest.distance / ext.getPixelSize() > HOVER_THRESHOLD) {
         return null;
       }
       closest.point = translateDisplayPoint(target, closest.displayPoint);
@@ -10160,6 +10301,45 @@
       return closest;
     }
 
+    // Try to form polygon shapes from an array of path shapes
+    // shapes: array of all shapes that have been drawn in the current session
+    function convertClosedPaths(shapes) {
+      var target = hit.getHitTarget();
+      // try to convert paths to polygons
+      // NOTE: added "no_cuts" option to prevent polygons function from modifying
+      // arcs, which would break undo/redo and cause other problems
+      var tmpLyr = {
+        geometry_type: 'polyline',
+        shapes: shapes.concat()
+      };
+      var output = mapshaper.cmd.polygons([tmpLyr], target.gui.source.dataset, {no_cuts: true});
+      var closedShapes = output[0].shapes;
+
+      // find paths that were not convertible to polygons
+      var isOpenPath = getOpenPathTest(closedShapes);
+      var openShapes = shapes.filter(function(shp) { return isOpenPath(shp); });
+
+      // retain both converted polygons and unconverted polylines
+      return openShapes.concat(closedShapes);
+    }
+
+    // Returns a function for testing if a shape is an unclosed path, and doesn't
+    // overlap with an array of polygon shapes
+    function getOpenPathTest(polygonShapes) {
+      var polygonArcs = [];
+      internal.forEachArcId(polygonShapes, function(arcId) {
+        polygonArcs.push(internal.absArcId(arcId));
+      });
+
+      return function(shp) {
+        // assume that any compound shape is a polygon
+        return shapeHasOneFwdArc(shp) && !polygonArcs.includes(shp[0][0]);
+      };
+    }
+
+    function shapeHasOneFwdArc(shp) {
+      return shp.length == 1 && shp[0].length == 1 && shp[0][0] >= 0;
+    }
   }
 
   function initPointDrawing(gui, ext, hit) {
@@ -10182,8 +10362,7 @@
     // x, y: pixel coordinates
     function addPoint(x, y) {
       var p = ext.translatePixelCoords(x, y);
-      var target = hit.getHitTarget();
-      var lyr = target.layer;
+      var lyr = hit.getHitTarget();
       var fid = lyr.shapes.length;
       var d = appendNewDataRecord(lyr);
       if (d) {
@@ -10192,194 +10371,17 @@
         lyr.data.getRecords()[fid] = d;
       }
       lyr.shapes[fid] = [p];
-      updatePointCoords(target, fid);
+      updatePointCoords(lyr, fid);
     }
 
 
   }
 
-  // pointer thresholds for hovering near a vertex or segment midpoint
-  var HOVER_THRESHOLD = 8;
-  var MIDPOINT_THRESHOLD = 11;
-
-  function initVertexDragging(gui, ext, hit) {
-    var insertionPoint;
-    var dragInfo;
-
-    function active() {
-      return gui.interaction.getMode() == 'vertices';
-    }
-
-    function dragging() {
-      return active() && !!dragInfo;
-    }
-
-    function setHoverVertex(id) {
-      var target = hit.getHitTarget();
-      hit.setHoverVertex(target.arcs.getVertex2(id));
-    }
-
-    function clearHoverVertex() {
-      hit.clearHoverVertex();
-    }
-
-    // return data on the nearest vertex (or identical vertices) to the pointer
-    // (if within a distance threshold)
-    //
-    function findDraggableVertices(e) {
-      var target = hit.getHitTarget();
-      var shp = target.layer.shapes[e.id];
-      var p = ext.translatePixelCoords(e.x, e.y);
-      var nearestIds = internal.findNearestVertices(p, shp, target.arcs);
-      var p2 = target.arcs.getVertex2(nearestIds[0]);
-      var dist = geom.distance2D(p[0], p[1], p2[0], p2[1]);
-      var pixelDist = dist / ext.getPixelSize();
-      if (pixelDist > HOVER_THRESHOLD) {
-        return null;
-      }
-      var points = nearestIds.map(function(i) {
-        return getVertexCoords(target, i); // data coordinates
-      });
-      return {
-        target: target,
-        ids: nearestIds,
-        points: points
-      };
-    }
-
-    function findVertexInsertionPoint(e) {
-      var target = hit.getHitTarget();
-      //// vertex insertion not supported with simplification
-      // if (!target.arcs.isFlat()) return null;
-      var p = ext.translatePixelCoords(e.x, e.y);
-      var midpoint = findNearestMidpoint(p, e.id, target);
-      if (!midpoint ||
-          midpoint.distance / ext.getPixelSize() > MIDPOINT_THRESHOLD) return null;
-      return midpoint;
-    }
-
-    hit.on('dragstart', function(e) {
-      if (!active()) return;
-      if (insertionPoint) {
-        var target = hit.getHitTarget();
-        insertVertex$1(target, insertionPoint.i, insertionPoint.point);
-        dragInfo = {
-          target: target,
-          insertion: true,
-          ids: [insertionPoint.i],
-          points: [insertionPoint.point]
-        };
-        insertionPoint = null;
-      } else {
-        dragInfo = findDraggableVertices(e);
-      }
-      if (dragInfo) {
-        setHoverVertex(dragInfo.ids[0]);
-      }
-    });
-
-    hit.on('drag', function(e) {
-      if (!dragging()) return;
-      var target = hit.getHitTarget();
-      var p = ext.translatePixelCoords(e.x, e.y);
-      if (gui.keyboard.shiftIsPressed()) {
-        internal.snapPointToArcEndpoint(p, dragInfo.ids, target.arcs);
-      }
-      internal.snapVerticesToPoint(dragInfo.ids, p, target.arcs);
-      setHoverVertex(dragInfo.ids[0]);
-      // redrawing the whole map updates the data layer as well as the overlay layer
-      // gui.dispatchEvent('map-needs-refresh');
-    });
-
-    hit.on('dragend', function(e) {
-      if (!dragging()) return;
-      // kludge to get dataset to recalculate internal bounding boxes
-      hit.getHitTarget().arcs.transformPoints(function() {});
-      clearHoverVertex();
-      updateVertexCoords(dragInfo.target, dragInfo.ids);
-      gui.dispatchEvent('vertex_dragend', dragInfo);
-      gui.dispatchEvent('map-needs-refresh');
-      dragInfo = null;
-    });
-
-    hit.on('dblclick', function(e) {
-      if (!active()) return;
-      var info = findDraggableVertices(e); // same selection criteria as for dragging
-      if (!info) return;
-      var target = hit.getHitTarget();
-      var vId = info.ids[0];
-      if (internal.vertexIsArcStart(vId, target.arcs) ||
-          internal.vertexIsArcEnd(vId, target.arcs)) {
-        // TODO: support removing arc endpoints
-        return;
-      }
-      gui.dispatchEvent('vertex_delete', {
-        target: target,
-        vertex_id: vId
-      });
-      deleteVertex$1(target, vId);
-      clearHoverVertex();
-      gui.dispatchEvent('map-needs-refresh');
-    });
-
-    // highlight hit vertex in path edit mode
-    hit.on('hover', function(e) {
-      insertionPoint = null;
-      if (!active() || dragging()) return; // no hover effect while dragging
-      var info = findDraggableVertices(e);
-      if (info) {
-        // hovering near a vertex: highlight the vertex
-        setHoverVertex(info.ids[0]);
-        return;
-      }
-      // if hovering near a segment midpoint: show the midpoint and save midpoint info
-      insertionPoint = findVertexInsertionPoint(e);
-      if (insertionPoint) {
-        hit.setHoverVertex(insertionPoint.displayPoint);
-      } else {
-        // pointer is not over a vertex: clear any hover effect
-        clearHoverVertex();
-      }
-    }, null, 100);
-  }
-
-
-  // Given a location @p (e.g. corresponding to the mouse pointer location),
-  // find the midpoint of two vertices on @shp suitable for inserting a new vertex
-  function findNearestMidpoint(p, fid, target) {
-    var arcs = target.arcs;
-    var shp = target.layer.shapes[fid];
-    var minDist = Infinity, v;
-    internal.forEachSegmentInShape(shp, arcs, function(i, j, xx, yy) {
-      var x1 = xx[i],
-          y1 = yy[i],
-          x2 = xx[j],
-          y2 = yy[j],
-          cx = (x1 + x2) / 2,
-          cy = (y1 + y2) / 2,
-          midpoint = [cx, cy],
-          dist = geom.distance2D(cx, cy, p[0], p[1]);
-      if (dist < minDist) {
-        minDist = dist;
-        v = {
-          i: (i < j ? i : j) + 1, // insertion point
-          segment: [i, j],
-          segmentLen: geom.distance2D(x1, y1, x2, y2),
-          displayPoint: midpoint,
-          point: translateDisplayPoint(target, midpoint),
-          distance: dist
-        };
-      }
-    });
-    return v || null;
-  }
-
   function initInteractiveEditing(gui, ext, hit) {
     initLabelDragging(gui, ext, hit);
     initPointDragging(gui, ext, hit);
     initPointDrawing(gui, ext, hit);
     initLineEditing(gui, ext, hit);
-    initVertexDragging(gui, ext, hit);
   }
 
   function MapExtent(_position) {
@@ -10753,47 +10755,46 @@
   }
 
   // TODO: consider moving this upstream
-  function getArcsForRendering(obj, ext) {
-    var dataset = obj.source.dataset;
+  function getArcsForRendering(lyr, ext) {
+    var dataset = lyr.gui.source.dataset;
     var sourceArcs = dataset.arcs;
-    if (obj.geographic && dataset.displayArcs) {
-      return dataset.displayArcs.getScaledArcs(ext);
+    if (lyr.gui.geographic && dataset.gui?.displayArcs) {
+      return dataset.gui.displayArcs.getScaledArcs(ext);
     }
-    return obj.arcs;
+    return lyr.gui.displayArcs;
   }
 
-  function drawOutlineLayerToCanvas(obj, canv, ext) {
+  function drawOutlineLayerToCanvas(lyr, canv, ext) {
     var arcs;
-    var style = obj.style;
+    var style = lyr.gui.style;
+    var arcCounts = lyr.gui.arcCounts;
     var darkStyle = {strokeWidth: style.strokeWidth, strokeColor: style.strokeColors[1]},
         lightStyle = {strokeWidth: style.strokeWidth, strokeColor: style.strokeColors[0]};
     var filter;
-    if (internal.layerHasPaths(obj.layer)) {
-      if (!obj.arcCounts) {
-        obj.arcCounts = new Uint8Array(obj.arcs.size());
-        internal.countArcsInShapes(obj.layer.shapes, obj.arcCounts);
+    if (internal.layerHasPaths(lyr.gui.displayLayer)) {
+      if (!arcCounts) {
+        arcCounts = lyr.gui.arcCounts = new Uint8Array(lyr.gui.displayArcs.size());
+        internal.countArcsInShapes(lyr.gui.displayLayer.shapes, arcCounts);
       }
-      if (obj.arcCounts) {
-        arcs = getArcsForRendering(obj, ext);
-        if (lightStyle.strokeColor) {
-          filter = getArcFilter(arcs, ext, false, obj.arcCounts);
-          canv.drawArcs(arcs, lightStyle, filter);
-        }
-        if (darkStyle.strokeColor && obj.layer.geometry_type != 'point') {
-          filter = getArcFilter(arcs, ext, true, obj.arcCounts);
-          canv.drawArcs(arcs, darkStyle, filter);
-        }
+      arcs = getArcsForRendering(lyr, ext);
+      if (lightStyle.strokeColor) {
+        filter = getArcFilter(arcs, ext, false, arcCounts);
+        canv.drawArcs(arcs, lightStyle, filter);
+      }
+      if (darkStyle.strokeColor && lyr.gui.displayLayer.geometry_type != 'point') {
+        filter = getArcFilter(arcs, ext, true, arcCounts);
+        canv.drawArcs(arcs, darkStyle, filter);
       }
     }
-    if (obj.layer.geometry_type == 'point') {
-      canv.drawSquareDots(obj.layer.shapes, style);
+    if (lyr.gui.displayLayer.geometry_type == 'point') {
+      canv.drawSquareDots(lyr.gui.displayLayer.shapes, style);
     }
   }
 
-  function drawStyledLayerToCanvas(obj, canv, ext) {
+  function drawStyledLayerToCanvas(lyr, canv, ext) {
     // TODO: add filter for out-of-view shapes
-    var style = obj.style;
-    var layer = obj.layer;
+    var style = lyr.gui.style;
+    var layer = lyr.gui.displayLayer;
     var arcs, filter;
     if (layer.geometry_type == 'point') {
       if (style.type == 'styled') {
@@ -10802,7 +10803,7 @@
         canv.drawSquareDots(layer.shapes, style);
       }
     } else {
-      arcs = getArcsForRendering(obj, ext);
+      arcs = getArcsForRendering(lyr, ext);
       filter = getShapeFilter(arcs, ext);
       canv.drawStyledPaths(layer.shapes, arcs, style, filter);
       if (style.vertices) {
@@ -11396,12 +11397,12 @@
       }
     };
 
-    el.reposition = function(target, type) {
+    el.reposition = function(lyr, type) {
       resize(ext);
-      reposition(target, type, ext);
+      reposition(lyr, type, ext);
     };
 
-    el.drawLayer = function(target, type) {
+    el.drawLayer = function(lyr, type) {
       var g = document.createElementNS('http://www.w3.org/2000/svg', 'g');
       var html = '';
       // generate a unique id so layer can be identified when symbols are repositioned
@@ -11409,24 +11410,25 @@
       var id = utils$1.getUniqueName();
       var classNames = [id, 'mapshaper-svg-layer', 'mapshaper-' + type + '-layer'];
       g.setAttribute('class', classNames.join(' '));
-      target.svg_id = id;
+      lyr.gui.svg_id = id;
+      lyr.gui.svg_container = g;
       resize(ext);
       if (type == 'label' || type == 'symbol') {
-        html = renderSymbols(target.layer, ext);
+        html = renderSymbols(lyr.gui.displayLayer, ext);
       } else if (type == 'furniture') {
-        html = renderFurniture(target.layer, ext);
+        html = renderFurniture(lyr.gui.displayLayer, ext);
       }
       g.innerHTML = html;
       svg.append(g);
 
       // prevent svg hit detection on inactive layers
-      if (!target.active) {
+      if (!lyr.active) {
         g.style.pointerEvents = 'none';
       }
     };
 
-    function reposition(target, type, ext) {
-      var container = el.findChild('.' + target.svg_id);
+    function reposition(lyr, type, ext) {
+      var container = el.findChild('.' + lyr.gui.svg_id);
       if (!container || !container.node()) {
         console.error('[reposition] missing SVG container');
         return;
@@ -11434,9 +11436,9 @@
       var elements;
       if (type == 'symbol') {
         elements = El.findAll('.mapshaper-svg-symbol', container.node());
-        repositionSymbols(elements, target.layer, ext);
+        repositionSymbols(elements, lyr.gui.displayLayer, ext);
       } else if (type == 'furniture') {
-        repositionFurniture(container.node(), target.layer, ext);
+        repositionFurniture(container.node(), lyr.gui.displayLayer, ext);
       } else {
         // container.getElementsByTagName('text')
         error('Unsupported symbol type:', type);
@@ -11470,7 +11472,7 @@
         _svg.clear();
       }
       layers.forEach(function(lyr) {
-        var isSvgLayer = internal.layerHasSvgSymbols(lyr.layer) || internal.layerHasLabels(lyr.layer);
+        var isSvgLayer = internal.layerHasSvgSymbols(lyr) || internal.layerHasLabels(lyr);
         if (isSvgLayer && !needSvgRedraw) {
           _svg.reposition(lyr, 'symbol');
         } else if (isSvgLayer) {
@@ -11519,7 +11521,7 @@
 
     function drawCanvasLayer(lyr, canv) {
       if (!lyr) return;
-      if (lyr.style.type == 'outline') {
+      if (lyr.gui.style.type == 'outline') {
         drawOutlineLayerToCanvas(lyr, canv, ext);
       } else {
         drawStyledLayerToCanvas(lyr, canv, ext);
@@ -11587,8 +11589,8 @@
     gui.on('interaction_mode_change', function(e) {
       if (e.mode === 'box') {
         gui.enterMode('box_tool');
-      } else if (gui.getMode() == 'box_tool') {
-        gui.clearMode();
+      } else if (_on) {
+        turnOff();
       }
     });
 
@@ -11691,8 +11693,8 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     gui.on('interaction_mode_change', function(e) {
       if (e.mode === 'rectangles') {
         gui.enterMode('rectangle_tool');
-      } else if (gui.getMode() == 'rectangle_tool') {
-        gui.clearMode();
+      } else if (_on) {
+        turnOff();
       }
     });
 
@@ -11702,8 +11704,8 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var id = e.id;
       if (e.id > -1 && e.pinned) {
         var target = hit.getHitTarget();
-        var path = target.layer.shapes[e.id][0];
-        var bbox = target.arcs.getSimpleShapeBounds(path).toArray();
+        var path = target.shapes[e.id][0];
+        var bbox = target.gui.displayArcs.getSimpleShapeBounds(path).toArray();
         box.setDataCoords(bbox);
         dragInfo = {
           id: e.id,
@@ -11711,7 +11713,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
           ids: [],
           points: []
         };
-        var iter = target.arcs.getShapeIter(path);
+        var iter = target.gui.displayArcs.getShapeIter(path);
         while (iter.hasNext()) {
           dragInfo.points.push([iter.x, iter.y]);
           dragInfo.ids.push(iter._arc.i);
@@ -11810,8 +11812,9 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     };
   }
 
-  function getDisplayLayerForTable(table) {
-    var n = table.size(),
+  function getDisplayLayerForTable(tableArg) {
+    var table = tableArg || new internal.DataTable(0),
+        n = table.size(),
         cellWidth = 12,
         cellHeight = 5,
         gutter = 6,
@@ -11869,54 +11872,45 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     };
   }
 
+  // lyr: a map layer with gui property
   // displayCRS: CRS to use for display, or null (which clears any current display CRS)
-  function projectDisplayLayer(lyr, displayCRS) {
-    var crsInfo = getDatasetCrsInfo(lyr.source.dataset);
-    var sourceCRS = crsInfo.crs;
-    var lyr2;
-    //if (!lyr.geographic || !sourceCRS) {
-    // let getDisplayLayer() handle case of unprojectable source
-    if (!lyr.geographic) {
-      return lyr;
-    }
-    if (lyr.dynamic_crs && internal.crsAreEqual(sourceCRS, lyr.dynamic_crs)) {
-      return lyr;
-    }
-    lyr2 = getDisplayLayer(lyr.source.layer, lyr.source.dataset, {crs: displayCRS});
-    // kludge: copy projection-related properties to original layer
-    lyr.dynamic_crs = lyr2.dynamic_crs;
-    lyr.layer = lyr2.layer;
-
-    if (lyr.style && lyr.style.ids) {
+  function projectLayerForDisplay(lyr, displayCRS) {
+    var crsInfo = getDatasetCrsInfo(lyr.gui.source.dataset);
+    var sourceCRS = crsInfo.crs || null; // let enhanceLayerForDisplay() handle null case
+    if (!lyr.gui.geographic) {
+      return;
+    }
+    if (lyr.gui.dynamic_crs && internal.crsAreEqual(sourceCRS, lyr.gui.dynamic_crs)) {
+      return;
+    }
+    enhanceLayerForDisplay(lyr, lyr.gui.source.dataset, {crs: displayCRS});
+    if (lyr.gui.style?.ids) {
       // re-apply layer filter
-      lyr.layer = filterLayerByIds(lyr.layer, lyr.style.ids);
+      lyr.gui.displayLayer = filterLayerByIds(lyr.gui.displayLayer, lyr.gui.style.ids);
     }
-    lyr.invertPoint = lyr2.invertPoint;
-    lyr.projectPoint = lyr2.projectPoint;
-    lyr.bounds = lyr2.bounds;
-    lyr.arcs = lyr2.arcs;
   }
 
 
-  // Wrap a layer in an object along with information needed for rendering
-  function getDisplayLayer(layer, dataset, opts) {
-    var obj = {
-      layer: null,
-      arcs: null,
-      // display_arcs: null,
+  // Supplement a layer with information needed for rendering
+  function enhanceLayerForDisplay(layer, dataset, opts) {
+    var gui = {
+      empty: internal.getFeatureCount(layer) === 0,
+      geographic: false,
+      displayArcs: null,
+      displayLayer: null,
+      source: {dataset},
+      bounds: null,
       style: null,
+      dynamic_crs: null,
       invertPoint: null,
-      projectPoint: null,
-      source: {
-        layer: layer,
-        dataset: dataset
-      },
-      empty: internal.getFeatureCount(layer) === 0
+      projectPoint: null
     };
 
     var displayCRS = opts.crs || null;
-    // display arcs may have been generated when another layer in the dataset was converted for display... re-use if available
-    var displayArcs = dataset.displayArcs || null;
+    // display arcs may have been generated when another layer in the dataset
+    // was converted for display... re-use if available
+    var displayArcs = dataset.gui?.displayArcs;
+    var unprojectable = false;
     var sourceCRS;
     var emptyArcs;
 
@@ -11924,8 +11918,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var crsInfo = getDatasetCrsInfo(dataset);
       if (crsInfo.error) {
         // unprojectable dataset -- return empty layer
-        obj.empty = true;
-        obj.geographic = true;
+        gui.unprojectable = true;
       } else {
         sourceCRS = crsInfo.crs;
       }
@@ -11933,7 +11926,8 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
 
     // Assume that dataset.displayArcs is in the display CRS
     // (it must be deleted upstream if reprojection is needed)
-    if (!obj.empty && dataset.arcs && !displayArcs) {
+    // if (!obj.empty && dataset.arcs && !displayArcs) {
+    if (dataset.arcs && !displayArcs && !gui.unprojectable) {
       // project arcs, if needed
       if (needReprojectionForDisplay(sourceCRS, displayCRS)) {
         displayArcs = projectArcsForDisplay(dataset.arcs, sourceCRS, displayCRS);
@@ -11943,50 +11937,46 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       }
 
       enhanceArcCollectionForDisplay(displayArcs);
-      dataset.displayArcs = displayArcs; // stash these in the dataset for other layers to use
+      dataset.gui = {displayArcs}; // stash these in the dataset for other layers to use
     }
 
     if (internal.layerHasFurniture(layer)) {
-      obj.layer = layer;
-      obj.tabular = true;
       // TODO: consider how to render furniture in GUI
-      // obj.furniture = true;
-      // obj.furniture_type = internal.getFurnitureLayerType(layer);
       // treating furniture layers (other than frame) as tabular for now,
       // so there is something to show if they are selected
-      // obj.tabular = obj.furniture_type != 'frame';
-    } else if (obj.empty) {
-      obj.layer = {shapes: []}; // ideally we should avoid empty layers
-    } else if (!layer.geometry_type) {
-      obj.tabular = true;
-    } else {
-      obj.geographic = true;
-      obj.layer = layer;
-      obj.arcs = displayArcs;
     }
 
-    if (obj.tabular) {
-      utils$1.extend(obj, getDisplayLayerForTable(layer.data));
+    if (gui.unprojectable) {
+      gui.displayLayer = {shapes: []}; // TODO: improve
+    } else if (layer.geometry_type) {
+      gui.geographic = true;
+      gui.displayLayer = layer;
+      gui.displayArcs = displayArcs;
+    } else {
+      var table = getDisplayLayerForTable(layer.data);
+      gui.tabular = true;
+      gui.displayLayer = table.layer;
+      gui.displayArcs = table.arcs;
     }
 
     // dynamic reprojection (arcs were already reprojected above)
-    if (obj.geographic && needReprojectionForDisplay(sourceCRS, displayCRS)) {
-      obj.dynamic_crs = displayCRS;
-      obj.invertPoint = internal.getProjTransform2(displayCRS, sourceCRS);
-      obj.projectPoint = internal.getProjTransform2(sourceCRS, displayCRS);
+    if (gui.geographic && needReprojectionForDisplay(sourceCRS, displayCRS)) {
+      gui.dynamic_crs = displayCRS;
+      gui.invertPoint = internal.getProjTransform2(displayCRS, sourceCRS);
+      gui.projectPoint = internal.getProjTransform2(sourceCRS, displayCRS);
       if (internal.layerHasPoints(layer)) {
-        obj.layer = projectPointsForDisplay(layer, sourceCRS, displayCRS);
+        gui.displayLayer = projectPointsForDisplay(layer, sourceCRS, displayCRS);
       } else if (internal.layerHasPaths(layer)) {
         emptyArcs = findEmptyArcs(displayArcs);
         if (emptyArcs.length > 0) {
           // Don't try to draw paths containing coordinates that failed to project
-          obj.layer = internal.filterPathLayerByArcIds(obj.layer, emptyArcs);
+          gui.displayLayer = internal.filterPathLayerByArcIds(gui.displayLayer, emptyArcs);
         }
       }
     }
 
-    obj.bounds = getDisplayBounds(obj.layer, obj.arcs);
-    return obj;
+    gui.bounds = getDisplayBounds(gui.displayLayer, gui.displayArcs);
+    layer.gui = gui;
   }
 
 
@@ -12300,13 +12290,19 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
 
     model.on('update', onUpdate);
 
+    document.addEventListener('visibilitychange', function(e) {
+      // refresh map when browser tab is re-activated (Chrome on mac has been
+      // blanking the canvas after several other tabs are visited)
+      if (document.visibilityState == 'visible') drawLayers();
+    });
 
     // Update display of segment intersections
     this.setIntersectionLayer = function(lyr, dataset) {
       if (lyr == _intersectionLyr) return; // no change
       if (lyr) {
-        _intersectionLyr = getDisplayLayer(lyr, dataset, getDisplayOptions());
-        _intersectionLyr.style = getIntersectionStyle(_intersectionLyr.layer, getGlobalStyleOptions());
+        enhanceLayerForDisplay(lyr, dataset, getDisplayOptions());
+        _intersectionLyr = lyr;
+        _intersectionLyr.gui.style = getIntersectionStyle(_intersectionLyr.gui.displayLayer, getGlobalStyleOptions());
       } else {
         _intersectionLyr = null;
       }
@@ -12335,9 +12331,9 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     };
 
     this.getDisplayCRS = function() {
-      if (!_activeLyr || !_activeLyr.geographic) return null;
-      if (_activeLyr.dynamic_crs) return _activeLyr.dynamic_crs;
-      var info = getDatasetCrsInfo(_activeLyr.source.dataset);
+      if (!_activeLyr || !_activeLyr.gui.geographic) return null;
+      if (_activeLyr.gui.dynamic_crs) return _activeLyr.gui.dynamic_crs;
+      var info = getDatasetCrsInfo(_activeLyr.gui.source.dataset);
       return info.crs || null;
     };
 
@@ -12376,7 +12372,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
 
       // Reproject all visible map layers
       getContentLayers().forEach(function(lyr) {
-        projectDisplayLayer(lyr, newCRS);
+        projectLayerForDisplay(lyr, newCRS);
       });
 
       // kludge to make sure all layers have styles
@@ -12387,13 +12383,13 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       updateFullBounds();
     };
 
-    function getGlobalStyleOptions() {
+    function getGlobalStyleOptions(opts) {
       var mode = gui.state.interaction_mode;
-      return {
+      return Object.assign({
         darkMode: !!gui.state.dark_basemap,
         outlineMode: mode == 'vertices',
         interactionMode: mode
-      };
+      }, opts);
     }
 
     // Refresh map display in response to data changes, layer selection, etc.
@@ -12401,14 +12397,13 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var prevLyr = _activeLyr || null;
       var fullBounds;
       var needReset;
-
       if (!prevLyr) {
         initMap(); // first call
       }
 
       if (arcsMayHaveChanged(e.flags)) {
         // regenerate filtered arcs the next time they are needed for rendering
-        // delete e.dataset.displayArcs;
+        // delete e.dataset.gui.displayArcs
         clearAllDisplayArcs();
 
         // reset simplification after projection (thresholds have changed)
@@ -12427,8 +12422,9 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
         return;
       }
 
-      _activeLyr = getDisplayLayer(e.layer, e.dataset, getDisplayOptions());
-      _activeLyr.style = getActiveLayerStyle(_activeLyr.layer, getGlobalStyleOptions());
+      enhanceLayerForDisplay(e.layer, e.dataset, getDisplayOptions());
+      _activeLyr = e.layer;
+      _activeLyr.gui.style = getActiveLayerStyle(_activeLyr.gui.displayLayer, getGlobalStyleOptions());
       _activeLyr.active = true;
 
       if (popupCanStayOpen(e.flags)) {
@@ -12442,7 +12438,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       updateVisibleMapLayers();
       fullBounds = calcFullBounds();
 
-      if (!prevLyr || prevLyr.tabular || _activeLyr.tabular) {
+      if (!prevLyr || prevLyr.gui.tabular || _activeLyr.gui.tabular) {
         needReset = true;
       } else {
         needReset = mapNeedsReset(fullBounds, _ext.getFullBounds(), _ext.getBounds(), e.flags);
@@ -12485,12 +12481,11 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     }
 
     function updateOverlayLayer(e) {
-      var style = getOverlayStyle(_activeLyr.layer, e, getGlobalStyleOptions());
+      var style = getOverlayStyle(_activeLyr.gui.displayLayer, e, getGlobalStyleOptions());
       if (style) {
-        _overlayLyr = utils$1.defaults({
-          layer: filterLayerByIds(_activeLyr.layer, style.ids),
-          style: style
-        }, _activeLyr);
+        var displayLayer = filterLayerByIds(_activeLyr.gui.displayLayer, style.ids);
+        var gui = Object.assign({}, _activeLyr.gui, {style, displayLayer});
+        _overlayLyr = utils$1.defaults({gui}, _activeLyr);
       } else {
         _overlayLyr = null;
       }
@@ -12521,7 +12516,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var b = new Bounds();
       var layers = getContentLayers();
       layers.forEach(function(lyr) {
-        b.mergeBounds(lyr.bounds);
+        b.mergeBounds(lyr.gui.bounds);
       });
 
       if (!b.hasBounds()) {
@@ -12544,7 +12539,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
       var widthPx = _ext.width();
       var marginPct = widthPx < 700 && 3.5 || widthPx < 800 && 3 || 2.5;
       if (isTableView()) {
-        var n = internal.getFeatureCount(_activeLyr.layer);
+        var n = internal.getFeatureCount(_activeLyr);
         marginPct = n < 5 && 20 || n < 100 && 10 || 4;
       }
       b.scale(1 + marginPct / 100 * 2);
@@ -12556,7 +12551,7 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     }
 
     function isActiveLayer(lyr) {
-      return _activeLyr && lyr == _activeLyr.source.layer || false;
+      return _activeLyr && lyr == _activeLyr || false;
     }
 
     function isVisibleLayer(lyr) {
@@ -12564,12 +12559,12 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     }
 
     function isTableView() {
-      return !!_activeLyr.tabular;
+      return !!_activeLyr.gui.tabular;
     }
 
     function findFrameLayer() {
       return getVisibleMapLayers().find(function(lyr) {
-        return internal.isFrameLayer(lyr.layer, lyr.arcs);
+        return internal.isFrameLayer(lyr.gui.displayLayer, lyr.gui.displayArcs);
       });
     }
 
@@ -12580,12 +12575,15 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
 
     function getFrameData() {
       var lyr = findFrameLayer();
-      return lyr && internal.getFrameLayerData(lyr.layer, lyr.arcs) || null;
+      return lyr && internal.getFrameLayerData(lyr, lyr.gui.displayArcs) || null;
     }
 
     function clearAllDisplayArcs() {
+      model.forEachLayer(function(lyr) {
+        if (lyr.gui) delete lyr.gui.arcCounts;
+      });
       model.getDatasets().forEach(function(o) {
-        delete o.displayArcs;
+        delete o.gui;
       });
     }
 
@@ -12596,7 +12594,8 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
         if (isActiveLayer(o.layer)) {
           layers.push(_activeLyr);
         } else if (!isTableView()) {
-          layers.push(getDisplayLayer(o.layer, o.dataset, getDisplayOptions()));
+          enhanceLayerForDisplay(o.layer, o.dataset, getDisplayOptions());
+          layers.push(o.layer);
         }
       });
       _visibleLayers = layers;
@@ -12618,13 +12617,13 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
         return findActiveLayer(layers);
       }
       return layers.filter(function(o) {
-        return !!o.geographic;
+        return !!o.gui.geographic;
       });
     }
 
     function getDrawableContentLayers() {
       return getContentLayers().filter(function(lyr) {
-        if (isActiveLayer(lyr.layer) && lyr.layer.hidden) return false;
+        if (isActiveLayer(lyr) && lyr.hidden) return false;
         return true;
       });
     }
@@ -12638,31 +12637,33 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
 
     function updateLayerStyles(layers) {
       layers.forEach(function(mapLayer, i) {
+        var style;
         if (mapLayer.active) {
           // regenerating active style everytime, to support style change when
           // switching between outline and preview modes.
-          mapLayer.style = getActiveLayerStyle(mapLayer.layer, getGlobalStyleOptions());
-          if (mapLayer.style.type != 'styled' && layers.length > 1 && mapLayer.style.strokeColors) {
+          style = getActiveLayerStyle(mapLayer.gui.displayLayer, getGlobalStyleOptions());
+          if (style.type != 'styled' && layers.length > 1 && style.strokeColors) {
             // kludge to hide ghosted layers when reference layers are present
             // TODO: consider never showing ghosted layers (which appear after
             // commands like dissolve and filter).
-            mapLayer.style = utils$1.defaults({
-              strokeColors: [null, mapLayer.style.strokeColors[1]]
-            }, mapLayer.style);
+            style = utils$1.defaults({
+              strokeColors: [null, style.strokeColors[1]]
+            }, style);
           }
         } else {
-          if (mapLayer.layer == _activeLyr.layer) {
+          if (mapLayer == _activeLyr) {
             console.error("Error: shared map layer");
           }
-          mapLayer.style = getReferenceLayerStyle(mapLayer.layer, getGlobalStyleOptions());
+          style = getReferenceLayerStyle(mapLayer.gui.displayLayer, getGlobalStyleOptions());
         }
+        mapLayer.gui.style = style;
       });
     }
 
     function sortMapLayers(layers) {
       layers.sort(function(a, b) {
         // assume that each layer has a menu_order (assigned by updateLayerStackOrder())
-        return a.source.layer.menu_order - b.source.layer.menu_order;
+        return a.menu_order - b.menu_order;
       });
     }
 
@@ -12742,6 +12743,99 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     });
   }
 
+  function ContextMenu() {
+    var body = document.querySelector('body');
+    var menu = El('div').addClass('contextmenu rollover').appendTo(body);
+    var _open = false;
+    var _openCount = 0;
+    document.addEventListener('mousedown', close);
+
+    this.isOpen = function() {
+      return _open;
+    };
+
+    function close() {
+      var count = _openCount;
+      if (!_open) return;
+      setTimeout(function() {
+        if (count == _openCount) {
+          menu.hide();
+          _open = false;
+        }
+      }, 300);
+    }
+
+    function addMenuItem(label, func) {
+      El('div')
+        .appendTo(menu)
+        .addClass('contextmenu-item')
+        .text(label)
+        .on('click', func)
+        .show();
+    }
+
+    this.open = function(e, lyr) {
+      _open = true;
+      _openCount++;
+      var rspace = body.clientWidth - e.pageX;
+      var xoffs = 10;
+      menu.empty().show();
+      if (rspace > 150) {
+        menu.css('left', e.pageX + xoffs + 'px');
+      } else {
+        menu.css('right', (body.clientWidth - e.pageX + xoffs) + 'px');
+      }
+      menu.css('top', (e.pageY - 15) + 'px');
+
+      // menu contents
+      if (e.deleteVertex) {
+        addMenuItem('Delete vertex', e.deleteVertex);
+      }
+      if (e.coordinates) {
+        addCopyCoords();
+      }
+      if (e.ids?.length) {
+        addMenuItem('Copy as GeoJSON', copyGeoJSON);
+      }
+
+      function addCopyCoords() {
+        var bbox = internal.getLayerBounds(lyr, lyr.gui.source.dataset.arcs).toArray();
+        var p = internal.getRoundedCoords(e.coordinates, internal.getBoundsPrecisionForDisplay(bbox));
+        var coordStr = p.join(',');
+        addMenuItem(coordStr, function() {
+          saveFileContentToClipboard(coordStr);
+        });
+      }
+
+      function copyGeoJSON() {
+        var opts = {
+          no_replace: true,
+          ids: e.ids,
+          quiet: true
+        };
+        var dataset = lyr.gui.source.dataset;
+        var layer = mapshaper.cmd.filterFeatures(lyr, dataset.arcs, opts);
+        // the drawing tool can send open paths with 'polygon' geometry type,
+        // should be changed to 'polyline'
+        if (layerHasOpenPaths(layer, dataset.arcs)) {
+          layer.geometry_type = 'polyline';
+        }
+        var features = internal.exportLayerAsGeoJSON(layer, dataset, {rfc7946: true, prettify: true}, true, 'string');
+        var str = internal.geojson.formatCollection({"type": "FeatureCollection"}, features);
+        saveFileContentToClipboard(str);
+      }
+    };
+  }
+
+
+  function layerHasOpenPaths(layer, arcs) {
+    var retn = false;
+    internal.editShapes(layer.shapes, function(part) {
+      if (!geom.pathIsClosed(part, arcs)) retn = true;
+    });
+    return retn;
+  }
+
   function GuiInstance(container, opts) {
     var gui = new ModeSwitcher();
     opts = utils$1.extend({
@@ -12760,15 +12854,15 @@ GUI and setting the size and crop of SVG output.</p><div><input type="text" clas
     gui.model = new Model(gui);
     gui.keyboard = new KeyboardEvents(gui);
     gui.buttons = new SidebarButtons(gui);
-    gui.map = new MshpMap(gui);
-    gui.interaction = new InteractionMode(gui);
     gui.session = new SessionHistory(gui);
+    gui.contextMenu = new ContextMenu();
     gui.undo = new Undo(gui);
-    gui.state = {};
-
+    gui.map = new MshpMap(gui);
     if (opts.saveControl) {
       new SessionSnapshots(gui);
     }
+    gui.interaction = new InteractionMode(gui);
+    gui.state = {};
 
     var msgCount = 0;
     var clearMsg;