From 32c7669b28148c04e1611ce30c840a91f4a36a27 Mon Sep 17 00:00:00 2001
From: Josh Hawkins <32435876+hawkeye217@users.noreply.github.com>
Date: Wed, 25 Sep 2024 13:45:42 -0500
Subject: [PATCH] Search UI tweaks (#13965)

* Prevent keyboard shortcuts from running when input is focused

* fix reset button and update time pickers when using input

* simplify css

* consistent button order and spacing
---
 .../components/filter/SearchFilterGroup.tsx   | 26 +++++++--------
 web/src/components/input/InputWithTags.tsx    | 32 +++++++++++--------
 web/src/components/ui/calendar-range.tsx      | 22 ++++++-------
 web/src/hooks/use-keyboard-listener.tsx       |  5 +--
 web/src/views/search/SearchView.tsx           | 14 ++++++--
 5 files changed, 56 insertions(+), 43 deletions(-)

diff --git a/web/src/components/filter/SearchFilterGroup.tsx b/web/src/components/filter/SearchFilterGroup.tsx
index 54618aefc..09d38c442 100644
--- a/web/src/components/filter/SearchFilterGroup.tsx
+++ b/web/src/components/filter/SearchFilterGroup.tsx
@@ -430,6 +430,13 @@ function TimeRangeFilterButton({
   const formattedSelectedAfter = useFormattedHour(config, selectedAfterHour);
   const formattedSelectedBefore = useFormattedHour(config, selectedBeforeHour);
 
+  useEffect(() => {
+    setSelectedAfterHour(afterHour);
+    setSelectedBeforeHour(beforeHour);
+    // only refresh when state changes
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [timeRange]);
+
   const trigger = (
     <Button
       size="sm"
@@ -447,18 +454,8 @@ function TimeRangeFilterButton({
     </Button>
   );
   const content = (
-    <div
-      className={cn(
-        "scrollbar-container flex h-auto max-h-[80dvh] flex-col overflow-y-auto overflow-x-hidden",
-        isDesktop ? "w-64" : "w-full gap-2 pt-2",
-      )}
-    >
-      <div
-        className={cn(
-          "mt-3 flex w-full items-center rounded-lg text-secondary-foreground",
-          isDesktop ? "mx-6 gap-2 px-2" : "justify-center gap-2",
-        )}
-      >
+    <div className="scrollbar-container h-auto max-h-[80dvh] overflow-y-auto overflow-x-hidden">
+      <div className="my-5 flex flex-row items-center justify-center gap-2">
         <Popover
           open={startOpen}
           onOpenChange={(open) => {
@@ -480,7 +477,7 @@ function TimeRangeFilterButton({
               {formattedSelectedAfter}
             </Button>
           </PopoverTrigger>
-          <PopoverContent className="flex flex-col items-center">
+          <PopoverContent className="flex flex-row items-center justify-center">
             <input
               className="text-md mx-4 w-full border border-input bg-background p-1 text-secondary-foreground hover:bg-accent hover:text-accent-foreground dark:[color-scheme:dark]"
               id="startTime"
@@ -534,8 +531,8 @@ function TimeRangeFilterButton({
             />
           </PopoverContent>
         </Popover>
-        <DropdownMenuSeparator />
       </div>
+      <DropdownMenuSeparator />
       <div className="flex items-center justify-evenly p-2">
         <Button
           variant="select"
@@ -558,6 +555,7 @@ function TimeRangeFilterButton({
           onClick={() => {
             setSelectedAfterHour(DEFAULT_TIME_RANGE_AFTER);
             setSelectedBeforeHour(DEFAULT_TIME_RANGE_BEFORE);
+            updateTimeRange(undefined);
           }}
         >
           Reset
diff --git a/web/src/components/input/InputWithTags.tsx b/web/src/components/input/InputWithTags.tsx
index 9d4632885..3e40b36c8 100644
--- a/web/src/components/input/InputWithTags.tsx
+++ b/web/src/components/input/InputWithTags.tsx
@@ -1,4 +1,4 @@
-import { useState, useRef, useEffect, useCallback } from "react";
+import React, { useState, useRef, useEffect, useCallback } from "react";
 import {
   LuX,
   LuFilter,
@@ -45,6 +45,8 @@ import useSWR from "swr";
 import { FrigateConfig } from "@/types/frigateConfig";
 
 type InputWithTagsProps = {
+  inputFocused: boolean;
+  setInputFocused: React.Dispatch<React.SetStateAction<boolean>>;
   filters: SearchFilter;
   setFilters: (filter: SearchFilter) => void;
   search: string;
@@ -55,6 +57,8 @@ type InputWithTagsProps = {
 };
 
 export default function InputWithTags({
+  inputFocused,
+  setInputFocused,
   filters,
   setFilters,
   search,
@@ -69,7 +73,6 @@ export default function InputWithTags({
   const [currentFilterType, setCurrentFilterType] = useState<FilterType | null>(
     null,
   );
-  const [inputFocused, setInputFocused] = useState(false);
   const [isSimilaritySearch, setIsSimilaritySearch] = useState(false);
   const inputRef = useRef<HTMLInputElement>(null);
   const commandRef = useRef<HTMLDivElement>(null);
@@ -381,7 +384,7 @@ export default function InputWithTags({
 
   const handleInputFocus = useCallback(() => {
     setInputFocused(true);
-  }, []);
+  }, [setInputFocused]);
 
   const handleClearInput = useCallback(() => {
     setInputFocused(false);
@@ -392,16 +395,19 @@ export default function InputWithTags({
     setFilters({});
     setCurrentFilterType(null);
     setIsSimilaritySearch(false);
-  }, [setFilters, resetSuggestions, setSearch]);
+  }, [setFilters, resetSuggestions, setSearch, setInputFocused]);
 
-  const handleInputBlur = useCallback((e: React.FocusEvent) => {
-    if (
-      commandRef.current &&
-      !commandRef.current.contains(e.relatedTarget as Node)
-    ) {
-      setInputFocused(false);
-    }
-  }, []);
+  const handleInputBlur = useCallback(
+    (e: React.FocusEvent) => {
+      if (
+        commandRef.current &&
+        !commandRef.current.contains(e.relatedTarget as Node)
+      ) {
+        setInputFocused(false);
+      }
+    },
+    [setInputFocused],
+  );
 
   const handleSuggestionClick = useCallback(
     (suggestion: string) => {
@@ -449,7 +455,7 @@ export default function InputWithTags({
       setInputFocused(false);
       inputRef?.current?.blur();
     },
-    [setSearch],
+    [setSearch, setInputFocused],
   );
 
   const handleInputKeyDown = useCallback(
diff --git a/web/src/components/ui/calendar-range.tsx b/web/src/components/ui/calendar-range.tsx
index 395cfb779..e3db0d251 100644
--- a/web/src/components/ui/calendar-range.tsx
+++ b/web/src/components/ui/calendar-range.tsx
@@ -414,17 +414,7 @@ export function DateRangePicker({
           </div>
         )}
       </div>
-      <div className="flex justify-center gap-2 py-2 pr-4">
-        <Button
-          onClick={() => {
-            setIsOpen(false);
-            resetValues();
-            onReset?.();
-          }}
-          variant="ghost"
-        >
-          Reset
-        </Button>
+      <div className="mx-auto flex w-64 items-center justify-evenly gap-2 py-2">
         <Button
           variant="select"
           onClick={() => {
@@ -439,6 +429,16 @@ export function DateRangePicker({
         >
           Apply
         </Button>
+        <Button
+          onClick={() => {
+            setIsOpen(false);
+            resetValues();
+            onReset?.();
+          }}
+          variant="ghost"
+        >
+          Reset
+        </Button>
       </div>
     </div>
   );
diff --git a/web/src/hooks/use-keyboard-listener.tsx b/web/src/hooks/use-keyboard-listener.tsx
index ad9462a05..ad776b303 100644
--- a/web/src/hooks/use-keyboard-listener.tsx
+++ b/web/src/hooks/use-keyboard-listener.tsx
@@ -10,6 +10,7 @@ export type KeyModifiers = {
 export default function useKeyboardListener(
   keys: string[],
   listener: (key: string | null, modifiers: KeyModifiers) => void,
+  preventDefault: boolean = true,
 ) {
   const keyDownListener = useCallback(
     (e: KeyboardEvent) => {
@@ -25,13 +26,13 @@ export default function useKeyboardListener(
       };
 
       if (keys.includes(e.key)) {
-        e.preventDefault();
+        if (preventDefault) e.preventDefault();
         listener(e.key, modifiers);
       } else if (e.key === "Shift" || e.key === "Control" || e.key === "Meta") {
         listener(null, modifiers);
       }
     },
-    [keys, listener],
+    [keys, listener, preventDefault],
   );
 
   const keyUpListener = useCallback(
diff --git a/web/src/views/search/SearchView.tsx b/web/src/views/search/SearchView.tsx
index 28f9ed579..3677612c7 100644
--- a/web/src/views/search/SearchView.tsx
+++ b/web/src/views/search/SearchView.tsx
@@ -209,9 +209,11 @@ export default function SearchView({
 
   // keyboard listener
 
+  const [inputFocused, setInputFocused] = useState(false);
+
   const onKeyboardShortcut = useCallback(
     (key: string | null, modifiers: KeyModifiers) => {
-      if (!modifiers.down || !uniqueResults) {
+      if (!modifiers.down || !uniqueResults || inputFocused) {
         return;
       }
 
@@ -236,10 +238,14 @@ export default function SearchView({
           break;
       }
     },
-    [uniqueResults],
+    [uniqueResults, inputFocused],
   );
 
-  useKeyboardListener(["ArrowLeft", "ArrowRight"], onKeyboardShortcut);
+  useKeyboardListener(
+    ["ArrowLeft", "ArrowRight"],
+    onKeyboardShortcut,
+    !inputFocused,
+  );
 
   // scroll into view
 
@@ -310,6 +316,8 @@ export default function SearchView({
         {config?.semantic_search?.enabled && (
           <div className={cn("z-[41] w-full lg:absolute lg:top-0 lg:w-1/3")}>
             <InputWithTags
+              inputFocused={inputFocused}
+              setInputFocused={setInputFocused}
               filters={searchFilter ?? {}}
               setFilters={setSearchFilter}
               search={search}