From 51dd2d4fcf02908eb801ab8dc2d92f33b3e3ba84 Mon Sep 17 00:00:00 2001
From: kurt-croix <kurt-croix@users.noreply.github.com>
Date: Mon, 4 May 2026 09:39:13 -0500
Subject: [PATCH] fix: select dropdowns unresponsive on mobile inside Sheet

Root cause: SelectContent z-index (z-50) was below Sheet overlay
(z-[2000]), so the dropdown rendered behind the sheet on mobile.

- Bump SelectContent z-index to z-[3000] to render above Sheet
- Use position='item-aligned' in ReportForm selects for better
  mobile behavior
---
 src/components/ReportForm.tsx | 4 ++--
 src/components/ui/select.tsx  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/components/ReportForm.tsx b/src/components/ReportForm.tsx
index fd59e91..af946de 100644
--- a/src/components/ReportForm.tsx
+++ b/src/components/ReportForm.tsx
@@ -375,7 +375,7 @@ export function ReportForm({ selectedLocation, onLocationSelect, onReportCreated
             <SelectTrigger className="mt-1 h-9 text-sm">
               <SelectValue placeholder="Select" />
             </SelectTrigger>
-            <SelectContent>
+            <SelectContent position="item-aligned">
               {HAZARD_TYPES.map((type) => (
                 <SelectItem key={type.value} value={type.value}>
                   {type.icon} {type.label}
@@ -390,7 +390,7 @@ export function ReportForm({ selectedLocation, onLocationSelect, onReportCreated
             <SelectTrigger className="mt-1 h-9 text-sm">
               <SelectValue placeholder="Select" />
             </SelectTrigger>
-            <SelectContent>
+            <SelectContent position="item-aligned">
               {SEVERITY_LEVELS.map((level) => (
                 <SelectItem key={level.value} value={level.value}>
                   <span className="flex items-center gap-2">
diff --git a/src/components/ui/select.tsx b/src/components/ui/select.tsx
index ea85bc1..93cd424 100644
--- a/src/components/ui/select.tsx
+++ b/src/components/ui/select.tsx
@@ -73,7 +73,7 @@ const SelectContent = React.forwardRef<
     <SelectPrimitive.Content
       ref={ref}
       className={cn(
-        "relative z-50 max-h-96 min-w-[8rem] overflow-hidden rounded-md border bg-popover text-popover-foreground shadow-md data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 data-[side=bottom]:slide-in-from-top-2 data-[side=left]:slide-in-from-right-2 data-[side=right]:slide-in-from-left-2 data-[side=top]:slide-in-from-bottom-2",
+        "relative z-[3000] max-h-96 min-w-[8rem] overflow-hidden rounded-md border bg-popover text-popover-foreground shadow-md data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 data-[side=bottom]:slide-in-from-top-2 data-[side=left]:slide-in-from-right-2 data-[side=right]:slide-in-from-left-2 data-[side=top]:slide-in-from-bottom-2",
         position === "popper" &&
           "data-[side=bottom]:translate-y-1 data-[side=left]:-translate-x-1 data-[side=right]:translate-x-1 data-[side=top]:-translate-y-1",
         className