mapshaper 0.6.45 → 0.6.46

package/mapshaper.js

@@ -1,6 +1,6 @@
 (function () {
 
-  var VERSION = "0.6.45";
+  var VERSION = "0.6.46";
 
 
   var utils = /*#__PURE__*/Object.freeze({
@@ -1813,6 +1813,23 @@
     return [xmin, ymin, xmax, ymax];
   }
 
+
+  function findArcIdFromVertexId(i, ii) {
+    // binary search
+    // possible optimization: use interpolation to find a better partition value.
+    var lower = 0, upper = ii.length - 1;
+    var middle;
+    while (lower < upper) {
+      middle = Math.ceil((lower + upper) / 2);
+      if (i < ii[middle]) {
+        upper = middle - 1;
+      } else {
+        lower = middle;
+      }
+    }
+    return lower; // assumes dataset is not empty
+  }
+
   function deleteVertex(arcs, i) {
     var data = arcs.getVertexData();
     var nn = data.nn;
@@ -1939,6 +1956,7 @@
     __proto__: null,
     absArcId: absArcId,
     calcArcBounds: calcArcBounds,
+    findArcIdFromVertexId: findArcIdFromVertexId,
     deleteVertex: deleteVertex,
     insertVertex: insertVertex,
     countFilteredVertices: countFilteredVertices,
@@ -3405,7 +3423,7 @@
     var item;
     for (var i=0; i<arr.length; i++) {
       item = arr[i];
-      if (item instanceof Array) {
+      if (Array.isArray(item)) {
         forEachArcId(item, cb);
       } else if (utils.isInteger(item)) {
         var val = cb(item);
@@ -6953,6 +6971,15 @@
     utils.defaults(destInfo, srcInfo);
   }
 
+  function copyDatasetInfo(info) {
+    // not a deep copy... objects like info.crs are read-only, so copy-by-reference
+    // should be ok
+    var info2 = Object.assign({}, info);
+    if (Array.isArray(info.input_files)) {
+      info2.input_files = info.input_files.concat();
+    }
+    return info2;
+  }
 
   function splitApartLayers(dataset, layers) {
     var datasets = [];
@@ -7119,6 +7146,7 @@
     __proto__: null,
     splitDataset: splitDataset,
     mergeDatasetInfo: mergeDatasetInfo,
+    copyDatasetInfo: copyDatasetInfo,
     splitApartLayers: splitApartLayers,
     copyDataset: copyDataset,
     copyDatasetForExport: copyDatasetForExport,
@@ -15481,6 +15509,17 @@
 
   // Keep track of whether positive or negative integer ids are 'used' or not.
 
+
+  function SimpleIdTestIndex(n) {
+    var index = new Uint8Array(n);
+    this.setId = function(id) {
+      index[id] = 1;
+    };
+    this.hasId = function(id) {
+      return index[id] === 1;
+    };
+  }
+
   function IdTestIndex(n) {
     var index = new Uint8Array(n);
     var setList = [];
@@ -15743,15 +15782,30 @@
     getHoleDivider: getHoleDivider
   });
 
-  // Convert an array of intersections into an ArcCollection (for display)
-  //
-
   function getIntersectionPoints(intersections) {
     return intersections.map(function(obj) {
           return [obj.x, obj.y];
         });
   }
 
+  function getIntersectionLayer(intersections, lyr, arcs) {
+    // return {geometry_type: 'point', shapes: [getIntersectionPoints(XX)]};
+    var ii = arcs.getVertexData().ii;
+    var index = new SimpleIdTestIndex(arcs.size());
+    forEachArcId(lyr.shapes, arcId => {
+      index.setId(absArcId(arcId));
+    });
+    var points = [];
+    intersections.forEach(obj => {
+      var arc1 = findArcIdFromVertexId(obj.a[0], ii);
+      var arc2 = findArcIdFromVertexId(obj.b[0], ii);
+      if (index.hasId(arc1) && index.hasId(arc2)) {
+        points.push([obj.x, obj.y]);
+      }
+    });
+    return {geometry_type: 'point', shapes: [points]};
+  }
+
   // Identify intersecting segments in an ArcCollection
   //
   // To find all intersections:
@@ -15999,6 +16053,7 @@
   var SegmentIntersection = /*#__PURE__*/Object.freeze({
     __proto__: null,
     getIntersectionPoints: getIntersectionPoints,
+    getIntersectionLayer: getIntersectionLayer,
     findSegmentIntersections: findSegmentIntersections,
     sortIntersections: sortIntersections,
     dedupIntersections: dedupIntersections,
@@ -33396,7 +33451,7 @@ ${svg}
     if (n != values.length) {
       stop('Mismatch in number of categories and number of values');
     }
-    return values;
+    return parseValues(values); // convert numerical strings to numbers
   }
 
   function getIndexes(n) {
@@ -33564,7 +33619,7 @@ ${svg}
     if (opts.index_field) {
       dataField = opts.index_field;
       fieldType = getColumnType(opts.field, records);
-   } else if (opts.field) {
+    } else if (opts.field) {
       dataField = opts.field;
       fieldType = getColumnType(opts.field, records);
     }
@@ -33596,6 +33651,9 @@ ${svg}
       if ((!opts.categories || opts.categories.includes('*')) && dataField) {
         opts.categories = getUniqFieldValues(records, dataField);
       }
+      if (opts.categories && fieldType == 'number') {
+        opts.categories = opts.categories.map(str => +str);
+      }
     }
 
     if (opts.classes) {
@@ -34740,34 +34798,48 @@ ${svg}
   }
 
   function getFeatureEditor(lyr, dataset) {
-    var changed = false;
     var api = {};
     // need to copy attribute to avoid circular references if geojson is assigned
     // to a data property.
     var copy = copyLayer(lyr);
     var features = exportLayerAsGeoJSON(copy, dataset, {}, true);
+    var features2 = [];
 
     api.get = function(i) {
+      if (i > 0) features[i-1] = null; // garbage-collect old features
       return features[i];
     };
 
     api.set = function(feat, i) {
-      changed = true;
+      var arr;
+
       if (utils.isString(feat)) {
         feat = JSON.parse(feat);
       }
-      features[i] = GeoJSON.toFeature(feat); // TODO: validate
+
+      if (!feat) return;
+
+      if (feat.type == 'GeometryCollection') {
+        arr = feat.geometries.map(geom => GeoJSON.toFeature(geom));
+      } else if (feat.type == 'FeatureCollection') {
+        arr = feat.features;
+      } else {
+        feat = GeoJSON.toFeature(feat);
+      }
+
+      if (arr) {
+        features2 = features2.concat(arr);
+      } else {
+        features2.push(feat);
+      }
     };
 
     api.done = function() {
-      if (!changed) return; // read-only expression
-      // TODO: validate number of features, etc.
+      if (features2.length === 0) return; // read-only expression
       var geojson = {
         type: 'FeatureCollection',
-        features: features
+        features: features2
       };
-
-      // console.log(JSON.stringify(geojson, null, 2))
       return importGeoJSON(geojson);
     };
     return api;
@@ -40530,9 +40602,8 @@ ${svg}
   cmd.polygonGrid = function(targetLayers, targetDataset, opts) {
     requireProjectedDataset(targetDataset);
     var params = getGridParams(targetLayers, targetDataset, opts);
-    var gridDataset = makeGridDataset(params);
-
-    gridDataset.info = targetDataset.info; // copy CRS to grid dataset // TODO: improve
+    var gridDataset = makeGridDataset(params); // grid is a new dataset
+    gridDataset.info = copyDatasetInfo(targetDataset.info);
     setOutputLayerName(gridDataset.layers[0], null, 'grid', opts);
     if (opts.debug) gridDataset.layers.push(cmd.pointGrid2(targetLayers, targetDataset, opts));
     return gridDataset;

package/package.json

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

package/www/index.html

@@ -156,7 +156,7 @@
 		<div class="export-formats option-menu">
 		</div>
 
-		<div style="height:10px"></div>
+		<div style="height:11px"></div>
 
 		<div class="option-menu"><input type="text" class="text-input advanced-options" placeholder="command line options" />
 		<a href="https://github.com/mbloch/mapshaper/wiki/Command-Reference#-o-output" target="_mapshaper_output_docs">
@@ -164,15 +164,15 @@
 		<div class="tip">Enter options from the command line interface for
 the -o command. Examples: bbox no-quantization
 precision=0.001. Click to see all options.</div></div></div></a>
-</div>
+
 
 	<!-- 	<div class="option-menu">
 	<input id="ofile-name" class="text-input" type="text" placeholder="output file name" />
 	</div> -->
-
-		<div class="save-btn btn dialog-btn">Export</div>
-		<!-- <span id="save-to-clipboard" class="inline-checkbox" style="display: inline-block;"><input type="checkbox"/>save to clipboard</span>		 -->
-		<span id="save-preference" class="inline-checkbox" style="display: none;"><input type="checkbox"/>choose directory</span>
+		<div><span id="save-preference" style="display: none;"><input type="checkbox"/>choose output directory</span></div>
+		<div><span id="save-to-clipboard" style="display: inline-block;"><input type="checkbox"/>save to clipboard</span></div>
+		</div>
+		<div id="export-btn" class="btn dialog-btn" style="margin-top: 4px;">Export</div>
 
 	</div>
 </div>

package/www/mapshaper-gui.js

@@ -3504,6 +3504,7 @@
         // keeping a reference to current arcs and intersections, so intersections
         // don't need to be recalculated when 'repair' button is pressed.
         _currArcs,
+        _currLayer,
         _currXX;
 
     gui.on('simplify_drag_start', hide);
@@ -3524,8 +3525,8 @@
     });
 
     repairBtn.on('click', function() {
-      var fixed = internal.repairIntersections(_currArcs, _currXX);
-      showIntersections(fixed, _currArcs);
+      _currXX = internal.repairIntersections(_currArcs, _currXX);
+      showIntersections();
       repairBtn.addClass('disabled');
       model.updated({repair: true});
       gui.session.simplificationRepair();
@@ -3573,13 +3574,17 @@
         showBtn = false;
       }
       el.show();
-      showIntersections(XX, arcs);
+      _currLayer = e.layer;
+      _currArcs = arcs;
+      _currXX = XX;
+      showIntersections();
       repairBtn.classed('disabled', !showBtn);
     }
 
     function reset() {
       _currArcs = null;
       _currXX = null;
+      _currLayer = null;
       hide();
     }
 
@@ -3590,13 +3595,11 @@
       reset();
     }
 
-    function showIntersections(XX, arcs) {
-      var n = XX.length, pointLyr;
-      _currXX = XX;
-      _currArcs = arcs;
+    function showIntersections() {
+      var n = _currXX.length, pointLyr;
       if (n > 0) {
         // console.log("first intersection:", internal.getIntersectionDebugData(XX[0], arcs));
-        pointLyr = {geometry_type: 'point', shapes: [internal.getIntersectionPoints(XX)]};
+        pointLyr = internal.getIntersectionLayer(_currXX, _currLayer, _currArcs);
         map.setIntersectionLayer(pointLyr, {layers:[pointLyr]});
         readout.html(utils$1.format('<span class="icon"></span>%s line intersection%s <img class="close-btn" src="images/close.png">', n, utils$1.pluralSuffix(n)));
         readout.findChild('.close-btn').on('click', dismiss);
@@ -3672,11 +3675,12 @@
       return;
     }
 
-    new SimpleButton(menu.findChild('.save-btn').addClass('default-btn')).on('click', onExportClick);
+    new SimpleButton(menu.findChild('#export-btn').addClass('default-btn')).on('click', onExportClick);
     gui.addMode('export', turnOn, turnOff, exportBtn);
     gui.keyboard.onMenuSubmit(menu, onExportClick);
+    var savePreferenceCheckbox;
     if (window.showSaveFilePicker) {
-      menu.findChild('#save-preference')
+      savePreferenceCheckbox = menu.findChild('#save-preference')
         .css('display', 'inline-block')
         .findChild('input')
         .on('change', function() {
@@ -3684,11 +3688,44 @@
         })
         .attr('checked', GUI.getSavedValue('choose-save-dir') || null);
     }
+    var clipboardCheckbox = menu.findChild('#save-to-clipboard')
+      .findChild('input')
+      .on('change', function() {
+        updateExportCheckboxes();
+      });
+
+    function setDisabled(inputEl, flag) {
+      if (!inputEl) return;
+      inputEl.node().disabled = !!flag;
+      inputEl.parent().css({color: flag ? '#bbb' : 'black'});
+    }
+
+    function checkboxOn(inputEl) {
+      if (!inputEl) return false;
+      return inputEl.node().checked && !inputEl.node().disabled;
+    }
+
+    function updateExportCheckboxes() {
+      // disable cliboard if not usable
+      var canUseClipboard = clipboardIsAvailable();
+      setDisabled(clipboardCheckbox, !canUseClipboard);
+
+      // disable save to directory checkbox if clipboard is selected
+      setDisabled(savePreferenceCheckbox, checkboxOn(clipboardCheckbox));
+    }
+
+    function clipboardIsAvailable() {
+      var layers = getSelectedLayerEntries();
+      var fmt = getSelectedFormat();
+      return layers.length == 1 && ['json', 'geojson', 'dsv', 'topojson'].includes(fmt);
+    }
+
 
     function turnOn() {
       layersArr = initLayerMenu();
       // initZipOption();
       initFormatMenu();
+      updateExportCheckboxes();
       menu.show();
     }
 
@@ -3697,22 +3734,25 @@
       menu.hide();
     }
 
-    function getSelectedLayers() {
-      var targets = layersArr.reduce(function(memo, o) {
+    function getSelectedLayerEntries() {
+      return layersArr.reduce(function(memo, o) {
         return o.checkbox.checked ? memo.concat(o.target) : memo;
       }, []);
-      return internal.groupLayersByDataset(targets);
+    }
+
+    function getExportTargets() {
+      return internal.groupLayersByDataset(getSelectedLayerEntries());
     }
 
     function onExportClick() {
-      var layers = getSelectedLayers();
-      if (layers.length === 0) {
+      var targets = getExportTargets();
+      if (targets.length === 0) {
         return gui.alert('No layers were selected');
       }
       gui.clearMode();
       gui.showProgressMessage('Exporting');
       setTimeout(function() {
-        exportMenuSelection(layers).catch(function(err) {
+        exportMenuSelection(targets).catch(function(err) {
           if (utils$1.isString(err)) {
             gui.alert(err);
           } else {
@@ -3742,12 +3782,22 @@
     }
 
     // done: function(string|Error|null)
-    async function exportMenuSelection(layers) {
+    async function exportMenuSelection(targets) {
       var opts = getExportOpts();
       // note: command line "target" option gets ignored
-      var files = await internal.exportTargetLayers(layers, opts);
+      var files = await internal.exportTargetLayers(targets, opts);
       gui.session.layersExported(getTargetLayerIds(), getExportOptsAsString());
-      await utils$1.promisify(internal.writeFiles)(files, opts);
+      if (files.length == 1 && checkboxOn(clipboardCheckbox)) {
+        await saveFileContentToClipboard(files[0].content);
+      } 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) {
@@ -3833,6 +3883,7 @@
     }
 
     function updateToggleBtn() {
+      updateExportCheckboxes(); // checkbox visibility is affected by number of export layers
       if (!toggleBtn) return;
       var state = getSelectionState();
       // style of intermediate checkbox state doesn't look right in Chrome --
@@ -3853,30 +3904,49 @@
       return 'some';
     }
 
-    function getInputFormats() {
-      return model.getDatasets().reduce(function(memo, d) {
-        var fmts = d.info && d.info.input_formats || [];
-        return memo.concat(fmts);
-      }, []);
-    }
-
     function getDefaultExportFormat() {
       var dataset = model.getActiveLayer().dataset;
-      return dataset.info && dataset.info.input_formats &&
-          dataset.info.input_formats[0] || 'geojson';
+      var inputFmt = dataset.info && dataset.info.input_formats &&
+          dataset.info.input_formats[0];
+      return getExportFormats().includes(inputFmt) ? inputFmt : 'geojson';
+    }
+
+    function getExportFormats() {
+      // return ['shapefile', 'geojson', 'topojson', 'json', 'dsv', 'kml', 'svg', internal.PACKAGE_EXT];
+      return ['shapefile', 'json', 'geojson', 'dsv', 'topojson', 'kml', internal.PACKAGE_EXT, 'svg'];
     }
 
     function initFormatMenu() {
-      var defaults = ['shapefile', 'geojson', 'topojson', 'json', 'dsv', 'kml', 'svg', internal.PACKAGE_EXT];
-      var formats = utils$1.uniq(defaults.concat(getInputFormats()));
+      var formats = getExportFormats();
+      // var formats = utils.uniq(getExportFormats().concat(getInputFormats()));
       var items = formats.map(function(fmt) {
-        return utils$1.format('<div><label><input type="radio" name="format" value="%s"' +
-          ' class="radio">%s</label></div>', fmt, internal.getFormatName(fmt));
+        return utils$1.format('<td><label><input type="radio" name="format" value="%s"' +
+          ' class="radio">%s</label></td>', fmt, internal.getFormatName(fmt));
       });
-      menu.findChild('.export-formats').html(items.join('\n'));
+      var table = '<table>';
+      for (var i=0; i<items.length; i+=2) {
+        table += '<tr>' + items[i] + items[i+1] + '<tr>';
+      }
+      table += '</table>';
+
+      // menu.findChild('.export-formats').html(items.join('\n'));
+      menu.findChild('.export-formats').html(table);
       menu.findChild('.export-formats input[value="' + getDefaultExportFormat() + '"]').node().checked = true;
+      // update save-as settings when value changes
+      menu.findChildren('input[type="radio"]').forEach(el => {
+        el.on('change', updateExportCheckboxes);
+      });
     }
 
+
+    // function getInputFormats() {
+    //   return model.getDatasets().reduce(function(memo, d) {
+    //     var fmts = d.info && d.info.input_formats || [];
+    //     return memo.concat(fmts);
+    //   }, []);
+    // }
+
+
     function initZipOption() {
       var html = `<label><input type="checkbox">Save to .zip file</label>`;
       menu.findChild('.export-zip-option').html(html);
@@ -6303,6 +6373,23 @@
     return [xmin, ymin, xmax, ymax];
   }
 
+
+  function findArcIdFromVertexId(i, ii) {
+    // binary search
+    // possible optimization: use interpolation to find a better partition value.
+    var lower = 0, upper = ii.length - 1;
+    var middle;
+    while (lower < upper) {
+      middle = Math.ceil((lower + upper) / 2);
+      if (i < ii[middle]) {
+        upper = middle - 1;
+      } else {
+        lower = middle;
+      }
+    }
+    return lower; // assumes dataset is not empty
+  }
+
   function deleteVertex(arcs, i) {
     var data = arcs.getVertexData();
     var nn = data.nn;
@@ -8177,8 +8264,8 @@
       var popup = showPopupAlert('', 'Add field');
       var el = popup.container();
       el.addClass('option-menu');
-      var html = `<input type="text" class="field-name text-input" placeholder="field name"><br>
-    <input type="text" class="field-value text-input" placeholder="value"><br>
+      var html = `<div><input type="text" class="field-name text-input" placeholder="field name"></div>
+    <div><input type="text" class="field-value text-input" placeholder="value"><div>
     <div tabindex="0" class="btn dialog-btn">Apply</div> <span class="inline-checkbox"><input type="checkbox" class="all" />assign value to all records</span>`;
       el.html(html);
 
@@ -11495,6 +11582,9 @@
       new SessionSnapshots(gui);
     }
 
+    var msgCount = 0;
+    var clearMsg;
+
     initModeRules(gui);
 
     gui.showProgressMessage = function(msg) {
@@ -11503,10 +11593,25 @@
           .appendTo('body');
       }
       El('<div>').text(msg).appendTo(gui.progressMessage.empty().show());
+      clearMsg = getClearFunction(msgCount);
     };
 
+    function getClearFunction(count) {
+      var time = Date.now();
+      // wait at least [min] milliseconds before closing
+      var min = 400;
+      msgCount = ++count;
+      return function() {
+        setTimeout(function() {
+          if (count != msgCount) return;
+          if (gui.progressMessage) gui.progressMessage.hide();
+        }, Math.max(min - (Date.now() - time), 0));
+      };
+    }
+
     gui.clearProgressMessage = function() {
-      if (gui.progressMessage) gui.progressMessage.hide();
+      clearMsg();
+      // if (gui.progressMessage) gui.progressMessage.hide();
     };
 
     gui.consoleIsOpen = function() {

package/www/mapshaper.js

@@ -1,6 +1,6 @@
 (function () {
 
-  var VERSION = "0.6.45";
+  var VERSION = "0.6.46";
 
 
   var utils = /*#__PURE__*/Object.freeze({
@@ -1813,6 +1813,23 @@
     return [xmin, ymin, xmax, ymax];
   }
 
+
+  function findArcIdFromVertexId(i, ii) {
+    // binary search
+    // possible optimization: use interpolation to find a better partition value.
+    var lower = 0, upper = ii.length - 1;
+    var middle;
+    while (lower < upper) {
+      middle = Math.ceil((lower + upper) / 2);
+      if (i < ii[middle]) {
+        upper = middle - 1;
+      } else {
+        lower = middle;
+      }
+    }
+    return lower; // assumes dataset is not empty
+  }
+
   function deleteVertex(arcs, i) {
     var data = arcs.getVertexData();
     var nn = data.nn;
@@ -1939,6 +1956,7 @@
     __proto__: null,
     absArcId: absArcId,
     calcArcBounds: calcArcBounds,
+    findArcIdFromVertexId: findArcIdFromVertexId,
     deleteVertex: deleteVertex,
     insertVertex: insertVertex,
     countFilteredVertices: countFilteredVertices,
@@ -3405,7 +3423,7 @@
     var item;
     for (var i=0; i<arr.length; i++) {
       item = arr[i];
-      if (item instanceof Array) {
+      if (Array.isArray(item)) {
         forEachArcId(item, cb);
       } else if (utils.isInteger(item)) {
         var val = cb(item);
@@ -6953,6 +6971,15 @@
     utils.defaults(destInfo, srcInfo);
   }
 
+  function copyDatasetInfo(info) {
+    // not a deep copy... objects like info.crs are read-only, so copy-by-reference
+    // should be ok
+    var info2 = Object.assign({}, info);
+    if (Array.isArray(info.input_files)) {
+      info2.input_files = info.input_files.concat();
+    }
+    return info2;
+  }
 
   function splitApartLayers(dataset, layers) {
     var datasets = [];
@@ -7119,6 +7146,7 @@
     __proto__: null,
     splitDataset: splitDataset,
     mergeDatasetInfo: mergeDatasetInfo,
+    copyDatasetInfo: copyDatasetInfo,
     splitApartLayers: splitApartLayers,
     copyDataset: copyDataset,
     copyDatasetForExport: copyDatasetForExport,
@@ -15481,6 +15509,17 @@
 
   // Keep track of whether positive or negative integer ids are 'used' or not.
 
+
+  function SimpleIdTestIndex(n) {
+    var index = new Uint8Array(n);
+    this.setId = function(id) {
+      index[id] = 1;
+    };
+    this.hasId = function(id) {
+      return index[id] === 1;
+    };
+  }
+
   function IdTestIndex(n) {
     var index = new Uint8Array(n);
     var setList = [];
@@ -15743,15 +15782,30 @@
     getHoleDivider: getHoleDivider
   });
 
-  // Convert an array of intersections into an ArcCollection (for display)
-  //
-
   function getIntersectionPoints(intersections) {
     return intersections.map(function(obj) {
           return [obj.x, obj.y];
         });
   }
 
+  function getIntersectionLayer(intersections, lyr, arcs) {
+    // return {geometry_type: 'point', shapes: [getIntersectionPoints(XX)]};
+    var ii = arcs.getVertexData().ii;
+    var index = new SimpleIdTestIndex(arcs.size());
+    forEachArcId(lyr.shapes, arcId => {
+      index.setId(absArcId(arcId));
+    });
+    var points = [];
+    intersections.forEach(obj => {
+      var arc1 = findArcIdFromVertexId(obj.a[0], ii);
+      var arc2 = findArcIdFromVertexId(obj.b[0], ii);
+      if (index.hasId(arc1) && index.hasId(arc2)) {
+        points.push([obj.x, obj.y]);
+      }
+    });
+    return {geometry_type: 'point', shapes: [points]};
+  }
+
   // Identify intersecting segments in an ArcCollection
   //
   // To find all intersections:
@@ -15999,6 +16053,7 @@
   var SegmentIntersection = /*#__PURE__*/Object.freeze({
     __proto__: null,
     getIntersectionPoints: getIntersectionPoints,
+    getIntersectionLayer: getIntersectionLayer,
     findSegmentIntersections: findSegmentIntersections,
     sortIntersections: sortIntersections,
     dedupIntersections: dedupIntersections,
@@ -33396,7 +33451,7 @@ ${svg}
     if (n != values.length) {
       stop('Mismatch in number of categories and number of values');
     }
-    return values;
+    return parseValues(values); // convert numerical strings to numbers
   }
 
   function getIndexes(n) {
@@ -33564,7 +33619,7 @@ ${svg}
     if (opts.index_field) {
       dataField = opts.index_field;
       fieldType = getColumnType(opts.field, records);
-   } else if (opts.field) {
+    } else if (opts.field) {
       dataField = opts.field;
       fieldType = getColumnType(opts.field, records);
     }
@@ -33596,6 +33651,9 @@ ${svg}
       if ((!opts.categories || opts.categories.includes('*')) && dataField) {
         opts.categories = getUniqFieldValues(records, dataField);
       }
+      if (opts.categories && fieldType == 'number') {
+        opts.categories = opts.categories.map(str => +str);
+      }
     }
 
     if (opts.classes) {
@@ -34740,34 +34798,48 @@ ${svg}
   }
 
   function getFeatureEditor(lyr, dataset) {
-    var changed = false;
     var api = {};
     // need to copy attribute to avoid circular references if geojson is assigned
     // to a data property.
     var copy = copyLayer(lyr);
     var features = exportLayerAsGeoJSON(copy, dataset, {}, true);
+    var features2 = [];
 
     api.get = function(i) {
+      if (i > 0) features[i-1] = null; // garbage-collect old features
       return features[i];
     };
 
     api.set = function(feat, i) {
-      changed = true;
+      var arr;
+
       if (utils.isString(feat)) {
         feat = JSON.parse(feat);
       }
-      features[i] = GeoJSON.toFeature(feat); // TODO: validate
+
+      if (!feat) return;
+
+      if (feat.type == 'GeometryCollection') {
+        arr = feat.geometries.map(geom => GeoJSON.toFeature(geom));
+      } else if (feat.type == 'FeatureCollection') {
+        arr = feat.features;
+      } else {
+        feat = GeoJSON.toFeature(feat);
+      }
+
+      if (arr) {
+        features2 = features2.concat(arr);
+      } else {
+        features2.push(feat);
+      }
     };
 
     api.done = function() {
-      if (!changed) return; // read-only expression
-      // TODO: validate number of features, etc.
+      if (features2.length === 0) return; // read-only expression
       var geojson = {
         type: 'FeatureCollection',
-        features: features
+        features: features2
       };
-
-      // console.log(JSON.stringify(geojson, null, 2))
       return importGeoJSON(geojson);
     };
     return api;
@@ -40530,9 +40602,8 @@ ${svg}
   cmd.polygonGrid = function(targetLayers, targetDataset, opts) {
     requireProjectedDataset(targetDataset);
     var params = getGridParams(targetLayers, targetDataset, opts);
-    var gridDataset = makeGridDataset(params);
-
-    gridDataset.info = targetDataset.info; // copy CRS to grid dataset // TODO: improve
+    var gridDataset = makeGridDataset(params); // grid is a new dataset
+    gridDataset.info = copyDatasetInfo(targetDataset.info);
     setOutputLayerName(gridDataset.layers[0], null, 'grid', opts);
     if (opts.debug) gridDataset.layers.push(cmd.pointGrid2(targetLayers, targetDataset, opts));
     return gridDataset;

package/www/page.css

@@ -90,7 +90,7 @@ body {
 .dialog-btn {
 	display: inline-block;
 	margin-bottom: 1px;
-	margin-top: 1px;
+	margin-top: 3px;
 	font-size: 13px;
 	color: white;
 	min-width: 28px;
@@ -232,6 +232,13 @@ body {
 	padding: 4px 6px 5px 6px;
 }
 
+.export-options table {
+	margin: 0;
+	border-collapse: collapse;
+	border-spacing: 0;
+	width: 100%;
+}
+
 .export-layer-list {
 	max-height: 160px;
 	overflow: hidden;
@@ -274,7 +281,7 @@ body {
 .alert-title {
 	line-height: 1.1;
 	font-weight: bold;
-	margin-bottom: 5px;
+	margin-bottom: 7px;
 }
 
 div.alert-title, div.error-message {
@@ -461,6 +468,7 @@ body.dragover #import-options-drop-area .drop-area {
 	margin: 0 0 5px 0;
 }
 
+
 #mshp-not-supported {
 	display: none;
 	z-index: 100;
@@ -549,10 +557,12 @@ body.dragover #import-options-drop-area .drop-area {
 	font-size: 90%;
 }
 
+
+
 /*.option-menu input[type="radio"],
 .option-menu input[type="checkbox"]  */
-.info-box input[type="radio"],
-.info-box input[type="checkbox"] 
+input[type="radio"],
+input[type="checkbox"] 
 {
 	position: relative;
 	top: 1px;
@@ -1126,7 +1136,7 @@ img.close-btn:hover,
 
 .basemap-styles > div {
 	display: inline-block;
-	margin-bottom:  8px;
+	margin-bottom:  6px;
 }
 
 .basemap-styles > div:nth-child(even) {
@@ -1234,9 +1244,7 @@ div.basemap-style-btn.active img {
 }
 
 .inline-checkbox {
-	position: relative;
-	top: 1px;
-	left: 5px;
+	margin-left: 5px;
 }
 
 #save-preference {