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

fix: size map item actions for touch

Browse Source
This commit is contained in:
Nico 2026-06-04 20:35:38 -07:00
parent 40776821a6
commit 05c9577c71
2 changed files with 22 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -798,7 +798,7 @@
}
.location-map-inline-state button {
min-height: 34px;
min-height: 40px;
min-width: 64px;
padding: 0.35rem 0.6rem;
border: 1px solid var(--color-border-light);
@ -817,7 +817,7 @@
.location-map-zone-items li,
.location-map-unmapped-list li {
min-height: 34px;
min-height: 40px;
display: flex;
align-items: center;
justify-content: space-between;
@ -837,7 +837,7 @@
.location-map-item-button {
width: 100%;
min-height: 34px;
min-height: 40px;
display: flex;
align-items: center;
justify-content: space-between;
@ -909,7 +909,7 @@
.location-map-list-more-button {
width: 100%;
min-height: 34px;
min-height: 40px;
padding: 0.35rem 0.55rem;
border: 0;
background: transparent;

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

@ -2909,6 +2909,15 @@ test("viewer shows a compact overflow cue for long unmapped lists", async ({ pag
await expect(page.getByText("unmapped item 8")).toBeVisible();
await expect(page.getByText("unmapped item 9")).toHaveCount(0);
await expect(page.getByRole("button", { name: "Show 3 more unmapped items" })).toHaveText("+3 more");
const firstUnmappedButtonBox = await page.getByRole("button", { name: "Mark unmapped item 1 bought" }).boundingBox();
const moreUnmappedButtonBox = await page.getByRole("button", { name: "Show 3 more unmapped items" }).boundingBox();
expect(firstUnmappedButtonBox).not.toBeNull();
expect(moreUnmappedButtonBox).not.toBeNull();
if (!firstUnmappedButtonBox || !moreUnmappedButtonBox) {
throw new Error("Unmapped item controls were not measurable");
}
expect(firstUnmappedButtonBox.height).toBeGreaterThanOrEqual(40);
expect(moreUnmappedButtonBox.height).toBeGreaterThanOrEqual(40);
await page.getByRole("button", { name: "Show 3 more unmapped items" }).click();
@ -3722,6 +3731,15 @@ test("viewer handles empty and many-item zone panels without default pins", asyn
await expect(page.locator(".location-map-zone-items .location-map-item-row").first()).toContainText("bulk item 1");
await expect(page.getByText("bulk item 9")).toHaveCount(0);
await expect(page.getByRole("button", { name: "Show 6 more zone items" })).toHaveText("+6 more");
const firstZoneItemButtonBox = await page.getByRole("button", { name: "Mark bulk item 1 bought" }).boundingBox();
const moreZoneItemsButtonBox = await page.getByRole("button", { name: "Show 6 more zone items" }).boundingBox();
expect(firstZoneItemButtonBox).not.toBeNull();
expect(moreZoneItemsButtonBox).not.toBeNull();
if (!firstZoneItemButtonBox || !moreZoneItemsButtonBox) {
throw new Error("Zone item controls were not measurable");
}
expect(firstZoneItemButtonBox.height).toBeGreaterThanOrEqual(40);
expect(moreZoneItemsButtonBox.height).toBeGreaterThanOrEqual(40);
await page.getByRole("button", { name: "Show 6 more zone items" }).click();