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

fix: close map object settings

Browse Source
This commit is contained in:
Nico 2026-06-03 15:12:49 -07:00
parent bc6cfb97a1
commit 757a5fb5b7
3 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
frontend/src/components/maps/LocationMapBottomSheet.jsx
View File

@ -42,6 +42,7 @@ export default function LocationMapBottomSheet({
onEditMode, onEditMode,
onObjectField, onObjectField,
onZoneLinkChange, onZoneLinkChange,
onClearSelection,
onDuplicateObject, onDuplicateObject,
onDeleteObject, onDeleteObject,
}) { }) {
@ -234,6 +235,7 @@ export default function LocationMapBottomSheet({
</label> </label>
</div> </div>
<div className="location-map-editor-actions"> <div className="location-map-editor-actions">
<button type="button" onClick={onClearSelection}>Done</button>
<button type="button" onClick={onDuplicateObject}>Duplicate</button> <button type="button" onClick={onDuplicateObject}>Duplicate</button>
<button type="button" className="danger" onClick={onDeleteObject}>Delete</button> <button type="button" className="danger" onClick={onDeleteObject}>Delete</button>
</div> </div>

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

@ -529,6 +529,7 @@ export default function LocationMapManager() {
onEditMode={handleEditMode} onEditMode={handleEditMode}
onObjectField={handleObjectField} onObjectField={handleObjectField}
onZoneLinkChange={handleZoneLinkChange} onZoneLinkChange={handleZoneLinkChange}
onClearSelection={() => setSelectedObjectKey(null)}
onDuplicateObject={handleDuplicateObject} onDuplicateObject={handleDuplicateObject}
onDeleteObject={requestDeleteObject} onDeleteObject={requestDeleteObject}
/> />

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

@ -914,6 +914,10 @@ test("mobile editor uses bottom sheet controls instead of desktop object toolbar
await expect(page.getByRole("button", { name: "Objects" })).toHaveClass(/active/); await expect(page.getByRole("button", { name: "Objects" })).toHaveClass(/active/);
await page.locator(".location-map-object", { hasText: "Bakery" }).locator("rect").first().click(); await page.locator(".location-map-object", { hasText: "Bakery" }).locator("rect").first().click();
await expect(page.getByRole("textbox", { name: "Label" })).toBeVisible(); await expect(page.getByRole("textbox", { name: "Label" })).toBeVisible();
await expect(page.locator(".location-map-sheet-header strong")).toHaveText("Bakery");
await page.getByRole("button", { name: "Done" }).click();
await expect(page.getByRole("textbox", { name: "Label" })).toHaveCount(0);
await expect(page.locator(".location-map-sheet-header strong")).toHaveText("Edit Objects");
}); });
test("members can view a published map but cannot edit it", async ({ page }) => { test("members can view a published map but cannot edit it", async ({ page }) => {