nalalangan/grocery-app
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

fix: declutter map setup state

Browse Source
This commit is contained in:
Nico 2026-06-03 12:27:37 -07:00
parent 023aef10ec
commit d370e23529
2 changed files with 16 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
frontend/src/pages/LocationMapManager.jsx
View File

@ -434,19 +434,21 @@ export default function LocationMapManager() {
<LocationMapTopbar location={location} status={status} onBack={handleBack} /> <LocationMapTopbar location={location} status={status} onBack={handleBack} />
<main className="location-map-workspace"> <main className="location-map-workspace">
<LocationMapToolbar {hasAnyMap && mode !== "setup" ? (
mode={mode} <LocationMapToolbar
editorTool={editorTool} mode={mode}
hasAnyMap={hasAnyMap} editorTool={editorTool}
canManage={canManage} hasAnyMap={hasAnyMap}
zoom={zoom} canManage={canManage}
setZoom={setZoom} zoom={zoom}
onFit={handleFitMap} setZoom={setZoom}
onView={handleViewMode} onFit={handleFitMap}
onEdit={handleEditMode} onView={handleViewMode}
onPanMode={() => setEditorTool("pan")} onEdit={handleEditMode}
onEditObjects={() => setEditorTool("edit")} onPanMode={() => setEditorTool("pan")}
/> onEditObjects={() => setEditorTool("edit")}
/>
) : null}
{!hasAnyMap || (mode === "setup" && mapState?.draft_map && !mapState?.published_map) ? ( {!hasAnyMap || (mode === "setup" && mapState?.draft_map && !mapState?.published_map) ? (
<LocationMapSetupPanel <LocationMapSetupPanel

1
frontend/tests/location-map-manager.spec.ts
View File

@ -263,6 +263,7 @@ test("admin creates a map from zones, saves a draft, and publishes it", async ({
await expect(page.getByText("Eastvale")).toBeVisible(); await expect(page.getByText("Eastvale")).toBeVisible();
await expect(page.getByRole("heading", { name: "No Map" })).toBeVisible(); await expect(page.getByRole("heading", { name: "No Map" })).toBeVisible();
await expect(page.getByRole("button", { name: "Create Map From Existing Zones" })).toBeVisible(); await expect(page.getByRole("button", { name: "Create Map From Existing Zones" })).toBeVisible();
await expect(page.getByLabel("Map controls")).toHaveCount(0);
await page.getByRole("button", { name: "Create Map From Existing Zones" }).click(); await page.getByRole("button", { name: "Create Map From Existing Zones" }).click();
await expect(page.getByText("Bakery")).toBeVisible(); await expect(page.getByText("Bakery")).toBeVisible();