Update pydantic requirement from ==2.8.* to ==2.10.* in /docker/main

Updates the requirements on [pydantic](https://github.com/pydantic/pydantic) to permit the latest version. - [Release notes](https://github.com/pydantic/pydantic/releases) - [Changelog](https://github.com/pydantic/pydantic/blob/main/HISTORY.md) - [Commits](https://github.com/pydantic/pydantic/compare/v2.8.0b1...v2.10.0) --- updated-dependencies: - dependency-name: pydantic dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
2024-11-21 11:10:58 +00:00
29 changed files with 197 additions and 608 deletions
--- a/docker/hailo8l/install_hailort.sh
+++ b/docker/hailo8l/install_hailort.sh
@@ -10,8 +10,10 @@ elif [[ "${TARGETARCH}" == "arm64" ]]; then
    arch="aarch64"
 fi

+mkdir -p /rootfs
+
 wget -qO- "https://github.com/frigate-nvr/hailort/releases/download/v${hailo_version}/hailort-${TARGETARCH}.tar.gz" |
-    tar -C / -xzf -
+    tar -C /rootfs/ -xzf -

 mkdir -p /hailo-wheels

--- a/docker/main/requirements-wheels.txt
+++ b/docker/main/requirements-wheels.txt
@@ -19,7 +19,7 @@ pandas == 2.2.*
 peewee == 3.17.*
 peewee_migrate == 1.13.*
 psutil == 6.1.*
-pydantic == 2.8.*
+pydantic == 2.10.*
 git+https://github.com/fbcotter/py3nvml#egg=py3nvml
 pytz == 2024.*
 pyzmq == 26.2.*
--- a/docs/docs/frigate/installation.md
+++ b/docs/docs/frigate/installation.md
@@ -193,7 +193,6 @@ services:
    container_name: frigate
    privileged: true # this may not be necessary for all setups
    restart: unless-stopped
-    stop_grace_period: 30s # allow enough time to shut down the various services
    image: ghcr.io/blakeblackshear/frigate:stable
    shm_size: "512mb" # update for your cameras based on calculation above
    devices:
@@ -225,7 +224,6 @@ If you can't use docker compose, you can run the container with something simila
 docker run -d \
  --name frigate \
  --restart=unless-stopped \
-  --stop-timeout 30 \
  --mount type=tmpfs,target=/tmp/cache,tmpfs-size=1000000000 \
  --device /dev/bus/usb:/dev/bus/usb \
  --device /dev/dri/renderD128 \
--- a/docs/docs/guides/getting_started.md
+++ b/docs/docs/guides/getting_started.md
@@ -115,7 +115,6 @@ services:
  frigate:
    container_name: frigate
    restart: unless-stopped
-    stop_grace_period: 30s
    image: ghcr.io/blakeblackshear/frigate:stable
    volumes:
      - ./config:/config
--- a/docs/static/frigate-api.yaml
+++ b/docs/static/frigate-api.yaml
@@ -3225,7 +3225,7 @@ components:
          title: Sub Label
        score:
          anyOf:
-            - type: number
+            - type: integer
            - type: 'null'
          title: Score
          default: 0
@@ -3264,7 +3264,7 @@ components:
      properties:
        end_time:
          anyOf:
-            - type: number
+            - type: integer
            - type: 'null'
          title: End Time
      type: object
--- a/frigate/api/defs/events_body.py
+++ b/frigate/api/defs/events_body.py
@@ -1,4 +1,4 @@
-from typing import List, Optional, Union
+from typing import Optional, Union

 from pydantic import BaseModel, Field

@@ -17,18 +17,14 @@ class EventsDescriptionBody(BaseModel):
 class EventsCreateBody(BaseModel):
    source_type: Optional[str] = "api"
    sub_label: Optional[str] = None
-    score: Optional[float] = 0
+    score: Optional[int] = 0
    duration: Optional[int] = 30
    include_recording: Optional[bool] = True
    draw: Optional[dict] = {}


 class EventsEndBody(BaseModel):
-    end_time: Optional[float] = None
-
-
-class EventsDeleteBody(BaseModel):
-    event_ids: List[str] = Field(title="The event IDs to delete")
+    end_time: Optional[int] = None


 class SubmitPlusBody(BaseModel):
--- a/frigate/api/event.py
+++ b/frigate/api/event.py
@@ -16,7 +16,6 @@ from playhouse.shortcuts import model_to_dict

 from frigate.api.defs.events_body import (
    EventsCreateBody,
-    EventsDeleteBody,
    EventsDescriptionBody,
    EventsEndBody,
    EventsSubLabelBody,
@@ -36,9 +35,8 @@ from frigate.const import (
    CLIPS_DIR,
 )
 from frigate.embeddings import EmbeddingsContext
-from frigate.events.external import ExternalEventProcessor
 from frigate.models import Event, ReviewSegment, Timeline
-from frigate.object_processing import TrackedObject, TrackedObjectProcessor
+from frigate.object_processing import TrackedObject
 from frigate.util.builtin import get_tz_modifiers

 logger = logging.getLogger(__name__)
@@ -1037,64 +1035,34 @@ def regenerate_description(
    )


-def delete_single_event(event_id: str, request: Request) -> dict:
+@router.delete("/events/{event_id}")
+def delete_event(request: Request, event_id: str):
    try:
        event = Event.get(Event.id == event_id)
    except DoesNotExist:
-        return {"success": False, "message": f"Event {event_id} not found"}
+        return JSONResponse(
+            content=({"success": False, "message": "Event " + event_id + " not found"}),
+            status_code=404,
+        )

    media_name = f"{event.camera}-{event.id}"
    if event.has_snapshot:
-        snapshot_paths = [
-            Path(f"{os.path.join(CLIPS_DIR, media_name)}.jpg"),
-            Path(f"{os.path.join(CLIPS_DIR, media_name)}-clean.png"),
-        ]
-        for media in snapshot_paths:
-            media.unlink(missing_ok=True)
+        media = Path(f"{os.path.join(CLIPS_DIR, media_name)}.jpg")
+        media.unlink(missing_ok=True)
+        media = Path(f"{os.path.join(CLIPS_DIR, media_name)}-clean.png")
+        media.unlink(missing_ok=True)

    event.delete_instance()
    Timeline.delete().where(Timeline.source_id == event_id).execute()
-
    # If semantic search is enabled, update the index
    if request.app.frigate_config.semantic_search.enabled:
        context: EmbeddingsContext = request.app.embeddings
        context.db.delete_embeddings_thumbnail(event_ids=[event_id])
        context.db.delete_embeddings_description(event_ids=[event_id])
-
-    return {"success": True, "message": f"Event {event_id} deleted"}
-
-
-@router.delete("/events/{event_id}")
-def delete_event(request: Request, event_id: str):
-    result = delete_single_event(event_id, request)
-    status_code = 200 if result["success"] else 404
-    return JSONResponse(content=result, status_code=status_code)
-
-
-@router.delete("/events/")
-def delete_events(request: Request, body: EventsDeleteBody):
-    if not body.event_ids:
-        return JSONResponse(
-            content=({"success": False, "message": "No event IDs provided."}),
-            status_code=404,
-        )
-
-    deleted_events = []
-    not_found_events = []
-
-    for event_id in body.event_ids:
-        result = delete_single_event(event_id, request)
-        if result["success"]:
-            deleted_events.append(event_id)
-        else:
-            not_found_events.append(event_id)
-
-    response = {
-        "success": True,
-        "deleted_events": deleted_events,
-        "not_found_events": not_found_events,
-    }
-    return JSONResponse(content=response, status_code=200)
+    return JSONResponse(
+        content=({"success": True, "message": "Event " + event_id + " deleted"}),
+        status_code=200,
+    )


@router.post("/events/{camera_name}/{label}/create")
@@ -1119,11 +1087,9 @@ def create_event(
        )

    try:
-        frame_processor: TrackedObjectProcessor = request.app.detected_frames_processor
-        external_processor: ExternalEventProcessor = request.app.external_processor
+        frame = request.app.detected_frames_processor.get_current_frame(camera_name)

-        frame = frame_processor.get_current_frame(camera_name)
-        event_id = external_processor.create_manual_event(
+        event_id = request.app.external_processor.create_manual_event(
            camera_name,
            label,
            body.source_type,
--- a/frigate/api/media.py
+++ b/frigate/api/media.py
@@ -36,7 +36,6 @@ from frigate.const import (
    RECORD_DIR,
 )
 from frigate.models import Event, Previews, Recordings, Regions, ReviewSegment
-from frigate.object_processing import TrackedObjectProcessor
 from frigate.util.builtin import get_tz_modifiers
 from frigate.util.image import get_image_from_recording

@@ -80,11 +79,7 @@ def mjpeg_feed(


 def imagestream(
-    detected_frames_processor: TrackedObjectProcessor,
-    camera_name: str,
-    fps: int,
-    height: int,
-    draw_options: dict[str, any],
+    detected_frames_processor, camera_name: str, fps: int, height: int, draw_options
 ):
    while True:
        # max out at specified FPS
@@ -123,7 +118,6 @@ def latest_frame(
    extension: Extension,
    params: MediaLatestFrameQueryParams = Depends(),
 ):
-    frame_processor: TrackedObjectProcessor = request.app.detected_frames_processor
    draw_options = {
        "bounding_boxes": params.bbox,
        "timestamp": params.timestamp,
@@ -135,14 +129,17 @@ def latest_frame(
    quality = params.quality

    if camera_name in request.app.frigate_config.cameras:
-        frame = frame_processor.get_current_frame(camera_name, draw_options)
+        frame = request.app.detected_frames_processor.get_current_frame(
+            camera_name, draw_options
+        )
        retry_interval = float(
            request.app.frigate_config.cameras.get(camera_name).ffmpeg.retry_interval
            or 10
        )

        if frame is None or datetime.now().timestamp() > (
-            frame_processor.get_current_frame_time(camera_name) + retry_interval
+            request.app.detected_frames_processor.get_current_frame_time(camera_name)
+            + retry_interval
        ):
            if request.app.camera_error_image is None:
                error_image = glob.glob("/opt/frigate/frigate/images/camera-error.jpg")
@@ -183,7 +180,7 @@ def latest_frame(
        )
    elif camera_name == "birdseye" and request.app.frigate_config.birdseye.restream:
        frame = cv2.cvtColor(
-            frame_processor.get_current_frame(camera_name),
+            request.app.detected_frames_processor.get_current_frame(camera_name),
            cv2.COLOR_YUV2BGR_I420,
        )

@@ -816,15 +813,15 @@ def grid_snapshot(
 ):
    if camera_name in request.app.frigate_config.cameras:
        detect = request.app.frigate_config.cameras[camera_name].detect
-        frame_processor: TrackedObjectProcessor = request.app.detected_frames_processor
-        frame = frame_processor.get_current_frame(camera_name, {})
+        frame = request.app.detected_frames_processor.get_current_frame(camera_name, {})
        retry_interval = float(
            request.app.frigate_config.cameras.get(camera_name).ffmpeg.retry_interval
            or 10
        )

        if frame is None or datetime.now().timestamp() > (
-            frame_processor.get_current_frame_time(camera_name) + retry_interval
+            request.app.detected_frames_processor.get_current_frame_time(camera_name)
+            + retry_interval
        ):
            return JSONResponse(
                content={"success": False, "message": "Unable to get valid frame"},
--- a/frigate/app.py
+++ b/frigate/app.py
@@ -36,7 +36,6 @@ from frigate.const import (
    EXPORT_DIR,
    MODEL_CACHE_DIR,
    RECORD_DIR,
-    SHM_FRAMES_VAR,
 )
 from frigate.db.sqlitevecq import SqliteVecQueueDatabase
 from frigate.embeddings import EmbeddingsContext, manage_embeddings
@@ -437,7 +436,7 @@ class FrigateApp:
            # pre-create shms
            for i in range(shm_frame_count):
                frame_size = config.frame_shape_yuv[0] * config.frame_shape_yuv[1]
-                self.frame_manager.create(f"{config.name}_{i}", frame_size)
+                self.frame_manager.create(f"{config.name}{i}", frame_size)

            capture_process = util.Process(
                target=capture_camera,
@@ -524,10 +523,7 @@ class FrigateApp:
        if cam_total_frame_size == 0.0:
            return 0

-        shm_frame_count = min(
-            int(os.environ.get(SHM_FRAMES_VAR, "50")),
-            int(available_shm / (cam_total_frame_size)),
-        )
+        shm_frame_count = min(200, int(available_shm / (cam_total_frame_size)))

        logger.debug(
            f"Calculated total camera size {available_shm} / {cam_total_frame_size} :: {shm_frame_count} frames for each camera in SHM"
--- a/frigate/config/config.py
+++ b/frigate/config/config.py
@@ -230,16 +230,12 @@ def verify_recording_segments_setup_with_reasonable_time(
    try:
        seg_arg_index = record_args.index("-segment_time")
    except ValueError:
-        raise ValueError(
-            f"Camera {camera_config.name} has no segment_time in \
-                         recording output args, segment args are required for record."
-        )
+        raise ValueError(f"Camera {camera_config.name} has no segment_time in \
+                         recording output args, segment args are required for record.")

    if int(record_args[seg_arg_index + 1]) > 60:
-        raise ValueError(
-            f"Camera {camera_config.name} has invalid segment_time output arg, \
-                         segment_time must be 60 or less."
-        )
+        raise ValueError(f"Camera {camera_config.name} has invalid segment_time output arg, \
+                         segment_time must be 60 or less.")


 def verify_zone_objects_are_tracked(camera_config: CameraConfig) -> None:
--- a/frigate/const.py
+++ b/frigate/const.py
@@ -13,8 +13,6 @@ FRIGATE_LOCALHOST = "http://127.0.0.1:5000"
 PLUS_ENV_VAR = "PLUS_API_KEY"
 PLUS_API_HOST = "https://api.frigate.video"

-SHM_FRAMES_VAR = "SHM_MAX_FRAMES"
-
 # Attribute & Object constants

 DEFAULT_ATTRIBUTE_LABEL_MAP = {
--- a/frigate/events/audio.py
+++ b/frigate/events/audio.py
@@ -216,10 +216,6 @@ class AudioEventMaintainer(threading.Thread):
                    "label": label,
                    "last_detection": datetime.datetime.now().timestamp(),
                }
-            else:
-                self.logger.warning(
-                    f"Failed to create audio event with status code {resp.status_code}"
-                )

    def expire_detections(self) -> None:
        now = datetime.datetime.now().timestamp()
--- a/frigate/events/cleanup.py
+++ b/frigate/events/cleanup.py
@@ -110,7 +110,7 @@ class EventCleanup(threading.Thread):
                .namedtuples()
                .iterator()
            )
-            logger.debug(f"{len(list(expired_events))} events can be expired")
+            logger.debug(f"{len(expired_events)} events can be expired")
            # delete the media from disk
            for expired in expired_events:
                media_name = f"{expired.camera}-{expired.id}"
--- a/frigate/events/external.py
+++ b/frigate/events/external.py
@@ -10,7 +10,6 @@ from enum import Enum
 from typing import Optional

 import cv2
-from numpy import ndarray

 from frigate.comms.detections_updater import DetectionPublisher, DetectionTypeEnum
 from frigate.comms.events_updater import EventUpdatePublisher
@@ -46,7 +45,7 @@ class ExternalEventProcessor:
        duration: Optional[int],
        include_recording: bool,
        draw: dict[str, any],
-        snapshot_frame: Optional[ndarray],
+        snapshot_frame: any,
    ) -> str:
        now = datetime.datetime.now().timestamp()
        camera_config = self.config.cameras.get(camera)
@@ -108,7 +107,6 @@ class ExternalEventProcessor:
                EventTypeEnum.api,
                EventStateEnum.end,
                None,
-                "",
                {"id": event_id, "end_time": end_time},
            )
        )
@@ -133,11 +131,8 @@ class ExternalEventProcessor:
        label: str,
        event_id: str,
        draw: dict[str, any],
-        img_frame: Optional[ndarray],
-    ) -> Optional[str]:
-        if img_frame is None:
-            return None
-
+        img_frame: any,
+    ) -> str:
        # write clean snapshot if enabled
        if camera_config.snapshots.clean_copy:
            ret, png = cv2.imencode(".png", img_frame)
--- a/frigate/object_processing.py
+++ b/frigate/object_processing.py
@@ -6,7 +6,7 @@ import queue
 import threading
 from collections import Counter, defaultdict
 from multiprocessing.synchronize import Event as MpEvent
-from typing import Callable, Optional
+from typing import Callable

 import cv2
 import numpy as np
@@ -784,18 +784,13 @@ class TrackedObjectProcessor(threading.Thread):
        else:
            return {}

-    def get_current_frame(
-        self, camera: str, draw_options: dict[str, any] = {}
-    ) -> Optional[np.ndarray]:
+    def get_current_frame(self, camera, draw_options={}):
        if camera == "birdseye":
            return self.frame_manager.get(
                "birdseye",
                (self.config.birdseye.height * 3 // 2, self.config.birdseye.width),
            )

-        if camera not in self.camera_states:
-            return None
-
        return self.camera_states[camera].get_current_frame(draw_options)

    def get_current_frame_time(self, camera) -> int:
--- a/frigate/review/maintainer.py
+++ b/frigate/review/maintainer.py
@@ -480,9 +480,7 @@ class ReviewSegmentMaintainer(threading.Thread):

            if not self.config.cameras[camera].record.enabled:
                if current_segment:
-                    self.update_existing_segment(
-                        current_segment, frame_name, frame_time, []
-                    )
+                    self.update_existing_segment(current_segment, frame_time, [])

                continue

--- a/frigate/util/builtin.py
+++ b/frigate/util/builtin.py
@@ -13,12 +13,12 @@ import urllib.parse
 from collections.abc import Mapping
 from pathlib import Path
 from typing import Any, Optional, Tuple, Union
-from zoneinfo import ZoneInfoNotFoundError

 import numpy as np
 import pytz
 from ruamel.yaml import YAML
 from tzlocal import get_localzone
+from zoneinfo import ZoneInfoNotFoundError

 from frigate.const import REGEX_HTTP_CAMERA_USER_PASS, REGEX_RTSP_CAMERA_USER_PASS

--- a/frigate/video.py
+++ b/frigate/video.py
@@ -113,7 +113,7 @@ def capture_frames(
        fps.value = frame_rate.eps()
        skipped_fps.value = skipped_eps.eps()
        current_frame.value = datetime.datetime.now().timestamp()
-        frame_name = f"{config.name}_{frame_index}"
+        frame_name = f"{config.name}{frame_index}"
        frame_buffer = frame_manager.write(frame_name)
        try:
            frame_buffer[:] = ffmpeg_process.stdout.read(frame_size)
--- a/web/package-lock.json
+++ b/web/package-lock.json
@@ -72,7 +72,6 @@
        "tailwind-merge": "^2.4.0",
        "tailwind-scrollbar": "^3.1.0",
        "tailwindcss-animate": "^1.0.7",
-        "use-long-press": "^3.2.0",
        "vaul": "^0.9.1",
        "vite-plugin-monaco-editor": "^1.1.0",
        "zod": "^3.23.8"
@@ -220,12 +219,12 @@
      "dev": true
    },
    "node_modules/@bundled-es-modules/cookie": {
-      "version": "2.0.1",
-      "resolved": "https://registry.npmjs.org/@bundled-es-modules/cookie/-/cookie-2.0.1.tgz",
-      "integrity": "sha512-8o+5fRPLNbjbdGRRmJj3h6Hh1AQJf2dk3qQ/5ZFb+PXkRNiSoMGGUKlsgLfrxneb72axVJyIYji64E2+nNfYyw==",
+      "version": "2.0.0",
+      "resolved": "https://registry.npmjs.org/@bundled-es-modules/cookie/-/cookie-2.0.0.tgz",
+      "integrity": "sha512-Or6YHg/kamKHpxULAdSqhGqnWFneIXu1NKvvfBBzKGwpVsYuFIQ5aBPHDnnoR3ghW1nvSkALd+EF9iMtY7Vjxw==",
      "dev": true,
      "dependencies": {
-        "cookie": "^0.7.2"
+        "cookie": "^0.5.0"
      }
    },
    "node_modules/@bundled-es-modules/statuses": {
@@ -4352,9 +4351,9 @@
      "integrity": "sha512-/Srv4dswyQNBfohGpz9o6Yb3Gz3SrUDqBH5rTuhGR7ahtlbYKnVxw2bCFMRljaA7EXHaXZ8wsHdodFvbkhKmqg=="
    },
    "node_modules/cookie": {
-      "version": "0.7.2",
-      "resolved": "https://registry.npmjs.org/cookie/-/cookie-0.7.2.tgz",
-      "integrity": "sha512-yki5XnKuf750l50uGTllt6kKILY4nQ1eNIQatoXEByZ5dWgnKqbnqmTrBE5B4N7lrMJKQ2ytWMiTO2o0v6Ew/w==",
+      "version": "0.5.0",
+      "resolved": "https://registry.npmjs.org/cookie/-/cookie-0.5.0.tgz",
+      "integrity": "sha512-YZ3GUyn/o8gfKJlnlX7g7xq4gyO6OSuhGPKaaGssGB2qgDUS0gPgtTvoyZLTt9Ab6dC4hfc9dV5arkvc/OCmrw==",
      "dev": true,
      "engines": {
        "node": ">= 0.6"
@@ -8710,15 +8709,6 @@
        "scheduler": ">=0.19.0"
      }
    },
-    "node_modules/use-long-press": {
-      "version": "3.2.0",
-      "resolved": "https://registry.npmjs.org/use-long-press/-/use-long-press-3.2.0.tgz",
-      "integrity": "sha512-uq5o2qFR1VRjHn8Of7Fl344/AGvgk7C5Mcb4aSb1ZRVp6PkgdXJJLdRrlSTJQVkkQcDuqFbFc3mDX4COg7mRTA==",
-      "license": "MIT",
-      "peerDependencies": {
-        "react": ">=16.8.0"
-      }
-    },
    "node_modules/use-sidecar": {
      "version": "1.1.2",
      "resolved": "https://registry.npmjs.org/use-sidecar/-/use-sidecar-1.1.2.tgz",
--- a/web/package.json
+++ b/web/package.json
@@ -78,7 +78,6 @@
    "tailwind-merge": "^2.4.0",
    "tailwind-scrollbar": "^3.1.0",
    "tailwindcss-animate": "^1.0.7",
-    "use-long-press": "^3.2.0",
    "vaul": "^0.9.1",
    "vite-plugin-monaco-editor": "^1.1.0",
    "zod": "^3.23.8"
--- a/web/src/components/card/SearchThumbnail.tsx
+++ b/web/src/components/card/SearchThumbnail.tsx
@@ -1,4 +1,4 @@
-import { useMemo } from "react";
+import { useCallback, useMemo } from "react";
 import { useApiHost } from "@/api";
 import { getIconForLabel } from "@/utils/iconUtil";
 import useSWR from "swr";
@@ -12,11 +12,10 @@ import { capitalizeFirstLetter } from "@/utils/stringUtil";
 import { SearchResult } from "@/types/search";
 import { cn } from "@/lib/utils";
 import { TooltipPortal } from "@radix-ui/react-tooltip";
-import useContextMenu from "@/hooks/use-contextmenu";

 type SearchThumbnailProps = {
  searchResult: SearchResult;
-  onClick: (searchResult: SearchResult, ctrl: boolean, detail: boolean) => void;
+  onClick: (searchResult: SearchResult) => void;
 };

 export default function SearchThumbnail({
@@ -29,9 +28,9 @@ export default function SearchThumbnail({

  // interactions

-  useContextMenu(imgRef, () => {
-    onClick(searchResult, true, false);
-  });
+  const handleOnClick = useCallback(() => {
+    onClick(searchResult);
+  }, [searchResult, onClick]);

  const objectLabel = useMemo(() => {
    if (
@@ -46,10 +45,7 @@ export default function SearchThumbnail({
  }, [config, searchResult]);

  return (
-    <div
-      className="relative size-full cursor-pointer"
-      onClick={() => onClick(searchResult, false, true)}
-    >
+    <div className="relative size-full cursor-pointer" onClick={handleOnClick}>
      <ImageLoadingIndicator
        className="absolute inset-0"
        imgLoaded={imgLoaded}
@@ -83,7 +79,7 @@ export default function SearchThumbnail({
                <div className="mx-3 pb-1 text-sm text-white">
                  <Chip
                    className={`z-0 flex items-center justify-between gap-1 space-x-1 bg-gray-500 bg-gradient-to-br from-gray-400 to-gray-500 text-xs`}
-                    onClick={() => onClick(searchResult, false, true)}
+                    onClick={() => onClick(searchResult)}
                  >
                    {getIconForLabel(objectLabel, "size-3 text-white")}
                    {Math.round(
--- a/web/src/components/filter/SearchActionGroup.tsx
+++ b/web/src/components/filter/SearchActionGroup.tsx
@@ -1,132 +0,0 @@
-import { useCallback, useState } from "react";
-import axios from "axios";
-import { Button, buttonVariants } from "../ui/button";
-import { isDesktop } from "react-device-detect";
-import { HiTrash } from "react-icons/hi";
-import {
-  AlertDialog,
-  AlertDialogAction,
-  AlertDialogCancel,
-  AlertDialogContent,
-  AlertDialogDescription,
-  AlertDialogFooter,
-  AlertDialogHeader,
-  AlertDialogTitle,
-} from "../ui/alert-dialog";
-import useKeyboardListener from "@/hooks/use-keyboard-listener";
-import { toast } from "sonner";
-
-type SearchActionGroupProps = {
-  selectedObjects: string[];
-  setSelectedObjects: (ids: string[]) => void;
-  pullLatestData: () => void;
-};
-export default function SearchActionGroup({
-  selectedObjects,
-  setSelectedObjects,
-  pullLatestData,
-}: SearchActionGroupProps) {
-  const onClearSelected = useCallback(() => {
-    setSelectedObjects([]);
-  }, [setSelectedObjects]);
-
-  const onDelete = useCallback(async () => {
-    await axios
-      .delete(`events/`, {
-        data: { event_ids: selectedObjects },
-      })
-      .then((resp) => {
-        if (resp.status == 200) {
-          toast.success("Tracked objects deleted successfully.", {
-            position: "top-center",
-          });
-          setSelectedObjects([]);
-          pullLatestData();
-        }
-      })
-      .catch(() => {
-        toast.error("Failed to delete tracked objects.", {
-          position: "top-center",
-        });
-      });
-  }, [selectedObjects, setSelectedObjects, pullLatestData]);
-
-  const [deleteDialogOpen, setDeleteDialogOpen] = useState(false);
-  const [bypassDialog, setBypassDialog] = useState(false);
-
-  useKeyboardListener(["Shift"], (_, modifiers) => {
-    setBypassDialog(modifiers.shift);
-  });
-
-  const handleDelete = useCallback(() => {
-    if (bypassDialog) {
-      onDelete();
-    } else {
-      setDeleteDialogOpen(true);
-    }
-  }, [bypassDialog, onDelete]);
-
-  return (
-    <>
-      <AlertDialog
-        open={deleteDialogOpen}
-        onOpenChange={() => setDeleteDialogOpen(!deleteDialogOpen)}
-      >
-        <AlertDialogContent>
-          <AlertDialogHeader>
-            <AlertDialogTitle>Confirm Delete</AlertDialogTitle>
-          </AlertDialogHeader>
-          <AlertDialogDescription>
-            Deleting these {selectedObjects.length} tracked objects removes the
-            snapshot, any saved embeddings, and any associated object lifecycle
-            entries. Recorded footage of these tracked objects in History view
-            will <em>NOT</em> be deleted.
-            <br />
-            <br />
-            Are you sure you want to proceed?
-            <br />
-            <br />
-            Hold the <em>Shift</em> key to bypass this dialog in the future.
-          </AlertDialogDescription>
-          <AlertDialogFooter>
-            <AlertDialogCancel>Cancel</AlertDialogCancel>
-            <AlertDialogAction
-              className={buttonVariants({ variant: "destructive" })}
-              onClick={onDelete}
-            >
-              Delete
-            </AlertDialogAction>
-          </AlertDialogFooter>
-        </AlertDialogContent>
-      </AlertDialog>
-
-      <div className="absolute inset-x-2 inset-y-0 flex items-center justify-between gap-2 bg-background py-2 md:left-auto">
-        <div className="mx-1 flex items-center justify-center text-sm text-muted-foreground">
-          <div className="p-1">{`${selectedObjects.length} selected`}</div>
-          <div className="p-1">{"|"}</div>
-          <div
-            className="cursor-pointer p-2 text-primary hover:rounded-lg hover:bg-secondary"
-            onClick={onClearSelected}
-          >
-            Unselect
-          </div>
-        </div>
-        <div className="flex items-center gap-1 md:gap-2">
-          <Button
-            className="flex items-center gap-2 p-2"
-            aria-label="Delete"
-            size="sm"
-            onClick={handleDelete}
-          >
-            <HiTrash className="text-secondary-foreground" />
-            {isDesktop && (
-              <div className="text-primary">
-                {bypassDialog ? "Delete Now" : "Delete"}
-              </div>
-            )}
-          </Button>
-        </div>
-      </div>
-    </>
-  );
-}
--- a/web/src/components/menu/SearchResultActions.tsx
+++ b/web/src/components/menu/SearchResultActions.tsx
@@ -108,15 +108,13 @@ export default function SearchResultActions({
          </a>
        </MenuItem>
      )}
-      {searchResult.data.type == "object" && (
-        <MenuItem
-          aria-label="Show the object lifecycle"
-          onClick={showObjectLifecycle}
-        >
-          <FaArrowsRotate className="mr-2 size-4" />
-          <span>View object lifecycle</span>
-        </MenuItem>
-      )}
+      <MenuItem
+        aria-label="Show the object lifecycle"
+        onClick={showObjectLifecycle}
+      >
+        <FaArrowsRotate className="mr-2 size-4" />
+        <span>View object lifecycle</span>
+      </MenuItem>
      {config?.semantic_search?.enabled && isContextMenu && (
        <MenuItem
          aria-label="Find similar tracked objects"
@@ -130,7 +128,6 @@ export default function SearchResultActions({
        config?.plus?.enabled &&
        searchResult.has_snapshot &&
        searchResult.end_time &&
-        searchResult.data.type == "object" &&
        !searchResult.plus_id && (
          <MenuItem aria-label="Submit to Frigate Plus" onClick={showSnapshot}>
            <FrigatePlusIcon className="mr-2 size-4 cursor-pointer text-primary" />
@@ -184,24 +181,22 @@ export default function SearchResultActions({
        </ContextMenu>
      ) : (
        <>
-          {config?.semantic_search?.enabled &&
-            searchResult.data.type == "object" && (
-              <Tooltip>
-                <TooltipTrigger>
-                  <MdImageSearch
-                    className="size-5 cursor-pointer text-primary-variant hover:text-primary"
-                    onClick={findSimilar}
-                  />
-                </TooltipTrigger>
-                <TooltipContent>Find similar</TooltipContent>
-              </Tooltip>
-            )}
+          {config?.semantic_search?.enabled && (
+            <Tooltip>
+              <TooltipTrigger>
+                <MdImageSearch
+                  className="size-5 cursor-pointer text-primary-variant hover:text-primary"
+                  onClick={findSimilar}
+                />
+              </TooltipTrigger>
+              <TooltipContent>Find similar</TooltipContent>
+            </Tooltip>
+          )}

          {!isMobileOnly &&
            config?.plus?.enabled &&
            searchResult.has_snapshot &&
            searchResult.end_time &&
-            searchResult.data.type == "object" &&
            !searchResult.plus_id && (
              <Tooltip>
                <TooltipTrigger>
--- a/web/src/components/overlay/detail/ReviewDetailDialog.tsx
+++ b/web/src/components/overlay/detail/ReviewDetailDialog.tsx
@@ -379,7 +379,6 @@ function EventItem({

              {event.has_snapshot &&
                event.plus_id == undefined &&
-                event.data.type == "object" &&
                config?.plus.enabled && (
                  <Tooltip>
                    <TooltipTrigger>
--- a/web/src/components/overlay/detail/SearchDetailDialog.tsx
+++ b/web/src/components/overlay/detail/SearchDetailDialog.tsx
@@ -452,7 +452,7 @@ function ObjectDetailsTab({
            draggable={false}
            src={`${apiHost}api/events/${search.id}/thumbnail.jpg`}
          />
-          {config?.semantic_search.enabled && search.data.type == "object" && (
+          {config?.semantic_search.enabled && (
            <Button
              aria-label="Find similar tracked objects"
              onClick={() => {
@@ -626,67 +626,65 @@ export function ObjectSnapshotTab({
                </div>
              )}
            </TransformComponent>
-            {search.data.type == "object" &&
-              search.plus_id !== "not_enabled" &&
-              search.end_time && (
-                <Card className="p-1 text-sm md:p-2">
-                  <CardContent className="flex flex-col items-center justify-between gap-3 p-2 md:flex-row">
-                    <div className={cn("flex flex-col space-y-3")}>
-                      <div
-                        className={
-                          "text-lg font-semibold leading-none tracking-tight"
-                        }
-                      >
-                        Submit To Frigate+
-                      </div>
-                      <div className="text-sm text-muted-foreground">
-                        Objects in locations you want to avoid are not false
-                        positives. Submitting them as false positives will
-                        confuse the model.
-                      </div>
+            {search.plus_id !== "not_enabled" && search.end_time && (
+              <Card className="p-1 text-sm md:p-2">
+                <CardContent className="flex flex-col items-center justify-between gap-3 p-2 md:flex-row">
+                  <div className={cn("flex flex-col space-y-3")}>
+                    <div
+                      className={
+                        "text-lg font-semibold leading-none tracking-tight"
+                      }
+                    >
+                      Submit To Frigate+
                    </div>
+                    <div className="text-sm text-muted-foreground">
+                      Objects in locations you want to avoid are not false
+                      positives. Submitting them as false positives will confuse
+                      the model.
+                    </div>
+                  </div>

-                    <div className="flex flex-row justify-center gap-2 md:justify-end">
-                      {state == "reviewing" && (
-                        <>
-                          <Button
-                            className="bg-success"
-                            aria-label="Confirm this label for Frigate Plus"
-                            onClick={() => {
-                              setState("uploading");
-                              onSubmitToPlus(false);
-                            }}
-                          >
-                            This is{" "}
-                            {/^[aeiou]/i.test(search?.label || "") ? "an" : "a"}{" "}
-                            {search?.label}
-                          </Button>
-                          <Button
-                            className="text-white"
-                            aria-label="Do not confirm this label for Frigate Plus"
-                            variant="destructive"
-                            onClick={() => {
-                              setState("uploading");
-                              onSubmitToPlus(true);
-                            }}
-                          >
-                            This is not{" "}
-                            {/^[aeiou]/i.test(search?.label || "") ? "an" : "a"}{" "}
-                            {search?.label}
-                          </Button>
-                        </>
-                      )}
-                      {state == "uploading" && <ActivityIndicator />}
-                      {state == "submitted" && (
-                        <div className="flex flex-row items-center justify-center gap-2">
-                          <FaCheckCircle className="text-success" />
-                          Submitted
-                        </div>
-                      )}
-                    </div>
-                  </CardContent>
-                </Card>
-              )}
+                  <div className="flex flex-row justify-center gap-2 md:justify-end">
+                    {state == "reviewing" && (
+                      <>
+                        <Button
+                          className="bg-success"
+                          aria-label="Confirm this label for Frigate Plus"
+                          onClick={() => {
+                            setState("uploading");
+                            onSubmitToPlus(false);
+                          }}
+                        >
+                          This is{" "}
+                          {/^[aeiou]/i.test(search?.label || "") ? "an" : "a"}{" "}
+                          {search?.label}
+                        </Button>
+                        <Button
+                          className="text-white"
+                          aria-label="Do not confirm this label for Frigate Plus"
+                          variant="destructive"
+                          onClick={() => {
+                            setState("uploading");
+                            onSubmitToPlus(true);
+                          }}
+                        >
+                          This is not{" "}
+                          {/^[aeiou]/i.test(search?.label || "") ? "an" : "a"}{" "}
+                          {search?.label}
+                        </Button>
+                      </>
+                    )}
+                    {state == "uploading" && <ActivityIndicator />}
+                    {state == "submitted" && (
+                      <div className="flex flex-row items-center justify-center gap-2">
+                        <FaCheckCircle className="text-success" />
+                        Submitted
+                      </div>
+                    )}
+                  </div>
+                </CardContent>
+              </Card>
+            )}
          </div>
        </TransformWrapper>
      </div>
--- a/web/src/hooks/use-press.ts
+++ b/web/src/hooks/use-press.ts
@@ -1,54 +0,0 @@
-// https://gist.github.com/cpojer/641bf305e6185006ea453e7631b80f95
-
-import { useCallback, useState } from "react";
-import {
-  LongPressCallbackMeta,
-  LongPressReactEvents,
-  useLongPress,
-} from "use-long-press";
-
-export default function usePress(
-  options: Omit<Parameters<typeof useLongPress>[1], "onCancel" | "onStart"> & {
-    onLongPress: NonNullable<Parameters<typeof useLongPress>[0]>;
-    onPress: (event: LongPressReactEvents<Element>) => void;
-  },
-) {
-  const { onLongPress, onPress, ...actualOptions } = options;
-  const [hasLongPress, setHasLongPress] = useState(false);
-
-  const onCancel = useCallback(() => {
-    if (hasLongPress) {
-      setHasLongPress(false);
-    }
-  }, [hasLongPress]);
-
-  const bind = useLongPress(
-    useCallback(
-      (
-        event: LongPressReactEvents<Element>,
-        meta: LongPressCallbackMeta<unknown>,
-      ) => {
-        setHasLongPress(true);
-        onLongPress(event, meta);
-      },
-      [onLongPress],
-    ),
-    {
-      ...actualOptions,
-      onCancel,
-      onStart: onCancel,
-    },
-  );
-
-  return useCallback(
-    () => ({
-      ...bind(),
-      onClick: (event: LongPressReactEvents<HTMLDivElement>) => {
-        if (!hasLongPress) {
-          onPress(event);
-        }
-      },
-    }),
-    [bind, hasLongPress, onPress],
-  );
-}
--- a/web/src/pages/Live.tsx
+++ b/web/src/pages/Live.tsx
@@ -62,7 +62,6 @@ function Live() {
    if (selectedCameraName) {
      const capitalized = selectedCameraName
        .split("_")
-        .filter((text) => text)
        .map((text) => text[0].toUpperCase() + text.substring(1));
      document.title = `${capitalized.join(" ")} - Live - Frigate`;
    } else if (cameraGroup && cameraGroup != "default") {
--- a/web/src/views/explore/ExploreView.tsx
+++ b/web/src/views/explore/ExploreView.tsx
@@ -26,7 +26,7 @@ type ExploreViewProps = {
  searchDetail: SearchResult | undefined;
  setSearchDetail: (search: SearchResult | undefined) => void;
  setSimilaritySearch: (search: SearchResult) => void;
-  onSelectSearch: (item: SearchResult, ctrl: boolean, page?: SearchTab) => void;
+  onSelectSearch: (item: SearchResult, index: number, page?: SearchTab) => void;
 };

 export default function ExploreView({
@@ -125,7 +125,7 @@ type ThumbnailRowType = {
  setSearchDetail: (search: SearchResult | undefined) => void;
  mutate: () => void;
  setSimilaritySearch: (search: SearchResult) => void;
-  onSelectSearch: (item: SearchResult, ctrl: boolean, page?: SearchTab) => void;
+  onSelectSearch: (item: SearchResult, index: number, page?: SearchTab) => void;
 };

 function ThumbnailRow({
@@ -205,7 +205,7 @@ type ExploreThumbnailImageProps = {
  setSearchDetail: (search: SearchResult | undefined) => void;
  mutate: () => void;
  setSimilaritySearch: (search: SearchResult) => void;
-  onSelectSearch: (item: SearchResult, ctrl: boolean, page?: SearchTab) => void;
+  onSelectSearch: (item: SearchResult, index: number, page?: SearchTab) => void;
 };
 function ExploreThumbnailImage({
  event,
@@ -225,11 +225,11 @@ function ExploreThumbnailImage({
  };

  const handleShowObjectLifecycle = () => {
-    onSelectSearch(event, false, "object lifecycle");
+    onSelectSearch(event, 0, "object lifecycle");
  };

  const handleShowSnapshot = () => {
-    onSelectSearch(event, false, "snapshot");
+    onSelectSearch(event, 0, "snapshot");
  };

  return (
--- a/web/src/views/search/SearchView.tsx
+++ b/web/src/views/search/SearchView.tsx
@@ -30,7 +30,6 @@ import {
 } from "@/components/ui/tooltip";
 import Chip from "@/components/indicators/Chip";
 import { TooltipPortal } from "@radix-ui/react-tooltip";
-import SearchActionGroup from "@/components/filter/SearchActionGroup";

 type SearchViewProps = {
  search: string;
@@ -182,53 +181,20 @@ export default function SearchView({

  // search interaction

-  const [selectedObjects, setSelectedObjects] = useState<string[]>([]);
+  const [selectedIndex, setSelectedIndex] = useState<number | null>(null);
  const itemRefs = useRef<(HTMLDivElement | null)[]>([]);

  const onSelectSearch = useCallback(
-    (item: SearchResult, ctrl: boolean, page: SearchTab = "details") => {
-      if (selectedObjects.length > 1 || ctrl) {
-        const index = selectedObjects.indexOf(item.id);
-
-        if (index != -1) {
-          if (selectedObjects.length == 1) {
-            setSelectedObjects([]);
-          } else {
-            const copy = [
-              ...selectedObjects.slice(0, index),
-              ...selectedObjects.slice(index + 1),
-            ];
-            setSelectedObjects(copy);
-          }
-        } else {
-          const copy = [...selectedObjects];
-          copy.push(item.id);
-          setSelectedObjects(copy);
-        }
-      } else {
-        setPage(page);
-        setSearchDetail(item);
-      }
+    (item: SearchResult, index: number, page: SearchTab = "details") => {
+      setPage(page);
+      setSearchDetail(item);
+      setSelectedIndex(index);
    },
-    [selectedObjects],
+    [],
  );

-  const onSelectAllObjects = useCallback(() => {
-    if (!uniqueResults || uniqueResults.length == 0) {
-      return;
-    }
-
-    if (selectedObjects.length < uniqueResults.length) {
-      setSelectedObjects(uniqueResults.map((value) => value.id));
-    } else {
-      setSelectedObjects([]);
-    }
-  }, [uniqueResults, selectedObjects]);
-
  useEffect(() => {
-    setSelectedObjects([]);
-    // unselect items when search term or filter changes
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    setSelectedIndex(0);
  }, [searchTerm, searchFilter]);

  // confidence score
@@ -277,44 +243,23 @@ export default function SearchView({
      }

      switch (key) {
-        case "a":
-          if (modifiers.ctrl) {
-            onSelectAllObjects();
-          }
-          break;
        case "ArrowLeft":
-          if (uniqueResults.length > 0) {
-            const currentIndex = searchDetail
-              ? uniqueResults.findIndex(
-                  (result) => result.id === searchDetail.id,
-                )
-              : -1;
-
+          setSelectedIndex((prevIndex) => {
            const newIndex =
-              currentIndex === -1
+              prevIndex === null
                ? uniqueResults.length - 1
-                : (currentIndex - 1 + uniqueResults.length) %
-                  uniqueResults.length;
-
+                : (prevIndex - 1 + uniqueResults.length) % uniqueResults.length;
            setSearchDetail(uniqueResults[newIndex]);
-          }
+            return newIndex;
+          });
          break;
-
        case "ArrowRight":
-          if (uniqueResults.length > 0) {
-            const currentIndex = searchDetail
-              ? uniqueResults.findIndex(
-                  (result) => result.id === searchDetail.id,
-                )
-              : -1;
-
+          setSelectedIndex((prevIndex) => {
            const newIndex =
-              currentIndex === -1
-                ? 0
-                : (currentIndex + 1) % uniqueResults.length;
-
+              prevIndex === null ? 0 : (prevIndex + 1) % uniqueResults.length;
            setSearchDetail(uniqueResults[newIndex]);
-          }
+            return newIndex;
+          });
          break;
        case "PageDown":
          contentRef.current?.scrollBy({
@@ -330,80 +275,32 @@ export default function SearchView({
          break;
      }
    },
-    [uniqueResults, inputFocused, onSelectAllObjects, searchDetail],
+    [uniqueResults, inputFocused],
  );

  useKeyboardListener(
-    ["a", "ArrowLeft", "ArrowRight", "PageDown", "PageUp"],
+    ["ArrowLeft", "ArrowRight", "PageDown", "PageUp"],
    onKeyboardShortcut,
    !inputFocused,
  );

  // scroll into view

-  const [prevSearchDetail, setPrevSearchDetail] = useState<
-    SearchResult | undefined
-  >();
-
-  // keep track of previous ref to outline thumbnail when dialog closes
-  const prevSearchDetailRef = useRef<SearchResult | undefined>();
-
  useEffect(() => {
-    if (searchDetail === undefined && prevSearchDetailRef.current) {
-      setPrevSearchDetail(prevSearchDetailRef.current);
+    if (
+      selectedIndex !== null &&
+      uniqueResults &&
+      itemRefs.current?.[selectedIndex]
+    ) {
+      scrollIntoView(itemRefs.current[selectedIndex], {
+        block: "center",
+        behavior: "smooth",
+        scrollMode: "if-needed",
+      });
    }
-    prevSearchDetailRef.current = searchDetail;
-  }, [searchDetail]);
-
-  useEffect(() => {
-    if (uniqueResults && itemRefs.current && prevSearchDetail) {
-      const selectedIndex = uniqueResults.findIndex(
-        (result) => result.id === prevSearchDetail.id,
-      );
-
-      const parent = itemRefs.current[selectedIndex];
-
-      if (selectedIndex !== -1 && parent) {
-        const target = parent.querySelector(".review-item-ring");
-        if (target) {
-          scrollIntoView(target, {
-            block: "center",
-            behavior: "smooth",
-            scrollMode: "if-needed",
-          });
-          target.classList.add(`outline-selected`);
-          target.classList.remove("outline-transparent");
-
-          setTimeout(() => {
-            target.classList.remove(`outline-selected`);
-            target.classList.add("outline-transparent");
-          }, 3000);
-        }
-      }
-    }
-    // we only want to scroll when the dialog closes
+    // we only want to scroll when the index changes
    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [prevSearchDetail]);
-
-  useEffect(() => {
-    if (uniqueResults && itemRefs.current && searchDetail) {
-      const selectedIndex = uniqueResults.findIndex(
-        (result) => result.id === searchDetail.id,
-      );
-
-      const parent = itemRefs.current[selectedIndex];
-
-      if (selectedIndex !== -1 && parent) {
-        scrollIntoView(parent, {
-          block: "center",
-          behavior: "smooth",
-          scrollMode: "if-needed",
-        });
-      }
-    }
-    // we only want to scroll when changing the detail pane
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [searchDetail]);
+  }, [selectedIndex]);

  // observer for loading more

@@ -472,39 +369,22 @@ export default function SearchView({
        {hasExistingSearch && (
          <ScrollArea className="w-full whitespace-nowrap lg:ml-[35%]">
            <div className="flex flex-row gap-2">
-              {selectedObjects.length == 0 ? (
-                <>
-                  <SearchFilterGroup
-                    className={cn(
-                      "w-full justify-between md:justify-start lg:justify-end",
-                    )}
-                    filter={searchFilter}
-                    onUpdateFilter={onUpdateFilter}
-                  />
-                  <SearchSettings
-                    columns={columns}
-                    setColumns={setColumns}
-                    defaultView={defaultView}
-                    setDefaultView={setDefaultView}
-                    filter={searchFilter}
-                    onUpdateFilter={onUpdateFilter}
-                  />
-                  <ScrollBar orientation="horizontal" className="h-0" />
-                </>
-              ) : (
-                <div
-                  className={cn(
-                    "scrollbar-container flex justify-center gap-2 overflow-x-auto",
-                    "h-10 w-full justify-between md:justify-start lg:justify-end",
-                  )}
-                >
-                  <SearchActionGroup
-                    selectedObjects={selectedObjects}
-                    setSelectedObjects={setSelectedObjects}
-                    pullLatestData={refresh}
-                  />
-                </div>
-              )}
+              <SearchFilterGroup
+                className={cn(
+                  "w-full justify-between md:justify-start lg:justify-end",
+                )}
+                filter={searchFilter}
+                onUpdateFilter={onUpdateFilter}
+              />
+              <SearchSettings
+                columns={columns}
+                setColumns={setColumns}
+                defaultView={defaultView}
+                setDefaultView={setDefaultView}
+                filter={searchFilter}
+                onUpdateFilter={onUpdateFilter}
+              />
+              <ScrollBar orientation="horizontal" className="h-0" />
            </div>
          </ScrollArea>
        )}
@@ -532,14 +412,14 @@ export default function SearchView({
          <div className={gridClassName}>
            {uniqueResults &&
              uniqueResults.map((value, index) => {
-                const selected = selectedObjects.includes(value.id);
+                const selected = selectedIndex === index;

                return (
                  <div
                    key={value.id}
                    ref={(item) => (itemRefs.current[index] = item)}
                    data-start={value.start_time}
-                    className="relative flex flex-col rounded-lg"
+                    className="review-item relative flex flex-col rounded-lg"
                  >
                    <div
                      className={cn(
@@ -548,20 +428,7 @@ export default function SearchView({
                    >
                      <SearchThumbnail
                        searchResult={value}
-                        onClick={(
-                          value: SearchResult,
-                          ctrl: boolean,
-                          detail: boolean,
-                        ) => {
-                          if (detail && selectedObjects.length == 0) {
-                            setSearchDetail(value);
-                          } else {
-                            onSelectSearch(
-                              value,
-                              ctrl || selectedObjects.length > 0,
-                            );
-                          }
-                        }}
+                        onClick={() => onSelectSearch(value, index)}
                      />
                      {(searchTerm ||
                        searchFilter?.search_type?.includes("similarity")) && (
@@ -602,10 +469,10 @@ export default function SearchView({
                        }}
                        refreshResults={refresh}
                        showObjectLifecycle={() =>
-                          onSelectSearch(value, false, "object lifecycle")
+                          onSelectSearch(value, index, "object lifecycle")
                        }
                        showSnapshot={() =>
-                          onSelectSearch(value, false, "snapshot")
+                          onSelectSearch(value, index, "snapshot")
                        }
                      />
                    </div>