From c9fbbafe58caa80e6f166e1ec9053497e40fa1c1 Mon Sep 17 00:00:00 2001
From: Nico <Nico>
Date: Wed, 17 Jun 2026 00:34:35 -0700
Subject: [PATCH] fix: focus selected map area editor

---
 .../src/components/maps/LocationMapBottomSheet.jsx     | 10 ----------
 frontend/tests/location-map-manager.spec.ts            | 10 +++-------
 2 files changed, 3 insertions(+), 17 deletions(-)

diff --git a/frontend/src/components/maps/LocationMapBottomSheet.jsx b/frontend/src/components/maps/LocationMapBottomSheet.jsx
index 378f6bd..1f2602c 100644
--- a/frontend/src/components/maps/LocationMapBottomSheet.jsx
+++ b/frontend/src/components/maps/LocationMapBottomSheet.jsx
@@ -309,16 +309,6 @@ export default function LocationMapBottomSheet({
         />
       ) : null}
 
-      {showSelectedObjectForm && secondaryDraftActions ? (
-        <div
-          className="location-map-editor-action-stack location-map-selected-secondary-actions"
-          role="group"
-          aria-label="More map draft actions"
-        >
-          {secondaryDraftActions}
-        </div>
-      ) : null}
-
       {showUnmappedPanel ? (
         <UnmappedItemsPanel
           visibleUnmappedItems={visibleUnmappedItems}
diff --git a/frontend/tests/location-map-manager.spec.ts b/frontend/tests/location-map-manager.spec.ts
index 241c201..6342c04 100644
--- a/frontend/tests/location-map-manager.spec.ts
+++ b/frontend/tests/location-map-manager.spec.ts
@@ -5003,27 +5003,22 @@ test("mobile editor uses bottom sheet controls instead of desktop object toolbar
   await expect(lockedFlag).toHaveAttribute("aria-pressed", "false");
   await expect(lockedFlag.locator(".location-map-object-flag-state.is-on")).toHaveCount(0);
   const selectedPrimaryActions = page.locator(".location-map-selected-primary-actions");
-  const selectedSecondaryActions = page.locator(".location-map-selected-secondary-actions");
   await expect(selectedPrimaryActions.getByRole("button", { name: "Save Draft" })).toHaveCount(0);
   await expect(selectedPrimaryActions.getByRole("button", { name: "Publish" })).toBeVisible();
-  await expect(selectedSecondaryActions.getByRole("button", { name: "Add Area" })).toBeVisible();
-  await expect(selectedSecondaryActions.getByRole("button", { name: "View Draft" })).toHaveCount(0);
+  await expect(page.locator(".location-map-selected-secondary-actions")).toHaveCount(0);
   await expect(page.getByRole("button", { name: "Bought" })).toHaveCount(0);
   const selectedPrimaryBox = await selectedPrimaryActions.boundingBox();
-  const selectedSecondaryButtonBox = await selectedSecondaryActions.getByRole("button", { name: "Add Area" }).boundingBox();
   const labelBox = await labelInput.boundingBox();
   const firstFieldRowBox = await fieldRows.first().boundingBox();
   const firstFlagButtonBox = await objectFlagButtons.first().boundingBox();
   expect(selectedPrimaryBox).not.toBeNull();
-  expect(selectedSecondaryButtonBox).not.toBeNull();
   expect(labelBox).not.toBeNull();
   expect(firstFieldRowBox).not.toBeNull();
   expect(firstFlagButtonBox).not.toBeNull();
-  if (!selectedPrimaryBox || !selectedSecondaryButtonBox || !labelBox || !firstFieldRowBox || !firstFlagButtonBox) {
+  if (!selectedPrimaryBox || !labelBox || !firstFieldRowBox || !firstFlagButtonBox) {
     throw new Error("Selected object draft actions were not measurable");
   }
   expect(selectedPrimaryBox.y).toBeLessThan(labelBox.y);
-  expect(selectedSecondaryButtonBox.height).toBeGreaterThanOrEqual(40);
   expect(firstFieldRowBox.height).toBeLessThanOrEqual(54);
   expect(firstFlagButtonBox.height).toBeGreaterThanOrEqual(40);
   const closeSelectedButton = page.getByRole("button", { name: "Close selected map area" });
@@ -5037,6 +5032,7 @@ test("mobile editor uses bottom sheet controls instead of desktop object toolbar
   await expect(page.getByRole("textbox", { name: "Label" })).toHaveCount(0);
   await expect(page.locator(".location-map-sheet-header strong")).toHaveText("Edit Areas");
   await expect(page.getByRole("button", { name: "Save Draft" })).toHaveCount(0);
+  await expect(page.locator(".location-map-secondary-actions").getByRole("button", { name: "Add Area" })).toBeVisible();
   await page.getByRole("button", { name: "View", exact: true }).click();
   await expect(page.getByRole("textbox", { name: "Label" })).toHaveCount(0);
 });