feat: burg group editor - form

modules/burgs-generator.js

@@ -262,26 +262,26 @@ window.Burgs = (() => {
 
   const getDefaultGroups = () => [
     {name: "capitals", active: true, features: {capital: true}, preview: "watabou-city-generator"},
-    {name: "cities", active: true, percentile: 90, population: [5, Infinity], preview: "watabou-city-generator"},
+    {name: "cities", active: true, percentile: 90, min: 5, preview: "watabou-city-generator"},
     {
       name: "forts",
       active: true,
       features: {citadel: true, walls: false, plaza: false, port: false},
-      population: [0, 1],
+      max: 1,
       preview: null
     },
     {
       name: "monasteries",
       active: true,
       features: {temple: true, walls: false, plaza: false, port: false},
-      population: [0, 0.8],
+      max: 0.8,
       preview: null
     },
     {
       name: "caravanserais",
       active: true,
       features: {port: false, plaza: true},
-      population: [0, 0.8],
+      max: 0.8,
       biomes: [1, 2, 3],
       preview: null
     },
@@ -289,14 +289,15 @@ window.Burgs = (() => {
       name: "trading_posts",
       active: true,
       features: {plaza: true},
-      population: [0, 0.8],
+      max: 0.8,
       biomes: [5, 6, 7, 8, 9, 10, 11, 12],
       preview: null
     },
     {
       name: "villages",
       active: true,
-      population: [0.1, 2],
+      min: 0.1,
+      max: 2,
       features: {walls: false},
       preview: "watabou-village-generator"
     },
@@ -304,7 +305,7 @@ window.Burgs = (() => {
       name: "hamlets",
       active: true,
       features: {walls: false, plaza: false},
-      population: [0, 0.1],
+      max: 0.1,
       preview: "watabou-village-generator"
     },
     {name: "towns", active: true, isDefault: true, preview: "watabou-city-generator"}
@@ -314,9 +315,13 @@ window.Burgs = (() => {
     for (const group of options.burgs.groups) {
       if (!group.active) continue;
 
-      if (group.population) {
-        const [min, max] = group.population;
-        const isFit = burg.population >= min && burg.population <= max;
+      if (group.min) {
+        const isFit = burg.population >= group.min;
+        if (!isFit) continue;
+      }
+
+      if (group.max) {
+        const isFit = burg.population <= group.max;
         if (!isFit) continue;
       }
 

modules/ui/burg-editor.js

@@ -25,7 +25,7 @@ function editBurg(id) {
   byId("burgName").on("input", changeName);
   byId("burgNameReRandom").on("click", generateNameRandom);
   byId("burgGroup").on("change", changeGroup);
-  byId("burgGroupEdit").on("change", editBurgGroups);
+  byId("burgGroupConfigure").on("click", editBurgGroups);
   byId("burgType").on("change", changeType);
   byId("burgCulture").on("change", changeCulture);
   byId("burgNameReCulture").on("click", generateNameCulture);

modules/ui/burg-group-editor.js

@@ -2,42 +2,57 @@
 
 function editBurgGroups() {
   if (customization) return;
-  if (!layerIsOn("toggleBurgs")) toggleBurgs();
-
   addLines();
 
   $("#burgGroupsEditor").dialog({
-    title: "Edit Burg groups",
+    title: "Configure Burg groups",
     resizable: false,
-    position: {my: "left top", at: "left+10 top+140", of: "#map"}
+    position: {my: "center", at: "center", of: "svg"},
+    buttons: {
+      Apply: () => byId("burgGroupsForm").requestSubmit(),
+      Cancel: function () {
+        $(this).dialog("close");
+      }
+    }
   });
 
   if (modules.editBurgGroups) return;
   modules.editBurgGroups = true;
 
   // add listeners
-  byId("burgGroupsEditorAdd").addEventListener("click", addGroup);
-  byId("burgGroupsEditorBody").on("click", ev => {
+  byId("burgGroupsForm").on("submit", submitForm);
+  byId("burgGroupsForm").on("change", validateForm);
+  byId("burgGroupsEditorAdd").on("click", addGroup);
+  byId("burgGroupsEditStyle").on("click", () => editStyle("burgIcons"));
+  byId("burgGroupsBody").on("click", ev => {
     const group = ev.target.closest(".states")?.dataset.id;
     if (ev.target.classList.contains("editStyle")) editStyle("burgs", group);
     else if (ev.target.classList.contains("removeGroup")) removeGroup(group);
   });
 
   function addLines() {
-    byId("burgGroupsEditorBody").innerHTML = "";
+    byId("burgGroupsBody").innerHTML = "";
 
-    const lines = Array.from(burgs.selectAll("g")._groups[0]).map(el => {
-      const count = el.children.length;
-      return /* html */ `<div data-id="${el.id}" class="states" style="display: flex; justify-content: space-between;">
-          <span>${el.id} (${count})</span>
-          <div style="width: auto; display: flex; gap: 0.4em;">
-            <span data-tip="Edit style" class="editStyle icon-brush pointer" style="font-size: smaller;"></span>
-            <span data-tip="Remove group" class="removeGroup icon-trash pointer"></span>
-          </div>
-        </div>`;
+    const lines = options.burgs.groups.map(group => {
+      const count = pack.burgs.filter(burg => !burg.removed && burg.group === group.name).length;
+      // prettier-ignore
+      return /* html */ `<tr name="${group.name}">
+        <td data-tip="Select group to be assigned if other groups are not passed"><input type="radio" name="isDefault" ${group.isDefault && "checked"}></td>
+        <td data-tip="Type group name. It can contain only text, digits and underscore"><input type="text" name="name" value="${group.name}" required pattern="\\w+" /></td>
+        <td data-tip="Set min population constraint"><input type="number" name="min" min="0" step="any" value="${group.min || ''}" /></td>
+        <td data-tip="Set max population constraint"><input type="number" name="max" min="0" step="any" value="${group.max || ''}" /></td>
+        <td data-tip="Set population percentile"><input type="number" name="percentile" min="0" max="100" step="any" value="${group.percentile || ''}" /></td>
+        <td data-tip="Select allowed biomes"><button name="biomes">${group.biomes ? "some" : "all"}</button></td>
+        <td data-tip="Select allowed states"><button name="states">${group.states ? "some" : "all"}</button></td>
+        <td data-tip="Select allowed cultures"><button name="cultures">${group.cultures ? "some" : "all"}</button></td>
+        <td data-tip="Select allowed religions"><button name="religions">${group.religions ? "some" : "all"}</button></td>
+        <td data-tip="Select allowed features" ><button name="features">${group.features ? "some" : "all"}</button></td>
+        <td data-tip="Number of burgs in group">${count}</td>
+        <td data-tip="Activate/deactivate group"><input type="checkbox" name="active" class="native" ${group.active && "checked"} /></td>
+      </tr>`;
     });
 
-    byId("burgGroupsEditorBody").innerHTML = lines.join("");
+    byId("burgGroupsBody").innerHTML = lines.join("");
   }
 
   const DEFAULT_GROUPS = ["roads", "trails", "seaburgs"];
@@ -81,4 +96,49 @@ function editBurgGroups() {
       }
     });
   }
+
+  function validateForm(event) {
+    const form = event.target.form;
+
+    const names = Array.from(form.name).map(input => input.value);
+    form.name.forEach(nameInput => {
+      const value = nameInput.value;
+      const isUnique = names.filter(n => n === value).length === 1;
+      nameInput.setCustomValidity(isUnique ? "" : "Group name should be unique");
+      nameInput.reportValidity();
+    });
+
+    const active = Array.from(form.active).map(input => input.checked);
+    form.active[0].setCustomValidity(active.includes(true) ? "" : "At least one group should be active");
+    form.active[0].reportValidity();
+  }
+
+  function submitForm(event) {
+    event.preventDefault();
+
+    function parseInput(input) {
+      if (input.name === "name") return sanitizeId(input.value);
+      if (input.type === "radio") return input.checked;
+      if (input.type === "checkbox") return input.checked;
+      if (input.type === "number") {
+        const value = input.valueAsNumber;
+        if (value === 0 || isNaN(value)) return null;
+        return value;
+      }
+      return input.value;
+    }
+
+    const lines = Array.from(byId("burgGroupsBody").children);
+    options.burgs.groups = lines.map(line => {
+      const inputs = line.querySelectorAll("input");
+      const group = Array.from(inputs).reduce((obj, input) => {
+        const value = parseInput(input);
+        if (value !== null) obj[input.name] = value;
+        return obj;
+      }, {});
+      return group;
+    });
+
+    $("#burgGroupsEditor").dialog("close");
+  }
 }

modules/ui/military-overview.js

@@ -414,7 +414,7 @@ function overviewMilitary() {
 
     function applyMilitaryOptions() {
       const unitLines = Array.from(tableBody.querySelectorAll("tr"));
-      const names = unitLines.map(r => r.querySelector("input").value.replace(/[&\/\\#, +()$~%.'":*?<>{}]/g, "_"));
+      const names = unitLines.map(r => sanitizeId(r.querySelector("input").value));
       if (new Set(names).size !== names.length) return tip("All units should have unique names", false, "error");
 
       $("#militaryOptions").dialog("close");