From 3816f255a06a483fe0afa50dbfd874083f449e28 Mon Sep 17 00:00:00 2001
From: Nico <Nico>
Date: Sun, 31 May 2026 19:21:08 -0700
Subject: [PATCH] fix: compact store delete count label

---
 docs/guides/management-modal-patterns.md                  | 2 +-
 .../src/components/manage/StoreAvailableItemsManager.jsx  | 2 +-
 frontend/tests/available-items-catalog.spec.ts            | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/guides/management-modal-patterns.md b/docs/guides/management-modal-patterns.md
index 61060b8..640686d 100644
--- a/docs/guides/management-modal-patterns.md
+++ b/docs/guides/management-modal-patterns.md
@@ -23,7 +23,7 @@ Use this guide for modals that manage scoped lists of app-owned records, such as
 
 ## Bulk Delete Pattern
 - Show a `Delete Items` button above the list for users with delete permission.
-- Clicking `Delete Items` enters delete mode, clears any previous selection, and changes the button to `Confirm Delete (# selected)`.
+- Clicking `Delete Items` enters delete mode, clears any previous selection, and changes the button to `Confirm Delete (#)`.
 - Show a `Cancel` button while delete mode is active.
 - Disable confirm while zero items are selected.
 - Clicking confirm opens `ConfirmSlideModal`; only the slide confirmation performs the mutation.
diff --git a/frontend/src/components/manage/StoreAvailableItemsManager.jsx b/frontend/src/components/manage/StoreAvailableItemsManager.jsx
index bcd6c42..2661193 100644
--- a/frontend/src/components/manage/StoreAvailableItemsManager.jsx
+++ b/frontend/src/components/manage/StoreAvailableItemsManager.jsx
@@ -246,7 +246,7 @@ export default function StoreAvailableItemsManager({ householdId, store, isAdmin
                   disabled={deleteMode ? selectedDeleteCount === 0 : loading}
                   onClick={deleteMode ? confirmSelectedDelete : startDeleteMode}
                 >
-                  {deleteMode ? `Confirm Delete (${selectedDeleteCount} selected)` : "Delete Items"}
+                  {deleteMode ? `Confirm Delete (${selectedDeleteCount})` : "Delete Items"}
                 </button>
                 {deleteMode ? (
                   <button
diff --git a/frontend/tests/available-items-catalog.spec.ts b/frontend/tests/available-items-catalog.spec.ts
index c6b294d..5887228 100644
--- a/frontend/tests/available-items-catalog.spec.ts
+++ b/frontend/tests/available-items-catalog.spec.ts
@@ -149,19 +149,19 @@ test("manage stores opens a modal to edit and delete household store items", asy
   await expect(managerModal.getByText("produce | Fruits | Produce & Fresh Vegetables")).toHaveCount(0);
 
   await managerModal.getByRole("button", { name: "Delete Items" }).click();
-  await expect(managerModal.getByRole("button", { name: "Confirm Delete (0 selected)" })).toBeDisabled();
+  await expect(managerModal.getByRole("button", { name: "Confirm Delete (0)" })).toBeDisabled();
   await expect(managerModal.getByRole("button", { name: "Cancel" })).toBeVisible();
 
   await milkRow.click();
-  await expect(managerModal.getByRole("button", { name: "Confirm Delete (1 selected)" })).toBeEnabled();
+  await expect(managerModal.getByRole("button", { name: "Confirm Delete (1)" })).toBeEnabled();
   await expect(milkRow).toHaveClass(/is-selected/);
 
   await milkRow.click();
-  await expect(managerModal.getByRole("button", { name: "Confirm Delete (0 selected)" })).toBeDisabled();
+  await expect(managerModal.getByRole("button", { name: "Confirm Delete (0)" })).toBeDisabled();
   await expect(milkRow).not.toHaveClass(/is-selected/);
 
   await milkRow.click();
-  await managerModal.getByRole("button", { name: "Confirm Delete (1 selected)" }).click();
+  await managerModal.getByRole("button", { name: "Confirm Delete (1)" }).click();
   const confirmModal = page.locator(".confirm-slide-modal");
   await expect(confirmModal).toBeVisible();
   await expect(confirmModal.getByText("Delete milk?")).toBeVisible();