From cf748c63b04db2f44186b2d69b201df951ef67e2 Mon Sep 17 00:00:00 2001
From: Martastain <martas@imm.cz>
Date: Thu, 25 Apr 2024 23:14:55 +0200
Subject: [PATCH] Added poster frame actions to video preview

---
 frontend/src/components/Dropdown.jsx          | 12 +--
 frontend/src/components/Video.jsx             | 97 +++++++++++++++----
 .../src/containers/Browser/Formatting.jsx     |  4 +-
 frontend/src/pages/AssetEditor/Preview.jsx    | 42 ++++++--
 4 files changed, 115 insertions(+), 40 deletions(-)

diff --git a/frontend/src/components/Dropdown.jsx b/frontend/src/components/Dropdown.jsx
index bc887b97..3914132a 100644
--- a/frontend/src/components/Dropdown.jsx
+++ b/frontend/src/components/Dropdown.jsx
@@ -60,18 +60,10 @@ const DropdownContainer = styled.div`
   }
 `
 
-const DropdownOption = ({
-  currentValue,
-  separator,
-  disabled,
-  ...props
-}) => (
+const DropdownOption = ({ currentValue, separator, disabled, ...props }) => (
   <span>
     {separator && <hr />}
-    <Button
-      {...props}
-      disabled={disabled || currentValue === props.value}
-    />
+    <Button {...props} disabled={disabled || currentValue === props.value} />
   </span>
 )
 
diff --git a/frontend/src/components/Video.jsx b/frontend/src/components/Video.jsx
index de4ba79c..28196ac8 100644
--- a/frontend/src/components/Video.jsx
+++ b/frontend/src/components/Video.jsx
@@ -20,18 +20,33 @@ const Trackbar = styled.input`
   flex-grow: 1;
   appearance: none;
   height: var(--input-height);
-  outline: none;
   padding: 0;
+  margin: 0;
   outline: 1px solid var(--color-surface-04);
+  border-radius: 4px;
 
   &::-webkit-slider-thumb {
+    padding: 0;
+    margin: 0;
     appearance: none;
-    width: 5px;
+    width: 4px;
     height: 25px;
     background-color: #d4d4d4;
     cursor: pointer;
   }
 
+  &:before {
+    content: '';
+    position: absolute;
+    display: ${(props) => (props.posterFrame ? 'block' : 'none')};
+    height: 4px;
+    bottom: 0;
+    width: 2px;
+    border: 0;
+    left: ${(props) => props.posterFrame}%;
+    background: var(--color-yellow);
+  }
+
   /* highlight the specified range using the highlightStart and highlightEnd props */
   background: linear-gradient(
     to right,
@@ -60,7 +75,15 @@ const PlayoutControls = styled.div`
   justify-content: center;
 `
 
-const Video = ({ src, style, showMarks, marks = {}, setMarks = () => {} }) => {
+const Video = ({
+  src,
+  style,
+  showMarks,
+  position,
+  setPosition,
+  marks = {},
+  setMarks = () => {},
+}) => {
   const videoRef = useRef(null)
   const sliderRef = useRef(null)
   const playButtonRef = useRef(null)
@@ -77,6 +100,17 @@ const Video = ({ src, style, showMarks, marks = {}, setMarks = () => {} }) => {
     setVideoDuration(0)
   }, [src])
 
+  useEffect(() => {
+    if (position !== undefined) {
+      videoRef.current.currentTime = position
+    }
+  }, [position])
+
+  useEffect(() => {
+    if (!setPosition) return
+    setPosition(videoPosition)
+  }, [videoPosition])
+
   const handleFinishSlider = useCallback(() => {
     if (playButtonRef.current) {
       playButtonRef.current.focus()
@@ -261,17 +295,28 @@ const Video = ({ src, style, showMarks, marks = {}, setMarks = () => {} }) => {
             videoRef.current.currentTime = value
           }}
         />
-        <Trackbar
-          value={trackbarPosition}
-          onInput={onSeek}
-          highlightStart={((marks.mark_in || 0) / videoDuration) * 100}
-          highlightEnd={
-            ((marks.mark_out || videoDuration) / videoDuration) * 100
-          }
-          ref={sliderRef}
-          onMouseLeave={handleFinishSlider}
-          onMouseUp={handleFinishSlider}
-        />
+        <div
+          style={{
+            flexGrow: 1,
+            position: 'relative',
+            display: 'flex',
+            padding: 0,
+            margin: 0,
+          }}
+        >
+          <Trackbar
+            value={trackbarPosition}
+            onInput={onSeek}
+            highlightStart={((marks.mark_in || 0) / videoDuration) * 100}
+            highlightEnd={
+              ((marks.mark_out || videoDuration) / videoDuration) * 100
+            }
+            posterFrame={((marks.poster_frame || 0) / videoDuration) * 100}
+            ref={sliderRef}
+            onMouseLeave={handleFinishSlider}
+            onMouseUp={handleFinishSlider}
+          />
+        </div>
         <InputTimecode value={videoDuration} readOnly={true} title="Duration" />
       </PlayoutControls>
 
@@ -280,42 +325,52 @@ const Video = ({ src, style, showMarks, marks = {}, setMarks = () => {} }) => {
           <>
             <Button
               icon="line_start_diamond"
-              title="Mark in"
+              tooltip="Mark in"
               onClick={() => onMarkIn(videoRef.current.currentTime)}
             />
             <Button
               icon="first_page"
-              title="Go to mark in"
+              tooltip="Go to mark in"
               onClick={onGoToIn}
             />
           </>
         )}
         <Button
           icon="keyboard_double_arrow_left"
+          tooltip="Back 5 frames"
           onClick={() => frameStep(-5)}
         />
-        <Button icon="chevron_left" onClick={() => frameStep(-1)} />
+        <Button
+          icon="chevron_left"
+          tooltip="Back 1 frame"
+          onClick={() => frameStep(-1)}
+        />
         <Button
           icon={playButtonIcon}
-          title="Play/Pause"
+          tooltip="Play/Pause"
           onClick={onPlay}
           ref={playButtonRef}
         />
-        <Button icon="chevron_right" onClick={() => frameStep(1)} />
+        <Button
+          icon="chevron_right"
+          tooltip="Forward 1 frame"
+          onClick={() => frameStep(1)}
+        />
         <Button
           icon="keyboard_double_arrow_right"
+          tooltip="Forward 5 frames"
           onClick={() => frameStep(5)}
         />
         {showMarks && (
           <>
             <Button
               icon="last_page"
-              title="Go to mark out"
+              tooltip="Go to mark out"
               onClick={onGoToOut}
             />
             <Button
               icon="line_end_diamond"
-              title="Mark out"
+              tooltip="Mark out"
               onClick={() => onMarkOut(videoRef.current.currentTime)}
             />
           </>
diff --git a/frontend/src/containers/Browser/Formatting.jsx b/frontend/src/containers/Browser/Formatting.jsx
index 3a380311..37b9df78 100644
--- a/frontend/src/containers/Browser/Formatting.jsx
+++ b/frontend/src/containers/Browser/Formatting.jsx
@@ -47,8 +47,7 @@ const getColumnWidth = (key) => {
 
 // Field formatters
 
-
-const getDefaultFormatter = (key) =>  {
+const getDefaultFormatter = (key) => {
   const metaType = nebula.metaType(key)
   switch (metaType.type) {
     case 'boolean':
@@ -91,7 +90,6 @@ const getDefaultFormatter = (key) =>  {
   } // switch metaType
 }
 
-
 const getFormatter = (key) => {
   if (['title', 'subtitle', 'description'].includes(key))
     // eslint-disable-next-line
diff --git a/frontend/src/pages/AssetEditor/Preview.jsx b/frontend/src/pages/AssetEditor/Preview.jsx
index 045e4718..fafeb434 100644
--- a/frontend/src/pages/AssetEditor/Preview.jsx
+++ b/frontend/src/pages/AssetEditor/Preview.jsx
@@ -4,6 +4,7 @@ import styled from 'styled-components'
 import { toast } from 'react-toastify'
 import { useState, useEffect } from 'react'
 import {
+  Dropdown,
   Video,
   Spacer,
   InputText,
@@ -98,6 +99,7 @@ const Preview = ({ assetData, setAssetData }) => {
   const accessToken = nebula.getAccessToken()
   const [selection, setSelection] = useState({})
   const [subclips, setSubclips] = useState([])
+  const [position, setPosition] = useState(0)
 
   useEffect(() => {
     setSelection({})
@@ -127,6 +129,31 @@ const Preview = ({ assetData, setAssetData }) => {
   const videoSrc =
     assetData.id && accessToken && `/proxy/${assetData.id}?token=${accessToken}`
 
+  const posterOptions = [
+    {
+      label: 'Set poster frame',
+      onClick: () => {
+        setAssetData((o) => {
+          return { ...o, poster_frame: position }
+        })
+      },
+    },
+    {
+      label: 'Go to poster frame',
+      onClick: () => {
+        setPosition(assetData.poster_frame)
+      },
+    },
+    {
+      label: 'Clear poster frame',
+      onClick: () => {
+        setAssetData((o) => {
+          return { ...o, poster_frame: null }
+        })
+      },
+    },
+  ]
+
   return (
     <div className="grow column" style={{ minWidth: 300, maxWidth: 600 }}>
       <section className="column">
@@ -134,8 +161,10 @@ const Preview = ({ assetData, setAssetData }) => {
           src={videoSrc}
           style={{ width: '100%' }}
           showMarks={true}
-          marks={selection}
+          marks={{ ...selection, poster_frame: assetData.poster_frame }}
           setMarks={setSelection}
+          position={position}
+          setPosition={setPosition}
         />
       </section>
       <section className="column">
@@ -143,21 +172,21 @@ const Preview = ({ assetData, setAssetData }) => {
           <InputTimecode
             value={assetData.mark_in}
             readOnly={true}
-            title="Content start"
+            tooltip="Content start"
           />
           <InputTimecode
             value={assetData.mark_out}
             readOnly={true}
-            title="Content end"
+            tooltip="Content end"
           />
           <Button
             icon="download"
-            title="Marks from selection"
+            tooltip="Marks from selection"
             onClick={onSetMarks}
           />
           <Button
             icon="upload"
-            title="Marks to selection"
+            tooltip="Marks to selection"
             onClick={() =>
               setSelection({
                 mark_in: assetData.mark_in || null,
@@ -168,7 +197,7 @@ const Preview = ({ assetData, setAssetData }) => {
           <Spacer />
           <Button
             icon="add"
-            label="New subclip"
+            tooltip="New subclip"
             onClick={() => {
               if (!(selection.mark_in && selection.mark_out)) {
                 toast.error('Please select a region first')
@@ -180,6 +209,7 @@ const Preview = ({ assetData, setAssetData }) => {
               ])
             }}
           />
+          <Dropdown icon="image" align="right" options={posterOptions} />
         </RegionRow>
       </section>