From 7af995f561e7c53ad5fec3140e75d1952d8c783f Mon Sep 17 00:00:00 2001
From: Nico <Nico>
Date: Thu, 4 Jun 2026 18:19:06 -0700
Subject: [PATCH] fix: hide unavailable map setup action

---
 .../src/components/maps/LocationMapSetupPanel.jsx | 15 ++++-----------
 frontend/tests/location-map-manager.spec.ts       |  8 +++-----
 2 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/frontend/src/components/maps/LocationMapSetupPanel.jsx b/frontend/src/components/maps/LocationMapSetupPanel.jsx
index 1bccf2a..4624298 100644
--- a/frontend/src/components/maps/LocationMapSetupPanel.jsx
+++ b/frontend/src/components/maps/LocationMapSetupPanel.jsx
@@ -11,9 +11,7 @@ export default function LocationMapSetupPanel({
   onCreateBlank,
 }) {
   const hasZones = zoneCount > 0;
-  const createFromZonesLabel = hasZones
-    ? `Create From ${zoneCount} Zone${zoneCount === 1 ? "" : "s"}`
-    : "Create From Zones";
+  const createFromZonesLabel = `Create From ${zoneCount} Zone${zoneCount === 1 ? "" : "s"}`;
   const createBlankLabel = savingAction === "create-blank" ? "Creating..." : "Create Blank Map";
   const createZonesButtonLabel = savingAction === "create-zones" ? "Creating..." : createFromZonesLabel;
   const publishLabel = savingAction === "publish" ? "Publishing..." : "Publish";
@@ -44,14 +42,9 @@ export default function LocationMapSetupPanel({
       ) : !hasAnyMap && canManage ? (
         <div className="location-map-setup-actions">
           {!hasZones ? (
-            <>
-              <button type="button" className="btn-primary" onClick={onCreateBlank} disabled={saving}>
-                {createBlankLabel}
-              </button>
-              <button type="button" className="btn-secondary" onClick={onCreateFromZones} disabled>
-                {createFromZonesLabel}
-              </button>
-            </>
+            <button type="button" className="btn-primary" onClick={onCreateBlank} disabled={saving}>
+              {createBlankLabel}
+            </button>
           ) : (
             <>
               <button type="button" className="btn-primary" onClick={onCreateFromZones} disabled={saving}>
diff --git a/frontend/tests/location-map-manager.spec.ts b/frontend/tests/location-map-manager.spec.ts
index f919a51..9f47fe0 100644
--- a/frontend/tests/location-map-manager.spec.ts
+++ b/frontend/tests/location-map-manager.spec.ts
@@ -440,14 +440,12 @@ test("admin setup uses compact status rows when no zones exist yet", async ({ pa
   await expect(setupDetails.getByText("Status")).toHaveCount(0);
   await expect(page.getByText("Not started")).toHaveCount(0);
   await expect(page.getByText("No zones are available yet.")).toHaveCount(0);
-  await expect(page.getByRole("button", { name: "Create From Zones" })).toBeDisabled();
+  await expect(page.getByRole("button", { name: "Create From Zones" })).toHaveCount(0);
   await expect(page.getByRole("button", { name: "Create Blank Map" })).toBeEnabled();
   const blankButtonBox = await page.getByRole("button", { name: "Create Blank Map" }).boundingBox();
-  const fromZonesButtonBox = await page.getByRole("button", { name: "Create From Zones" }).boundingBox();
   expect(blankButtonBox).not.toBeNull();
-  expect(fromZonesButtonBox).not.toBeNull();
-  if (!blankButtonBox || !fromZonesButtonBox) throw new Error("No-zone setup actions were not measurable");
-  expect(blankButtonBox.y).toBeLessThan(fromZonesButtonBox.y);
+  if (!blankButtonBox) throw new Error("No-zone setup action was not measurable");
+  expect(blankButtonBox.height).toBeGreaterThanOrEqual(40);
   await expect(page.getByLabel("Map controls")).toHaveCount(0);
 });