Refactored getBrushPosition helper

src/components/map/MapDrawing.js

@@ -55,8 +55,8 @@ function MapDrawing({
       return getBrushPositionForTool(
         map,
         getRelativePointerPositionNormalized(mapImage),
-        toolId,
+        map.snapToGrid && isShape,
-        toolSettings,
+        false,
         gridSize,
         shapes
       );

src/components/map/MapFog.js

@@ -61,8 +61,8 @@ function MapFog({
       return getBrushPositionForTool(
         map,
         getRelativePointerPositionNormalized(mapImage),
-        toolId,
+        map.snapToGrid && toolSettings.type === "polygon",
-        toolSettings,
+        toolSettings.useEdgeSnapping,
         gridSize,
         shapes
       );

src/components/map/MapMeasure.js

@@ -38,8 +38,8 @@ function MapMeasure({ map, selectedToolSettings, active, gridSize }) {
       return getBrushPositionForTool(
         map,
         getRelativePointerPositionNormalized(mapImage),
-        "drawing",
+        map.snapToGrid,
-        { type: "line" },
+        false,
         gridSize,
         []
       );

src/helpers/drawing.js

@@ -8,22 +8,13 @@ const snappingThreshold = 1 / 5;
 export function getBrushPositionForTool(
   map,
   brushPosition,
-  tool,
+  useGridSnappning,
-  toolSettings,
+  useEdgeSnapping,
   gridSize,
   shapes
 ) {
   let position = brushPosition;
 
-  const useGridSnappning =
-    map.snapToGrid &&
-    ((tool === "drawing" &&
-      (toolSettings.type === "line" ||
-        toolSettings.type === "rectangle" ||
-        toolSettings.type === "circle" ||
-        toolSettings.type === "triangle")) ||
-      (tool === "fog" && toolSettings.type === "polygon"));
-
   if (useGridSnappning) {
     // Snap to corners of grid
     // Subtract offset to transform into offset space then add it back transform back
@@ -58,8 +49,6 @@ export function getBrushPositionForTool(
     }
   }
 
-  const useEdgeSnapping = tool === "fog" && toolSettings.useEdgeSnapping;
-
   if (useEdgeSnapping) {
     const minGrid = Vector2.min(gridSize);
     let closestDistance = Number.MAX_VALUE;