From 5e8ad2e996632d105abd5f80920a667320cec921 Mon Sep 17 00:00:00 2001
From: Jip Fr <jipfrijlink@gmail.com>
Date: Sun, 12 Mar 2023 22:19:13 +0100
Subject: [PATCH] Localization, center loading, create divider action, rename
 season/episode route in EpisodeSelectionPopout

---
 src/components/popout/FloatingCard.tsx        |   2 +-
 src/setup/locales/en/translation.json         |   7 +
 src/video/components/VideoPlayer.tsx          |   3 +-
 src/video/components/VideoPlayerBase.tsx      |   4 +-
 .../components/actions/DividerAction.tsx      |  12 ++
 .../popouts/CaptionSelectionPopout.tsx        |   4 +-
 .../popouts/EpisodeSelectionPopout.tsx        | 127 ++----------------
 .../popouts/SourceSelectionPopout.tsx         |   4 +-
 8 files changed, 39 insertions(+), 124 deletions(-)
 create mode 100644 src/video/components/actions/DividerAction.tsx

diff --git a/src/components/popout/FloatingCard.tsx b/src/components/popout/FloatingCard.tsx
index 6cd0d363..8d894f16 100644
--- a/src/components/popout/FloatingCard.tsx
+++ b/src/components/popout/FloatingCard.tsx
@@ -172,7 +172,7 @@ export const FloatingCardView = {
   },
   Content(props: { children: React.ReactNode; noSection?: boolean }) {
     return (
-      <div className="grid h-full grid-rows-[auto,minmax(0,1fr)]">
+      <div className="grid h-full grid-rows-[1fr]">
         {props.noSection ? (
           <div className="relative h-full overflow-y-auto bg-ash-300">
             {props.children}
diff --git a/src/setup/locales/en/translation.json b/src/setup/locales/en/translation.json
index 8eb776ec..630709b6 100644
--- a/src/setup/locales/en/translation.json
+++ b/src/setup/locales/en/translation.json
@@ -75,6 +75,13 @@
       "errors": {
         "loadingWentWong": "Something went wrong loading the episodes for {{seasonTitle}}",
         "embedsError": "Something went wrong loading the embeds for this thing that you like"
+      },
+      "descriptions": {
+        "sources": "What provider do you want to use?",
+        "embeds": "Choose which video to view",
+        "seasons": "Choose which season you want to watch",
+        "episode": "Pick an episode",
+        "captions": "Choose a subtitle language"
       }
     },
     "errors": {
diff --git a/src/video/components/VideoPlayer.tsx b/src/video/components/VideoPlayer.tsx
index dea6526f..f86a5161 100644
--- a/src/video/components/VideoPlayer.tsx
+++ b/src/video/components/VideoPlayer.tsx
@@ -28,6 +28,7 @@ import { PopoutProviderAction } from "@/video/components/popouts/PopoutProviderA
 import { ChromecastAction } from "@/video/components/actions/ChromecastAction";
 import { CastingTextAction } from "@/video/components/actions/CastingTextAction";
 import { SettingsAction } from "./actions/SettingsAction";
+import { DividerAction } from "./actions/DividerAction";
 
 type Props = VideoPlayerBaseProps;
 
@@ -150,7 +151,7 @@ export function VideoPlayer(props: Props) {
                       <LeftSideControls />
                       <div className="flex-1" />
                       <SeriesSelectionAction />
-                      <div className="mx-2 h-6 w-px bg-white opacity-50" />
+                      <DividerAction />
                       <SettingsAction />
                       <ChromecastAction />
                       <AirplayAction />
diff --git a/src/video/components/VideoPlayerBase.tsx b/src/video/components/VideoPlayerBase.tsx
index 954bf0d1..57549729 100644
--- a/src/video/components/VideoPlayerBase.tsx
+++ b/src/video/components/VideoPlayerBase.tsx
@@ -27,7 +27,9 @@ function VideoPlayerBaseWithState(props: VideoPlayerBaseProps) {
 
   const children =
     typeof props.children === "function"
-      ? props.children({ isFullscreen: videoInterface.isFullscreen })
+      ? props.children({
+          isFullscreen: videoInterface.isFullscreen,
+        })
       : props.children;
 
   // TODO move error boundary to only decorated, <VideoPlayer /> shouldn't have styling
diff --git a/src/video/components/actions/DividerAction.tsx b/src/video/components/actions/DividerAction.tsx
new file mode 100644
index 00000000..ac1090f7
--- /dev/null
+++ b/src/video/components/actions/DividerAction.tsx
@@ -0,0 +1,12 @@
+import { useVideoPlayerDescriptor } from "@/video/state/hooks";
+import { useMeta } from "@/video/state/logic/meta";
+import { MWMediaType } from "@/backend/metadata/types";
+
+export function DividerAction() {
+  const descriptor = useVideoPlayerDescriptor();
+  const meta = useMeta(descriptor);
+
+  if (meta?.meta.meta.type !== MWMediaType.SERIES) return null;
+
+  return <div className="mx-2 h-6 w-px bg-white opacity-50" />;
+}
diff --git a/src/video/components/popouts/CaptionSelectionPopout.tsx b/src/video/components/popouts/CaptionSelectionPopout.tsx
index 43ed729f..d464e82e 100644
--- a/src/video/components/popouts/CaptionSelectionPopout.tsx
+++ b/src/video/components/popouts/CaptionSelectionPopout.tsx
@@ -51,8 +51,8 @@ export function CaptionSelectionPopout(props: {
       height={500}
     >
       <FloatingCardView.Header
-        title={t("videoPlayer.popouts.sources")}
-        description="What provider do you want to use?"
+        title={t("videoPlayer.popouts.captions")}
+        description={t("videoPlayer.popouts.descriptions.captions")}
         goBack={() => props.router.navigate("/")}
       />
       <FloatingCardView.Content noSection>
diff --git a/src/video/components/popouts/EpisodeSelectionPopout.tsx b/src/video/components/popouts/EpisodeSelectionPopout.tsx
index 6d5bed39..61218170 100644
--- a/src/video/components/popouts/EpisodeSelectionPopout.tsx
+++ b/src/video/components/popouts/EpisodeSelectionPopout.tsx
@@ -22,7 +22,7 @@ export function EpisodeSelectionPopout() {
     media: string;
   }>();
   const { t } = useTranslation();
-  const { pageProps, navigate } = useFloatingRouter("/season/episodes");
+  const { pageProps, navigate } = useFloatingRouter("/episodes");
 
   const descriptor = useVideoPlayerDescriptor();
   const meta = useMeta(descriptor);
@@ -84,7 +84,7 @@ export function EpisodeSelectionPopout() {
   const setSeason = (id: string) => {
     requestSeason(id);
     setCurrentVisibleSeason({ seasonId: id });
-    navigate("/season");
+    navigate("/episodes");
   };
 
   const { watched } = useWatchedContext();
@@ -95,11 +95,11 @@ export function EpisodeSelectionPopout() {
 
   return (
     <>
-      <FloatingView {...pageProps("episodes")} height={600} width={375}>
+      <FloatingView {...pageProps("seasons")} height={600} width={375}>
         <FloatingCardView.Header
-          title="Seasons"
-          description="Choose which season you want to watch"
-          goBack={() => navigate("/season")}
+          title={t("videoPlayer.popouts.seasons")}
+          description={t("videoPlayer.popouts.descriptions.seasons")}
+          goBack={() => navigate("/episodes")}
           backText={`To ${currentSeasonInfo?.title.toLowerCase()}`}
         />
         <FloatingCardView.Content>
@@ -116,16 +116,16 @@ export function EpisodeSelectionPopout() {
             : "No season"}
         </FloatingCardView.Content>
       </FloatingView>
-      <FloatingView {...pageProps("season")} height={600} width={375}>
+      <FloatingView {...pageProps("episodes")} height={600} width={375}>
         <FloatingCardView.Header
           title={currentSeasonInfo?.title ?? "Unknown season"}
-          description="Pick an episode"
+          description={t("videoPlayer.popouts.descriptions.episode")}
           goBack={closePopout}
           close
           action={
             <button
               type="button"
-              onClick={() => navigate("/season/episodes")}
+              onClick={() => navigate("/episodes/seasons")}
               className="flex cursor-pointer items-center space-x-2 transition-colors duration-200 hover:text-white"
             >
               <span>Other seasons</span>
@@ -146,7 +146,7 @@ export function EpisodeSelectionPopout() {
                   className="text-xl text-bink-600"
                 />
                 <p className="mt-6 w-full text-center">
-                  {t("videoPLayer.popouts.errors.loadingWentWrong", {
+                  {t("videoPlayer.popouts.errors.loadingWentWrong", {
                     seasonTitle: currentSeasonInfo?.title?.toLowerCase(),
                   })}
                 </p>
@@ -185,112 +185,5 @@ export function EpisodeSelectionPopout() {
         </FloatingCardView.Content>
       </FloatingView>
     </>
-    // <FloatingView show height={500} width={320}>
-    //   <div className="grid h-full grid-rows-[auto,minmax(0,1fr)]">
-    //     <PopoutSection className="bg-ash-100 font-bold text-white">
-    //       <div className="relative flex items-center">
-    //         <button
-    //           className={[
-    //             "-m-1.5 rounded-lg p-1.5 transition-opacity duration-100 hover:bg-ash-200",
-    //             isPickingSeason ? "pointer-events-none opacity-0" : "opacity-1",
-    //           ].join(" ")}
-    //           onClick={toggleIsPickingSeason}
-    //           type="button"
-    //         >
-    //           <Icon icon={Icons.CHEVRON_LEFT} />
-    //         </button>
-    //         <span
-    //           className={[
-    //             titlePositionClass,
-    //             !isPickingSeason ? "opacity-1" : "opacity-0",
-    //           ].join(" ")}
-    //         >
-    //           {currentSeasonInfo?.title || ""}
-    //         </span>
-    //         <span
-    //           className={[
-    //             titlePositionClass,
-    //             isPickingSeason ? "opacity-1" : "opacity-0",
-    //           ].join(" ")}
-    //         >
-    //           {t("videoPlayer.popouts.seasons")}
-    //         </span>
-    //       </div>
-    //     </PopoutSection>
-    //     <div className="relative grid h-full grid-rows-[minmax(1px,1fr)]">
-    //       <PopoutSection
-    //         className={[
-    //           "absolute inset-0 z-30 overflow-y-auto border-ash-400 bg-ash-100 transition-[max-height,padding] duration-200",
-    //           isPickingSeason
-    //             ? "max-h-full border-t"
-    //             : "max-h-0 overflow-hidden py-0",
-    //         ].join(" ")}
-    //       >
-    //         {currentSeasonInfo
-    //           ? meta?.seasons?.map?.((season) => (
-    //               <PopoutListEntry
-    //                 key={season.id}
-    //                 active={meta?.episode?.seasonId === season.id}
-    //                 onClick={() => setSeason(season.id)}
-    //                 isOnDarkBackground
-    //               >
-    //                 {season.title}
-    //               </PopoutListEntry>
-    //             ))
-    //           : "No season"}
-    //       </PopoutSection>
-    //       <PopoutSection className="relative h-full overflow-y-auto">
-    //         {loading ? (
-    //           <div className="flex h-full w-full items-center justify-center">
-    //             <Loading />
-    //           </div>
-    //         ) : error ? (
-    //           <div className="flex h-full w-full items-center justify-center">
-    //             <div className="flex flex-col flex-wrap items-center text-slate-400">
-    //               <IconPatch
-    //                 icon={Icons.EYE_SLASH}
-    //                 className="text-xl text-bink-600"
-    //               />
-    //               <p className="mt-6 w-full text-center">
-    //                 {t("videoPLayer.popouts.errors.loadingWentWrong", {
-    //                   seasonTitle: currentSeasonInfo?.title?.toLowerCase(),
-    //                 })}
-    //               </p>
-    //             </div>
-    //           </div>
-    //         ) : (
-    //           <div>
-    //             {currentSeasonEpisodes && currentSeasonInfo
-    //               ? currentSeasonEpisodes.map((e) => (
-    //                   <PopoutListEntry
-    //                     key={e.id}
-    //                     active={e.id === meta?.episode?.episodeId}
-    //                     onClick={() => {
-    //                       if (e.id === meta?.episode?.episodeId)
-    //                         controls.closePopout();
-    //                       else setCurrent(currentSeasonInfo.id, e.id);
-    //                     }}
-    //                     percentageCompleted={
-    //                       watched.items.find(
-    //                         (item) =>
-    //                           item.item?.series?.seasonId ===
-    //                             currentSeasonInfo.id &&
-    //                           item.item?.series?.episodeId === e.id
-    //                       )?.percentage
-    //                     }
-    //                   >
-    //                     {t("videoPlayer.popouts.episode", {
-    //                       index: e.number,
-    //                       title: e.title,
-    //                     })}
-    //                   </PopoutListEntry>
-    //                 ))
-    //               : "No episodes"}
-    //           </div>
-    //         )}
-    //       </PopoutSection>
-    //     </div>
-    //   </div>
-    // </FloatingView>
   );
 }
diff --git a/src/video/components/popouts/SourceSelectionPopout.tsx b/src/video/components/popouts/SourceSelectionPopout.tsx
index fe2d5ca2..9651b306 100644
--- a/src/video/components/popouts/SourceSelectionPopout.tsx
+++ b/src/video/components/popouts/SourceSelectionPopout.tsx
@@ -181,7 +181,7 @@ export function SourceSelectionPopout(props: {
       >
         <FloatingCardView.Header
           title={t("videoPlayer.popouts.sources")}
-          description="What provider do you want to use?"
+          description={t("videoPlayer.popouts.descriptions.sources")}
           goBack={() => props.router.navigate("/")}
         />
         <FloatingCardView.Content>
@@ -206,7 +206,7 @@ export function SourceSelectionPopout(props: {
       >
         <FloatingCardView.Header
           title={selectedProviderPopulated?.displayName ?? ""}
-          description="Choose which video to view"
+          description={t("videoPlayer.popouts.descriptions.embeds")}
           goBack={() => props.router.navigate(`/${props.prefix}`)}
         />
         <FloatingCardView.Content>