From 1305246d7b3a477afb3f3f0a87f94027a527b6e5 Mon Sep 17 00:00:00 2001
From: Azgaar <azgaar.fmg@yandex.com>
Date: Tue, 22 Oct 2024 02:21:48 +0200
Subject: [PATCH] refactor: submap - recalculateMapSize

---
 index.html                   |  8 ++++++
 main.js                      |  8 ------
 modules/resample.js          |  4 ++-
 modules/ui/submap-tool.js    | 48 +++++++++++++++++++++++++-----------
 modules/ui/transform-tool.js |  3 +--
 5 files changed, 46 insertions(+), 25 deletions(-)

diff --git a/index.html b/index.html
index 250fafd0..078d5411 100644
--- a/index.html
+++ b/index.html
@@ -5771,6 +5771,14 @@
           This operation is destructive and irreversible. It will create a completely new map based on the current one.
           Don't forget to save the .map file to your machine first!
         </p>
+
+        <div style="display: flex; gap: 1em">
+          <div>Points number</div>
+          <div>
+            <input id="submapPointsInput" type="range" min="1" max="13" value="4" />
+            <output id="submapPointsFormatted" style="color: #053305">10K</output>
+          </div>
+        </div>
       </div>
 
       <div id="transformTool" style="display: none" class="dialog">
diff --git a/main.js b/main.js
index 86df7d65..ed06c9e5 100644
--- a/main.js
+++ b/main.js
@@ -495,14 +495,6 @@ function resetZoom(d = 1000) {
   svg.transition().duration(d).call(zoom.transform, d3.zoomIdentity);
 }
 
-// calculate x y extreme points of viewBox
-function getViewBoxExtent() {
-  return [
-    [Math.abs(viewX / scale), Math.abs(viewY / scale)],
-    [Math.abs(viewX / scale) + graphWidth / scale, Math.abs(viewY / scale) + graphHeight / scale]
-  ];
-}
-
 // active zooming feature
 function invokeActiveZooming() {
   const isOptimized = shapeRendering.value === "optimizeSpeed";
diff --git a/modules/resample.js b/modules/resample.js
index 22f565af..a7496e13 100644
--- a/modules/resample.js
+++ b/modules/resample.js
@@ -8,7 +8,9 @@ window.Resample = (function () {
     inverse: f(Number, Number) -> [Number, Number]
     scale: Number
   */
-  function process({parentMap, projection, inverse, scale}) {
+  function process({projection, inverse, scale}) {
+    const parentMap = {grid: deepCopy(grid), pack: deepCopy(pack), notes: deepCopy(notes)};
+
     grid = generateGrid();
     pack = {};
     notes = parentMap.notes;
diff --git a/modules/ui/submap-tool.js b/modules/ui/submap-tool.js
index f2e334f4..8ff7d8d7 100644
--- a/modules/ui/submap-tool.js
+++ b/modules/ui/submap-tool.js
@@ -20,36 +20,48 @@ function openSubmapTool() {
   if (modules.openSubmapTool) return;
   modules.openSubmapTool = true;
 
+  // add listeners
+  byId("submapPointsInput").on("input", handleCellsChange);
+
   function generateSubmap() {
     INFO && console.group("generateSubmap");
-    const [[x0, y0]] = getViewBoxExtent();
 
-    // converting map position on the planet. TODO: fix, coordinates are wrong
-    const mapSizeOutput = byId("mapSizeOutput");
-    const latitudeOutput = byId("latitudeOutput");
-    const latN = 90 - ((180 - (mapSizeInput.value / 100) * 180) * latitudeOutput.value) / 100;
-    const newLatN = latN - ((y0 / graphHeight) * mapSizeOutput.value * 180) / 100;
-    mapSizeOutput.value /= scale;
-    latitudeOutput.value = ((90 - newLatN) / (180 - (mapSizeOutput.value / 100) * 180)) * 100;
-    byId("mapSizeInput").value = mapSizeOutput.value;
-    byId("latitudeInput").value = latitudeOutput.value;
+    const [x0, y0] = [Math.abs(viewX / scale), Math.abs(viewY / scale)]; // top-left corner
+    recalculateMapSize(x0, y0);
 
-    distanceScale = distanceScaleInput.value = rn(distanceScale / scale, 2);
-    populationRate = populationRateInput.value = rn(populationRate / scale, 2);
+    const cellsNumber = +byId("submapPointsInput").value;
+    changeCellsDensity(cellsNumber);
 
-    const parentMap = {grid: deepCopy(grid), pack: deepCopy(pack), notes: deepCopy(notes)};
     const projection = (x, y) => [(x - x0) * scale, (y - y0) * scale];
     const inverse = (x, y) => [x / scale + x0, y / scale + y0];
 
     resetZoom(0);
     undraw();
-    Resample.process({parentMap, projection, inverse, scale});
+    Resample.process({projection, inverse, scale});
     rescaleBurgStyles(scale);
     drawLayers();
 
     INFO && console.groupEnd("generateSubmap");
   }
 
+  function recalculateMapSize(x0, y0) {
+    const mapSize = +byId("mapSizeOutput").value;
+    byId("mapSizeOutput").value = byId("mapSizeInput").value = rn(mapSize / scale, 2);
+
+    const latT = mapCoordinates.latT / scale;
+    const latN = getLatitude(y0);
+    const latShift = (90 - latN) / (180 - latT);
+    byId("latitudeOutput").value = byId("latitudeInput").value = rn(latShift * 100, 2);
+
+    const lotT = mapCoordinates.lonT / scale;
+    const lonE = getLongitude(x0 + graphWidth / scale);
+    const lonShift = (180 - lonE) / (360 - lotT);
+    byId("longitudeOutput").value = byId("longitudeInput").value = rn(lonShift * 100, 2);
+
+    distanceScale = distanceScaleInput.value = rn(distanceScale / scale, 2);
+    populationRate = populationRateInput.value = rn(populationRate / scale, 2);
+  }
+
   function rescaleBurgStyles(scale) {
     const burgIcons = [...byId("burgIcons").querySelectorAll("g")];
     for (const bi of burgIcons) {
@@ -65,4 +77,12 @@ function openSubmapTool() {
       bl.dataset["size"] = Math.max(rn((size + size / scale) / 2, 2), 1) * scale;
     }
   }
+
+  function handleCellsChange() {
+    const cells = cellsDensityMap[+this.value] || 1000;
+    this.dataset.cells = cells;
+    const output = byId("submapPointsFormatted");
+    output.value = getCellsDensityValue(cells);
+    output.style.color = getCellsDensityColor(cells);
+  }
 }
diff --git a/modules/ui/transform-tool.js b/modules/ui/transform-tool.js
index d610da03..32a14b92 100644
--- a/modules/ui/transform-tool.js
+++ b/modules/ui/transform-tool.js
@@ -126,12 +126,11 @@ async function openTransformTool() {
     const cellsNumber = +byId("transformPointsInput").value;
     changeCellsDensity(cellsNumber);
 
-    const parentMap = {grid: deepCopy(grid), pack: deepCopy(pack), notes: deepCopy(notes)};
     const [projection, inverse] = getProjection();
 
     resetZoom(0);
     undraw();
-    Resample.process({parentMap, projection, inverse, scale: 1});
+    Resample.process({projection, inverse, scale: 1});
     drawLayers();
 
     INFO && console.groupEnd("transformMap");