Merge pull request #1385 from JulianKniephoff/nitpicks

Some stylistic nitpicks
opencast · Jun 21, 2024 · ddeb5db · ddeb5db
2 parents 055966f + a2a4e2a
commit ddeb5db
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/src/main/VideoPlayers.tsx b/src/main/VideoPlayers.tsx
@@ -304,12 +304,12 @@ export const VideoPlayer = React.forwardRef<VideoPlayerForwardRef, VideoPlayerPr
     };
 
     /**
-   * Workaround for subtitles not appearing in Firefox (or only appearing on inital mount, then disappearing
-   * when changed). Removes old tracks and readds them, because letting React to it does not seem
-   * to work properly.
-   * Fairly hacky, currently only works for a page with only one video
-   * https://github.com/CookPete/react-player/issues/490
-   */
+     * Workaround for subtitles not appearing in Firefox (or only appearing on inital mount, then disappearing
+     * when changed). Removes old tracks and readds them, because letting React to it does not seem
+     * to work properly.
+     * Fairly hacky, currently only works for a page with only one video
+     * https://github.com/CookPete/react-player/issues/490
+     */
     function reAddTrack() {
       const video = document.querySelector("video");
 
@@ -393,10 +393,14 @@ export const VideoPlayer = React.forwardRef<VideoPlayerForwardRef, VideoPlayerPr
       aspectRatio: "16 / 9",    // Hard-coded for now because there are problems with updating this value at runtime
 
       overflow: "hidden", // Required for borderRadius to show
-      ...(first) && { borderTopLeftRadius: "5px" },
-      ...(first) && { borderBottomLeftRadius: "5px" },
-      ...(last) && { borderTopRightRadius: "5px" },
-      ...(last) && { borderBottomRightRadius: "5px" },
+      ...first && {
+        borderTopLeftRadius: "5px",
+        borderBottomLeftRadius: "5px",
+      },
+      ...last && {
+        borderTopLeftRadius: "5px",
+        borderBottomRightRadius: "5px",
+      },
     });
 
     const videoPlayerWrapperStyles = css({