From 9c6822a1acecdd2dbde223af11f03a89bd257208 Mon Sep 17 00:00:00 2001
From: pietervdvn <pietervdvn@posteo.net>
Date: Fri, 24 Dec 2021 02:51:01 +0100
Subject: [PATCH] Improve docs, formatting

---
 Logic/Osm/Actions/ReplaceGeometryAction.ts |  7 ++---
 UI/Popup/ImportButton.ts                   | 30 +++++++++++-----------
 2 files changed, 19 insertions(+), 18 deletions(-)

diff --git a/Logic/Osm/Actions/ReplaceGeometryAction.ts b/Logic/Osm/Actions/ReplaceGeometryAction.ts
index 08848b7c2..651676dc5 100644
--- a/Logic/Osm/Actions/ReplaceGeometryAction.ts
+++ b/Logic/Osm/Actions/ReplaceGeometryAction.ts
@@ -249,9 +249,10 @@ export default class ReplaceGeometryAction extends OsmChangeAction {
     }
 
     /**
-     * For 'this.feature`, gets a corresponding closest node that alreay exsists
-     * @constructor
-     * @private
+     * For 'this.feature`, gets a corresponding closest node that alreay exsists.
+     * 
+     * This method contains the main logic for this module, as it decides which node gets moved where.
+     * 
      */
     private async GetClosestIds(): Promise<{
 
diff --git a/UI/Popup/ImportButton.ts b/UI/Popup/ImportButton.ts
index 2650312b1..af272dc57 100644
--- a/UI/Popup/ImportButton.ts
+++ b/UI/Popup/ImportButton.ts
@@ -309,7 +309,7 @@ export class ConflateButton extends AbstractImportButton {
                      tagSource: UIEventSource<any>, guiState: DefaultGuiState, feature: any, onCancelClicked: () => void): BaseUIElement {
 
         return new FixedUiElement("ReplaceGeometry is currently very broken - use mapcomplete.osm.be for now").SetClass("alert")
-        
+
         const nodesMustMatch = args.snap_onto_layers?.split(";")?.map((tag, i) => TagUtils.Tag(tag, "TagsSpec for import button " + i))
 
         const mergeConfigs = []
@@ -365,19 +365,19 @@ export class ImportWayButton extends AbstractImportButton {
                 {
                     name: "move_osm_point_if",
                     doc: "Moves the OSM-point to the newly imported point if these conditions are met",
-                },{
-                    name:"max_move_distance",
+                }, {
+                name: "max_move_distance",
                 doc: "If an OSM-point is moved, the maximum amount of meters it is moved. Capped on 20m",
                 defaultValue: "1"
-            },{
-            name:"snap_onto_layers",
-                doc:"If no existing nearby point exists, but a line of a specified layer is closeby, snap to this layer instead",
-                
-            },{
-                name:"snap_to_layer_max_distance",
-            doc:"Distance to distort the geometry to snap to this layer",
-defaultValue: "0.1"
-    }],
+            }, {
+                name: "snap_onto_layers",
+                doc: "If no existing nearby point exists, but a line of a specified layer is closeby, snap to this layer instead",
+
+            }, {
+                name: "snap_to_layer_max_distance",
+                doc: "Distance to distort the geometry to snap to this layer",
+                defaultValue: "0.1"
+            }],
             false
         )
     }
@@ -428,14 +428,14 @@ defaultValue: "0.1"
             }
             mergeConfigs.push(mergeConfig)
         }
-        
+
 
         const moveOsmPointIfTags = args["move_osm_point_if"]?.split(";")?.map((tag, i) => TagUtils.Tag(tag, "TagsSpec for import button " + i))
 
         if (nodesMustMatch !== undefined && moveOsmPointIfTags.length > 0) {
-           const moveDistance = Math.min(20, Number(args["max_move_distance"]))
+            const moveDistance = Math.min(20, Number(args["max_move_distance"]))
             const mergeConfig: MergePointConfig = {
-                mode: "move_osm_point" ,
+                mode: "move_osm_point",
                 ifMatches: new And(moveOsmPointIfTags),
                 withinRangeOfM: moveDistance
             }