bugfixes and small UI tweaks (#11360)

blakeblackshear · May 13, 2024 · fbec083 · fbec083
1 parent 9680f2a
commit fbec083
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/web/src/components/filter/CameraGroupSelector.tsx b/web/src/components/filter/CameraGroupSelector.tsx
@@ -440,7 +440,7 @@ export function CameraGroupRow({
     <>
       <div
         key={group[0]}
-        className="flex md:p-1 rounded-lg flex-row items-center justify-between md:mx-2 my-1.5 transition-background duration-100"
+        className="flex md:p-1 rounded-lg flex-row items-center justify-between my-1.5 transition-background duration-100"
       >
         <div className={`flex items-center`}>
           <p className="cursor-default">{group[0]}</p>

diff --git a/web/src/components/settings/PolygonItem.tsx b/web/src/components/settings/PolygonItem.tsx
@@ -206,7 +206,7 @@ export default function PolygonItem({
 
       <div
         key={index}
-        className="flex p-1 rounded-lg flex-row items-center justify-between mx-2 my-1.5 transition-background duration-100"
+        className="flex p-1 rounded-lg flex-row items-center justify-between my-1.5 transition-background duration-100"
         data-index={index}
         onMouseEnter={() => setHoveredPolygonIndex(index)}
         onMouseLeave={() => setHoveredPolygonIndex(null)}

diff --git a/web/src/components/timeline/EventReviewTimeline.tsx b/web/src/components/timeline/EventReviewTimeline.tsx
@@ -147,9 +147,11 @@ export function EventReviewTimeline({
         });
       }
     }
+    // don't scroll when segments update from unreviewed -> reviewed
+    // we know that these deps are correct
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [
     selectedTimelineRef,
-    segments,
     showMinimap,
     alignStartDateToTimeline,
     visibleTimestamps,

diff --git a/web/src/views/events/EventView.tsx b/web/src/views/events/EventView.tsx
@@ -103,9 +103,9 @@ export default function EventView({
 
     if (filter?.showReviewed == 1) {
       return {
-        alert: summary.total_alert,
-        detection: summary.total_detection,
-        significant_motion: summary.total_motion,
+        alert: summary.total_alert ?? 0,
+        detection: summary.total_detection ?? 0,
+        significant_motion: summary.total_motion ?? 0,
       };
     } else {
       return {