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

fix: trim map message empty state

Browse Source
This commit is contained in:
Nico 2026-06-17 10:36:26 -07:00
parent 0580914516
commit 815cd1b37f
3 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -1,6 +1,6 @@
import LocationMapTopbar from "./LocationMapTopbar"; import LocationMapTopbar from "./LocationMapTopbar";
export function LocationMapMessageState({ message, location, status = "Loading", onBack }) { export function LocationMapMessageState({ message = "", location, status = "Loading", onBack }) {
const showTopbar = Boolean(location && onBack); const showTopbar = Boolean(location && onBack);
return ( return (
@ -11,7 +11,7 @@ export function LocationMapMessageState({ message, location, status = "Loading",
<main className="location-map-workspace location-map-message-workspace"> <main className="location-map-workspace location-map-message-workspace">
<section className="location-map-setup location-map-message-card" aria-live="polite"> <section className="location-map-setup location-map-message-card" aria-live="polite">
<h2>{status}</h2> <h2>{status}</h2>
<p className="location-map-state-message">{message}</p> {message ? <p className="location-map-state-message">{message}</p> : null}
</section> </section>
</main> </main>
</div> </div>

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

@ -344,7 +344,6 @@ export default function LocationMapManager() {
return ( return (
<LocationMapMessageState <LocationMapMessageState
status="Select Household" status="Select Household"
message="Select a household to manage maps."
/> />
); );
} }

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

@ -481,8 +481,8 @@ test("map message prompts for a household with compact direct text", async ({ pa
await expect(page.getByRole("heading", { name: "Select Household" })).toBeVisible(); await expect(page.getByRole("heading", { name: "Select Household" })).toBeVisible();
const messageCard = page.locator(".location-map-message-card"); const messageCard = page.locator(".location-map-message-card");
await expect(messageCard.getByText("Select a household to manage maps.")).toBeVisible(); await expect(messageCard.getByText("Select a household to manage maps.")).toHaveCount(0);
await expect(messageCard.locator(".location-map-state-message")).toHaveText("Select a household to manage maps."); await expect(messageCard.locator(".location-map-state-message")).toHaveCount(0);
await expect(messageCard.getByRole("group", { name: "Map message details" })).toHaveCount(0); await expect(messageCard.getByRole("group", { name: "Map message details" })).toHaveCount(0);
await expect(messageCard.locator(".location-map-setup-row", { hasText: "Message" })).toHaveCount(0); await expect(messageCard.locator(".location-map-setup-row", { hasText: "Message" })).toHaveCount(0);
await expect(messageCard.locator(".location-map-setup-row", { hasText: "Status" })).toHaveCount(0); await expect(messageCard.locator(".location-map-setup-row", { hasText: "Status" })).toHaveCount(0);