From d366bdc268429d71b2ae0a949b82fac7b229b10b Mon Sep 17 00:00:00 2001
From: Nicolas Mowen <nickmowen213@gmail.com>
Date: Mon, 29 Apr 2024 10:11:09 -0600
Subject: [PATCH] Use variable for preview FPS

---
 web/src/components/player/PreviewThumbnailPlayer.tsx | 6 +++---
 web/src/types/preview.ts                             | 3 ++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/web/src/components/player/PreviewThumbnailPlayer.tsx b/web/src/components/player/PreviewThumbnailPlayer.tsx
index 4eb9c9e96..c36a2e129 100644
--- a/web/src/components/player/PreviewThumbnailPlayer.tsx
+++ b/web/src/components/player/PreviewThumbnailPlayer.tsx
@@ -23,7 +23,7 @@ import useContextMenu from "@/hooks/use-contextmenu";
 import ActivityIndicator from "../indicators/activity-indicator";
 import { TimeRange } from "@/types/timeline";
 import { NoThumbSlider } from "../ui/slider";
-import { PREVIEW_PADDING } from "@/types/preview";
+import { PREVIEW_FPS, PREVIEW_PADDING } from "@/types/preview";
 
 type PreviewPlayerProps = {
   review: ReviewSegment;
@@ -398,7 +398,7 @@ export function VideoPreview({
       setManualPlayback(true);
     } else {
       playerRef.current.currentTime = playerStartTime;
-      playerRef.current.playbackRate = 8;
+      playerRef.current.playbackRate = PREVIEW_FPS;
     }
 
     // we know that these deps are correct
@@ -470,7 +470,7 @@ export function VideoPreview({
         playerRef.current.currentTime = playerStartTime + counter;
         counter += 1;
       }
-    }, 125);
+    }, 1000 / PREVIEW_FPS);
     return () => clearInterval(intervalId);
 
     // we know that these deps are correct
diff --git a/web/src/types/preview.ts b/web/src/types/preview.ts
index a050705b1..a9f958f7f 100644
--- a/web/src/types/preview.ts
+++ b/web/src/types/preview.ts
@@ -8,4 +8,5 @@ export type Preview = {
   end: number;
 };
 
-export const PREVIEW_PADDING = REVIEW_PADDING * 2;
+export const PREVIEW_FPS = 8;
+export const PREVIEW_PADDING = REVIEW_PADDING * PREVIEW_FPS;