From 66e5c415636d9da351df5d6bc3f21eafb642b936 Mon Sep 17 00:00:00 2001
From: Nico <Nico>
Date: Wed, 3 Jun 2026 11:01:56 -0700
Subject: [PATCH] fix: prioritize map object settings

---
 .../maps/LocationMapBottomSheet.jsx           | 35 +++++++++++--------
 1 file changed, 20 insertions(+), 15 deletions(-)

diff --git a/frontend/src/components/maps/LocationMapBottomSheet.jsx b/frontend/src/components/maps/LocationMapBottomSheet.jsx
index 904b49e..ccc1653 100644
--- a/frontend/src/components/maps/LocationMapBottomSheet.jsx
+++ b/frontend/src/components/maps/LocationMapBottomSheet.jsx
@@ -55,6 +55,22 @@ export default function LocationMapBottomSheet({
     : selectedObject
       ? selectedTitle
       : "Map Details";
+  const showEditorControls = mode === "edit" && canManage;
+  const showSelectedObjectForm = Boolean(mode === "edit" && editorTool === "edit" && selectedObject);
+  const editorActions = showEditorControls ? (
+    <div className="location-map-editor-actions location-map-primary-actions">
+      <button type="button" onClick={onAddObject}>Add Area</button>
+      <button type="button" onClick={onPreviewDraft}>Preview Draft</button>
+      <button type="button" className="primary" onClick={onSaveDraft} disabled={saving || !hasUnsavedChanges}>Save Draft</button>
+      <button type="button" className="primary" onClick={onPublish} disabled={saving || !mapState?.draft_map}>Publish</button>
+      <button type="button" onClick={onUndo} disabled={history.length === 0}>Undo</button>
+      <button type="button" onClick={onRedo} disabled={future.length === 0}>Redo</button>
+    </div>
+  ) : canManage && mapState?.published_map ? (
+    <div className="location-map-editor-actions">
+      <button type="button" onClick={onEditMode}>Edit Map</button>
+    </div>
+  ) : null;
 
   return (
     <aside className="location-map-bottom-sheet">
@@ -92,22 +108,9 @@ export default function LocationMapBottomSheet({
         </div>
       ) : null}
 
-      {mode === "edit" && canManage ? (
-        <div className="location-map-editor-actions location-map-primary-actions">
-          <button type="button" onClick={onAddObject}>Add Area</button>
-          <button type="button" onClick={onPreviewDraft}>Preview Draft</button>
-          <button type="button" className="primary" onClick={onSaveDraft} disabled={saving || !hasUnsavedChanges}>Save Draft</button>
-          <button type="button" className="primary" onClick={onPublish} disabled={saving || !mapState?.draft_map}>Publish</button>
-          <button type="button" onClick={onUndo} disabled={history.length === 0}>Undo</button>
-          <button type="button" onClick={onRedo} disabled={future.length === 0}>Redo</button>
-        </div>
-      ) : canManage && mapState?.published_map ? (
-        <div className="location-map-editor-actions">
-          <button type="button" onClick={onEditMode}>Edit Map</button>
-        </div>
-      ) : null}
+      {showSelectedObjectForm ? null : editorActions}
 
-      {mode === "edit" && canManage ? (
+      {showEditorControls ? (
         <div className="location-map-mobile-tool-switch" aria-label="Edit tool">
           <button
             type="button"
@@ -232,6 +235,8 @@ export default function LocationMapBottomSheet({
         </p>
       )}
 
+      {showSelectedObjectForm ? editorActions : null}
+
       {filters.showUnmapped && visibleUnmappedItems.length > 0 ? (
         <div className="location-map-unmapped-list">
           <strong>Unmapped Items</strong>