Merge pull request #881 from AppQuality/fix-set-current-time

FIX(ObservationTooltip.tsx): fix seekPlayer function call duplication

Author: iDome89

src/pages/Video/components/ObservationTooltip.tsx

@@ -59,8 +59,8 @@ export const ObservationTooltip = ({
       size="large"
       color={color}
       onClick={() => {
-        seekPlayer?.(start);
         setOpenAccordion({ id: observationId });
+        seekPlayer?.(start);
       }}
       isSelecting={isSelecting}
     >

src/pages/Video/components/Player.tsx

@@ -52,7 +52,7 @@ const CorePlayer = ({ video }: { video: GetVideosByVidApiResponse }) => {
   const [postVideoByVidObservations] = usePostVideosByVidObservationsMutation();
   const [patchObservation] = usePatchVideosByVidObservationsAndOidMutation();
   const [start, setStart] = useState<number | undefined>(undefined);
-  const { context, setIsPlaying } = usePlayerContext();
+  const { context } = usePlayerContext();
   const { currentTime } = context.player || { currentTime: 0 };
   const { addToast } = useToast();
 
@@ -127,8 +127,6 @@ const CorePlayer = ({ video }: { video: GetVideosByVidApiResponse }) => {
     (time: number) => {
       if (videoRef && videoRef.current) {
         videoRef.current.currentTime = time;
-        setIsPlaying(true);
-        videoRef.current.play();
       }
     },
     [videoRef]
@@ -159,8 +157,8 @@ const CorePlayer = ({ video }: { video: GetVideosByVidApiResponse }) => {
           />
         ),
         onClick: () => {
-          seekPlayer(obs.start);
           setOpenAccordion({ id: obs.id });
+          seekPlayer(obs.start);
         },
         tags: obs.tags,
       })),