From 17ba5f3043ebfe380102bf91a69e0695c00d0a72 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Wed, 29 Apr 2026 13:55:17 +0100
Subject: [PATCH] Bound Patrol configuration popover

---
 .../src/features/patrol/PatrolIntelligenceHeader.tsx   |  2 +-
 .../patrol/__tests__/patrolCommercialBoundary.test.ts  | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/frontend-modern/src/features/patrol/PatrolIntelligenceHeader.tsx b/frontend-modern/src/features/patrol/PatrolIntelligenceHeader.tsx
index 9ad783c33..cda8533b9 100644
--- a/frontend-modern/src/features/patrol/PatrolIntelligenceHeader.tsx
+++ b/frontend-modern/src/features/patrol/PatrolIntelligenceHeader.tsx
@@ -138,7 +138,7 @@ export function PatrolIntelligenceHeader(props: { state: PatrolIntelligenceState
             </button>
 
             <Show when={state.showAdvancedSettings()}>
-              <div class="fixed right-4 top-32 z-[9999] isolate w-[calc(100vw-2rem)] p-5 bg-surface rounded-md shadow-sm border border-border animate-slide-up transform origin-top-right sm:right-8 sm:top-[17.5rem] sm:w-[340px]">
+              <div class="fixed right-4 top-32 z-[9999] isolate max-h-[calc(100vh-10rem)] w-[calc(100vw-2rem)] overflow-y-auto rounded-md border border-border bg-surface p-5 shadow-sm animate-slide-up transform origin-top-right sm:right-8 sm:top-[13rem] sm:max-h-[36rem] sm:w-[340px]">
                 <div class="flex items-center justify-between mb-5 pb-3 border-b border-border-subtle">
                   <h4 class="text-base font-semibold tracking-tight text-base-content">
                     Patrol Configuration
diff --git a/frontend-modern/src/features/patrol/__tests__/patrolCommercialBoundary.test.ts b/frontend-modern/src/features/patrol/__tests__/patrolCommercialBoundary.test.ts
index e5a249b3a..779463644 100644
--- a/frontend-modern/src/features/patrol/__tests__/patrolCommercialBoundary.test.ts
+++ b/frontend-modern/src/features/patrol/__tests__/patrolCommercialBoundary.test.ts
@@ -28,4 +28,14 @@ describe('patrol commercial boundary', () => {
     expect(patrolIntelligenceHeaderSource).toContain('label="Run Every"');
     expect(patrolIntelligenceHeaderSource).not.toContain('<select');
   });
+
+  it('keeps the patrol configuration panel above lower page actions', () => {
+    expect(patrolIntelligenceHeaderSource).toContain('fixed right-4 top-32 z-[9999] isolate');
+    expect(patrolIntelligenceHeaderSource).toContain('max-h-[calc(100vh-10rem)]');
+    expect(patrolIntelligenceHeaderSource).toContain('sm:max-h-[36rem]');
+    expect(patrolIntelligenceHeaderSource).toContain('overflow-y-auto');
+    expect(patrolIntelligenceHeaderSource).toContain('sm:top-[13rem]');
+    expect(patrolIntelligenceHeaderSource).not.toContain('sm:top-[17.5rem]');
+    expect(patrolIntelligenceHeaderSource).not.toContain('disabled:opacity-70');
+  });
 });