From c17524bc3cbfbccf84654491e6463927c9dd2b6c Mon Sep 17 00:00:00 2001
From: Josh Hawkins <32435876+hawkeye217@users.noreply.github.com>
Date: Sat, 14 Sep 2024 15:08:46 -0500
Subject: [PATCH] Explore tweaks (#13744)

* Fix loading indicator and explore view when entering a search term

* don't hide iOS video mute button

* use searchTerm instead of search for trigger
---
 .../overlay/detail/SearchDetailDialog.tsx        | 11 ++++++++---
 web/src/views/search/SearchView.tsx              | 16 +++++++++-------
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/web/src/components/overlay/detail/SearchDetailDialog.tsx b/web/src/components/overlay/detail/SearchDetailDialog.tsx
index 441e48df6..d0504fb76 100644
--- a/web/src/components/overlay/detail/SearchDetailDialog.tsx
+++ b/web/src/components/overlay/detail/SearchDetailDialog.tsx
@@ -350,7 +350,7 @@ function ObjectDetailsTab({
       <div className="flex flex-col gap-1.5">
         <div className="text-sm text-primary/40">Description</div>
         <Textarea
-          className="md:h-64"
+          className="h-64"
           placeholder="Description of the tracked object"
           value={desc}
           onChange={(e) => setDesc(e.target.value)}
@@ -447,8 +447,13 @@ function VideoTab({ search, config }: VideoTabProps) {
           />
         </div>
       </div>
-      {!isLoading && (
-        <div className="absolute right-2 top-2 flex items-center">
+      {!isLoading && reviewItem && (
+        <div
+          className={cn(
+            "absolute top-2 flex items-center",
+            isIOS ? "right-8" : "right-2",
+          )}
+        >
           <Tooltip>
             <TooltipTrigger>
               <Chip
diff --git a/web/src/views/search/SearchView.tsx b/web/src/views/search/SearchView.tsx
index 42c8c0833..e031d211c 100644
--- a/web/src/views/search/SearchView.tsx
+++ b/web/src/views/search/SearchView.tsx
@@ -243,8 +243,8 @@ export default function SearchView({
 
         {uniqueResults?.length == 0 &&
           isLoading &&
-          searchFilter &&
-          Object.keys(searchFilter).length !== 0 && (
+          (searchTerm ||
+            (searchFilter && Object.keys(searchFilter).length !== 0)) && (
             <ActivityIndicator className="absolute left-1/2 top-1/2 -translate-x-1/2 -translate-y-1/2" />
           )}
 
@@ -319,11 +319,13 @@ export default function SearchView({
           </>
         )}
       </div>
-      {searchFilter && Object.keys(searchFilter).length === 0 && (
-        <div className="scrollbar-container flex size-full flex-col overflow-y-auto">
-          <ExploreView onSelectSearch={onSelectSearch} />
-        </div>
-      )}
+      {searchFilter &&
+        Object.keys(searchFilter).length === 0 &&
+        !searchTerm && (
+          <div className="scrollbar-container flex size-full flex-col overflow-y-auto">
+            <ExploreView onSelectSearch={onSelectSearch} />
+          </div>
+        )}
     </div>
   );
 }