From a09cc787fb280c94eb42c4d3ac44fbf7d15979ba Mon Sep 17 00:00:00 2001
From: Nicolas Mowen <nickmowen213@gmail.com>
Date: Thu, 12 Sep 2024 08:16:41 -0600
Subject: [PATCH] Handle case when user switches to element missing the
 previously selected tab

---
 web/src/components/overlay/detail/SearchDetailDialog.tsx | 9 ++++++++-
 web/src/components/overlay/dialog/FrigatePlusDialog.tsx  | 2 +-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/web/src/components/overlay/detail/SearchDetailDialog.tsx b/web/src/components/overlay/detail/SearchDetailDialog.tsx
index e2c9e93fa..2586a9ea4 100644
--- a/web/src/components/overlay/detail/SearchDetailDialog.tsx
+++ b/web/src/components/overlay/detail/SearchDetailDialog.tsx
@@ -35,7 +35,6 @@ import { cn } from "@/lib/utils";
 import ActivityIndicator from "@/components/indicators/activity-indicator";
 import { ASPECT_VERTICAL_LAYOUT, ASPECT_WIDE_LAYOUT } from "@/types/record";
 import { FaImage, FaRegListAlt, FaVideo } from "react-icons/fa";
-import FrigatePlusIcon from "@/components/icons/FrigatePlusIcon";
 import { FaRotate } from "react-icons/fa6";
 import ObjectLifecycle from "./ObjectLifecycle";
 
@@ -87,6 +86,14 @@ export default function SearchDetailDialog({
     return views;
   }, [config, search]);
 
+  useEffect(() => {
+    if (!searchTabs.includes(page)) {
+      setPage("details");
+    }
+    // we know that these deps are correct
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [searchTabs]);
+
   if (!search) {
     return;
   }
diff --git a/web/src/components/overlay/dialog/FrigatePlusDialog.tsx b/web/src/components/overlay/dialog/FrigatePlusDialog.tsx
index e96b53d63..0c4801f8e 100644
--- a/web/src/components/overlay/dialog/FrigatePlusDialog.tsx
+++ b/web/src/components/overlay/dialog/FrigatePlusDialog.tsx
@@ -79,7 +79,7 @@ export function FrigatePlusDialog({
 
   const content = (
     <TransformWrapper minScale={1.0} wheel={{ smoothStep: 0.005 }}>
-      <DialogHeader>
+      <DialogHeader className={state == "submitted" ? "sr-only" : ""}>
         <DialogTitle>Submit To Frigate+</DialogTitle>
         <DialogDescription>
           Objects in locations you want to avoid are not false positives.