From eb5f75bc9df330bfb9f53d0b47572c076ca6d0e5 Mon Sep 17 00:00:00 2001
From: austin <austin@thebennett.net>
Date: Sun, 26 Jul 2026 15:16:09 -0500
Subject: [PATCH] Notes move both ways, and prove a deleted table takes its
 cells with it
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Two small things found reading it back: a note could be moved up but not down, and nothing exercised
`remove` against the real foreign keys — a delete that strands its children fails in Postgres rather
than in memory, and it's a button on the screen.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 frontend/src/components/admin/Catering.tsx       |  9 +++++++++
 .../com/itsthevine/web/CateringMenuTest.java     | 16 ++++++++++++++++
 2 files changed, 25 insertions(+)

diff --git a/frontend/src/components/admin/Catering.tsx b/frontend/src/components/admin/Catering.tsx
index ec8b3a8..653ff78 100644
--- a/frontend/src/components/admin/Catering.tsx
+++ b/frontend/src/components/admin/Catering.tsx
@@ -87,6 +87,15 @@ const Notes = ({
             >
               <Icon d={ARROW_UP} />
             </button>
+            <button
+              type="button"
+              className={iconButton}
+              disabled={disabled || i === notes.length - 1}
+              onClick={() => onChange(shift(notes, i, 1))}
+              aria-label="Move note down"
+            >
+              <Icon d={ARROW_DOWN} />
+            </button>
             <button
               type="button"
               className={iconButton}
diff --git a/src/test/java/com/itsthevine/web/CateringMenuTest.java b/src/test/java/com/itsthevine/web/CateringMenuTest.java
index e3588af..3806fd2 100644
--- a/src/test/java/com/itsthevine/web/CateringMenuTest.java
+++ b/src/test/java/com/itsthevine/web/CateringMenuTest.java
@@ -204,6 +204,22 @@ class CateringMenuTest {
                 .contains("Holiday boxes");
     }
 
+    @Test
+    void deletingATableTakesItsColumnsLinesAndCellsWithIt() {
+        CateringMenu.PackageView weddings = catering.everything().packages().get(2);
+
+        catering.remove(weddings.id());
+        // Flushed on purpose: a delete that leaves its children behind fails against the real foreign
+        // keys, not in memory, and this one is a button on the admin screen.
+        entityManager.flush();
+        entityManager.clear();
+
+        assertThat(catering.everything().packages()).extracting(CateringMenu.PackageView::name)
+                .containsExactly("Office", "Parties");
+        // The page's own terms outlive any one table.
+        assertThat(catering.menu().notes()).hasSize(2);
+    }
+
     @Test
     void putsTheTablesWhereTheEditorLeftThem() {
         List<CateringMenu.PackageView> tables = catering.everything().packages();