From e2a2a5923901a6b55962c07bc30b1eddd0aae55e Mon Sep 17 00:00:00 2001
From: Eddasol <esole@equinor.com>
Date: Wed, 11 Dec 2024 14:46:18 +0100
Subject: [PATCH] Remove hardcoded video name

---
 .../Pages/MissionPage/VideoStream/FullScreenVideo.tsx      | 6 +++---
 .../MissionPage/VideoStream/VideoPlayerSimpleStream.tsx    | 7 +------
 .../Pages/MissionPage/VideoStream/VideoStreamCards.tsx     | 6 +++---
 .../Pages/MissionPage/VideoStream/VideoStreamWindow.tsx    | 6 ++----
 4 files changed, 9 insertions(+), 16 deletions(-)

diff --git a/frontend/src/components/Pages/MissionPage/VideoStream/FullScreenVideo.tsx b/frontend/src/components/Pages/MissionPage/VideoStream/FullScreenVideo.tsx
index 17376911d..d457c0a00 100644
--- a/frontend/src/components/Pages/MissionPage/VideoStream/FullScreenVideo.tsx
+++ b/frontend/src/components/Pages/MissionPage/VideoStream/FullScreenVideo.tsx
@@ -29,7 +29,7 @@ const FullScreenCard = styled.div`
 
 interface IFullScreenVideoStreamCardProps {
     videoStream: MediaStream
-    videoStreamName: string
+    videoStreamName?: string
     toggleFullScreenMode: VoidFunction
 }
 
@@ -56,8 +56,8 @@ export const FullScreenVideoStreamCard = ({
     // Rotated stream is not supported for simpleplayer
     return (
         <FullScreenCard>
-            <Typography variant="h5">{videoStreamName}</Typography>
-            <VideoPlayerSimpleStream videoStream={videoStream} videoStreamName={videoStreamName} />
+            {videoStreamName && <Typography variant="h5">{videoStreamName}</Typography>}
+            <VideoPlayerSimpleStream videoStream={videoStream} />
             {fullScreenExitButton(false)}
         </FullScreenCard>
     )
diff --git a/frontend/src/components/Pages/MissionPage/VideoStream/VideoPlayerSimpleStream.tsx b/frontend/src/components/Pages/MissionPage/VideoStream/VideoPlayerSimpleStream.tsx
index c77b30f65..2fb6432e3 100644
--- a/frontend/src/components/Pages/MissionPage/VideoStream/VideoPlayerSimpleStream.tsx
+++ b/frontend/src/components/Pages/MissionPage/VideoStream/VideoPlayerSimpleStream.tsx
@@ -1,9 +1,4 @@
-interface IVideoPlayerProps {
-    videoStream: MediaStream
-    videoStreamName: string
-}
-
-export const VideoPlayerSimpleStream = ({ videoStream, videoStreamName }: IVideoPlayerProps) => (
+export const VideoPlayerSimpleStream = ({ videoStream }: { videoStream: MediaStream }) => (
     <video
         autoPlay
         muted
diff --git a/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamCards.tsx b/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamCards.tsx
index 1d56512a3..c9628310c 100644
--- a/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamCards.tsx
+++ b/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamCards.tsx
@@ -23,7 +23,7 @@ const StyledVideoSection = styled.div`
 
 interface IVideoStreamCardProps {
     videoStream: MediaStream
-    videoStreamName: string
+    videoStreamName?: string
     toggleFullScreenMode: VoidFunction
     setFullScreenStream: Function
 }
@@ -49,11 +49,11 @@ export const VideoStreamCard = ({
         <VideoCard style={{ boxShadow: tokens.elevation.raised }}>
             <div onDoubleClick={turnOnFullScreen}>
                 <StyledVideoSection>
-                    <VideoPlayerSimpleStream videoStream={videoStream} videoStreamName={videoStreamName} />
+                    <VideoPlayerSimpleStream videoStream={videoStream} />
                     {fullScreenButton}
                 </StyledVideoSection>
             </div>
-            <Typography variant="h5">{videoStreamName}</Typography>
+            {videoStreamName && <Typography variant="h5">{videoStreamName}</Typography>}
         </VideoCard>
     )
 }
diff --git a/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamWindow.tsx b/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamWindow.tsx
index 983f6d72f..d8dcd6dae 100644
--- a/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamWindow.tsx
+++ b/frontend/src/components/Pages/MissionPage/VideoStream/VideoStreamWindow.tsx
@@ -42,13 +42,11 @@ export const VideoStreamWindow = ({ videoStreams }: VideoStreamWindowProps) => {
         toggleFullScreenMode()
     }
 
-    const videoStreamName = 'video' // TODO: decide if we want a name for it
-
     const videoCards = videoStreams.map((videoStream, index) => (
         <VideoStreamCard
             key={index}
             videoStream={new MediaStream([videoStream])}
-            videoStreamName={videoStreamName}
+            videoStreamName={undefined}
             toggleFullScreenMode={toggleFullScreenMode}
             setFullScreenStream={updateFullScreenStream}
         />
@@ -64,7 +62,7 @@ export const VideoStreamWindow = ({ videoStreams }: VideoStreamWindowProps) => {
                     <VideoFullScreen isOpen={fullScreenMode} onRequestClose={toggleFullScreenMode}>
                         <FullScreenVideoStreamCard
                             videoStream={videoStream}
-                            videoStreamName={videoStreamName}
+                            videoStreamName={undefined}
                             toggleFullScreenMode={toggleFullScreenMode}
                         />
                     </VideoFullScreen>