Improve filters UI and add audio labels

web/src/components/filter/ReviewFilterGroup.tsx

@@ -1,4 +1,4 @@
-import { LuCheck, LuVideo } from "react-icons/lu";
+import { LuCheck } from "react-icons/lu";
 import { Button } from "../ui/button";
 import { Popover, PopoverContent, PopoverTrigger } from "../ui/popover";
 import useSWR from "swr";
@@ -41,14 +41,21 @@ export default function ReviewFilterGroup({
     const cameras = filter?.cameras || Object.keys(config.cameras);
 
     cameras.forEach((camera) => {
-      config.cameras[camera].objects.track.forEach((label) => {
+      const cameraConfig = config.cameras[camera];
+      cameraConfig.objects.track.forEach((label) => {
         if (!ATTRIBUTES.includes(label)) {
           labels.add(label);
         }
       });
+
+      if (cameraConfig.audio.enabled_in_config) {
+        cameraConfig.audio.listen.forEach((label) => {
+          labels.add(label);
+        });
+      }
     });
 
-    return [...labels];
+    return [...labels].sort();
   }, [config, filter]);
 
   const filterValues = useMemo(
@@ -125,16 +132,20 @@ function CamerasFilterButton({
   selectedCameras,
   updateCameraFilter,
 }: CameraFilterButtonProps) {
+  const [open, setOpen] = useState(false);
   const [currentCameras, setCurrentCameras] = useState<string[] | undefined>(
     selectedCameras,
   );
 
   return (
     <DropdownMenu
+      open={open}
       onOpenChange={(open) => {
         if (!open) {
-          updateCameraFilter(currentCameras);
+          setCurrentCameras(selectedCameras);
         }
+
+        setOpen(open);
       }}
     >
       <DropdownMenuTrigger asChild>
@@ -206,6 +217,17 @@ function CamerasFilterButton({
           />
         ))}
         <DropdownMenuSeparator />
+        <div className="flex justify-center items-center">
+          <Button
+            variant="select"
+            onClick={() => {
+              updateCameraFilter(currentCameras);
+              setOpen(false);
+            }}
+          >
+            Apply
+          </Button>
+        </div>
       </DropdownMenuContent>
     </DropdownMenu>
   );
@@ -219,6 +241,7 @@ function CalendarFilterButton({
   day,
   updateSelectedDay,
 }: CalendarFilterButtonProps) {
+  const [open, setOpen] = useState(false);
   const [selectedDay, setSelectedDay] = useState(day);
   const disabledDates = useMemo(() => {
     const tomorrow = new Date();
@@ -234,10 +257,13 @@ function CalendarFilterButton({
 
   return (
     <Popover
+      open={open}
       onOpenChange={(open) => {
         if (!open) {
-          updateSelectedDay(selectedDay);
+          setSelectedDay(day);
         }
+
+        setOpen(open);
       }}
     >
       <PopoverTrigger asChild>
@@ -257,6 +283,18 @@ function CalendarFilterButton({
             setSelectedDay(day);
           }}
         />
+        <DropdownMenuSeparator />
+        <div className="flex justify-center items-center">
+          <Button
+            variant="select"
+            onClick={() => {
+              updateSelectedDay(selectedDay);
+              setOpen(false);
+            }}
+          >
+            Apply
+          </Button>
+        </div>
       </PopoverContent>
     </Popover>
   );
@@ -276,70 +314,47 @@ function GeneralFilterButton({
   showReviewed,
   setShowReviewed,
 }: GeneralFilterButtonProps) {
+  const [open, setOpen] = useState(false);
+  const [reviewed, setReviewed] = useState(showReviewed ?? 0);
+  const [currentLabels, setCurrentLabels] = useState<string[] | undefined>(
+    selectedLabels,
+  );
+
   return (
-    <Popover>
+    <Popover
+      open={open}
+      onOpenChange={(open) => {
+        if (!open) {
+          setReviewed(showReviewed ?? 0);
+          setCurrentLabels(selectedLabels);
+        }
+
+        setOpen(open);
+      }}
+    >
       <PopoverTrigger asChild>
         <Button size="sm" className="ml-1" variant="secondary">
           <FaFilter className="md:mr-[10px] text-muted-foreground" />
           <div className="hidden md:block">Filter</div>
         </Button>
       </PopoverTrigger>
-      <PopoverContent side="left" asChild>
+      <PopoverContent side="left">
-        <div className="w-80 flex">
+        <>
-          <LabelsFilterButton
-            allLabels={allLabels}
-            selectedLabels={selectedLabels}
-            updateLabelFilter={updateLabelFilter}
-          />
           <Button
-            className="capitalize flex justify-between items-center cursor-pointer w-full"
+            className="capitalize flex justify-between items-center w-full"
-            variant="secondary"
+            variant="ghost"
-            onClick={() => setShowReviewed(showReviewed == 0 ? 1 : 0)}
+            onClick={() => setReviewed(reviewed == 0 ? 1 : 0)}
           >
-            {showReviewed ? (
+            {reviewed ? (
               <LuCheck className="w-6 h-6" />
             ) : (
               <div className="w-6 h-6" />
             )}
-            <div className="ml-1 w-full flex justify-start">Show Reviewed</div>
+            <div className="ml-1 w-full flex justify-start text-primary-foreground">
-          </Button>
+              Show Reviewed
             </div>
-      </PopoverContent>
-    </Popover>
-  );
-}
-
-type LabelFilterButtonProps = {
-  allLabels: string[];
-  selectedLabels: string[] | undefined;
-  updateLabelFilter: (labels: string[] | undefined) => void;
-};
-function LabelsFilterButton({
-  allLabels,
-  selectedLabels,
-  updateLabelFilter,
-}: LabelFilterButtonProps) {
-  const [currentLabels, setCurrentLabels] = useState<string[] | undefined>(
-    selectedLabels,
-  );
-
-  return (
-    <DropdownMenu
-      onOpenChange={(open) => {
-        if (!open) {
-          updateLabelFilter(currentLabels);
-        }
-      }}
-    >
-      <DropdownMenuTrigger asChild>
-        <Button className="mx-1 capitalize" variant="secondary">
-          <LuVideo className=" mr-[10px]" />
-          {selectedLabels == undefined
-            ? "All Labels"
-            : `${selectedLabels.length} Labels`}
           </Button>
-      </DropdownMenuTrigger>
+          <DropdownMenuSeparator />
-      <DropdownMenuContent>
           <DropdownMenuLabel>Filter Labels</DropdownMenuLabel>
           <DropdownMenuSeparator />
           <FilterCheckBox
@@ -376,8 +391,27 @@ function LabelsFilterButton({
             />
           ))}
           <DropdownMenuSeparator />
-      </DropdownMenuContent>
+          <div className="flex justify-center items-center">
-    </DropdownMenu>
+            <Button
+              variant="select"
+              onClick={() => {
+                if (reviewed != showReviewed) {
+                  setShowReviewed(reviewed);
+                }
+
+                if (selectedLabels != currentLabels) {
+                  updateLabelFilter(currentLabels);
+                }
+
+                setOpen(false);
+              }}
+            >
+              Apply
+            </Button>
+          </div>
+        </>
+      </PopoverContent>
+    </Popover>
   );
 }
 

web/src/components/ui/button.tsx

@@ -10,7 +10,7 @@ const buttonVariants = cva(
     variants: {
       variant: {
         default: "bg-primary text-primary-foreground hover:bg-primary/90",
-        select: "bg-select text-white hover:bg-select/90",
+        select: "bg-selected text-white hover:bg-opacity-90",
         destructive:
           "bg-destructive text-destructive-foreground hover:bg-destructive/90",
         outline:

web/src/components/ui/calendar.tsx

@@ -41,8 +41,8 @@ function Calendar({
         ),
         day_range_end: "day-range-end",
         day_selected:
-          "bg-primary text-primary-foreground hover:bg-primary hover:text-primary-foreground focus:bg-primary focus:text-primary-foreground",
+          "bg-selected text-white hover:bg-selected hover:text-white focus:bg-selected focus:text-white",
-        day_today: "bg-accent text-accent-foreground",
+        day_today: "bg-muted text-muted-foreground",
         day_outside:
           "day-outside text-muted-foreground opacity-50 aria-selected:bg-accent/50 aria-selected:text-muted-foreground aria-selected:opacity-30",
         day_disabled: "text-muted-foreground opacity-50",