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

Add store location map manager #20

Open
nalalangan wants to merge 206 commits from feature/location-map-manager into feature/store-selector-modal
pull from: feature/location-map-manager
merge into: nalalangan:feature/store-selector-modal
Conversation 0 Commits 206 Files Changed 47 +12 -5
3 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 0361420813 - Show all commits

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

@ -115,14 +115,14 @@ export default function LocationMapBottomSheet({
<div className="location-map-mobile-tool-switch" aria-label="Edit tool"> <div className="location-map-mobile-tool-switch" aria-label="Edit tool">
<button <button
type="button" type="button"
className={editorTool === "pan" ? "active" : ""} className={[editorTool === "pan" ? "active" : "", "is-pan"].filter(Boolean).join(" ")}
onClick={onPanMode} onClick={onPanMode}
> >
Pan Pan
</button> </button>
<button <button
type="button" type="button"
className={editorTool === "edit" ? "active" : ""} className={[editorTool === "edit" ? "active" : "", "is-edit"].filter(Boolean).join(" ")}
onClick={onEditObjects} onClick={onEditObjects}
> >
Objects Objects

7
frontend/src/styles/pages/LocationMapManager.css
View File

@ -415,7 +415,12 @@
font-weight: 800; font-weight: 800;
} }
.location-map-mobile-tool-switch button.active { .location-map-mobile-tool-switch button.is-pan.active {
border-color: rgba(245, 158, 11, 0.78);
background: rgba(245, 158, 11, 0.22);
}
.location-map-mobile-tool-switch button.is-edit.active {
border-color: rgb(45, 212, 191); border-color: rgb(45, 212, 191);
background: rgba(20, 184, 166, 0.24); background: rgba(20, 184, 166, 0.24);
} }

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

@ -521,8 +521,9 @@ test("mobile editor uses bottom sheet controls instead of desktop object toolbar
await expect(page.getByLabel("Map controls")).toBeVisible(); await expect(page.getByLabel("Map controls")).toBeVisible();
await expect(page.locator(".location-map-tool-buttons")).toBeHidden(); await expect(page.locator(".location-map-tool-buttons")).toBeHidden();
await expect(page.locator(".location-map-mobile-tool-switch")).toBeVisible(); await expect(page.locator(".location-map-mobile-tool-switch")).toBeVisible();
await expect(page.getByRole("button", { name: "Pan" })).toBeVisible(); await expect(page.getByRole("button", { name: "Pan" })).toHaveClass(/active/);
await expect(page.getByRole("button", { name: "Objects" })).toBeVisible(); await expect(page.getByRole("button", { name: "Pan" })).toHaveClass(/is-pan/);
await expect(page.getByRole("button", { name: "Objects" })).toHaveClass(/is-edit/);
await expect(page.locator(".location-map-pin")).toHaveCount(0); await expect(page.locator(".location-map-pin")).toHaveCount(0);
const canvasBox = await page.locator(".location-map-canvas-shell").boundingBox(); const canvasBox = await page.locator(".location-map-canvas-shell").boundingBox();
@ -534,6 +535,7 @@ test("mobile editor uses bottom sheet controls instead of desktop object toolbar
expect(sheetBox.height).toBeLessThanOrEqual(360); expect(sheetBox.height).toBeLessThanOrEqual(360);
await page.getByRole("button", { name: "Objects" }).click(); await page.getByRole("button", { name: "Objects" }).click();
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();
}); });