From 23742b8fddd08361f848f9bc3cfb539879783d3d Mon Sep 17 00:00:00 2001
From: an-lee <an.lee.work@gmail.com>
Date: Wed, 7 Aug 2024 14:22:31 +0800
Subject: [PATCH] Fix undefined issue and warnings (#943)

* fix undefined

* remove warnings
---
 enjoy/src/main/providers/youtube-provider.ts                | 2 +-
 .../renderer/components/medias/media-current-recording.tsx  | 3 ++-
 .../components/medias/media-transcription-read-button.tsx   | 1 +
 .../src/renderer/components/messages/assistant-message.tsx  | 3 +++
 enjoy/src/renderer/components/misc/bandu-login-form.tsx     | 6 +++++-
 enjoy/src/renderer/components/misc/github-login-form.tsx    | 6 +++++-
 enjoy/src/renderer/components/misc/mixin-login-form.tsx     | 6 +++++-
 .../renderer/components/stories/story-preview-toolbar.tsx   | 2 +-
 .../renderer/components/stories/story-vocabulary-sheet.tsx  | 6 +++++-
 .../components/transcriptions/transcription-edit-button.tsx | 5 ++++-
 enjoy/src/renderer/components/videos/ted-talks-segment.tsx  | 2 +-
 enjoy/src/renderer/components/videos/videos-component.tsx   | 4 ++--
 .../renderer/components/videos/youtube-videos-segment.tsx   | 2 +-
 enjoy/src/renderer/pages/conversation.tsx                   | 2 +-
 enjoy/src/renderer/pages/conversations.tsx                  | 4 ++--
 .../src/renderer/pages/pronunciation-assessments/index.tsx  | 3 ++-
 16 files changed, 41 insertions(+), 16 deletions(-)

diff --git a/enjoy/src/main/providers/youtube-provider.ts b/enjoy/src/main/providers/youtube-provider.ts
index 94883472..99aa0d57 100755
--- a/enjoy/src/main/providers/youtube-provider.ts
+++ b/enjoy/src/main/providers/youtube-provider.ts
@@ -59,7 +59,7 @@ export class YoutubeProvider {
             videoId: video.richItemRenderer.content.videoRenderer.videoId,
             duration:
               video.richItemRenderer.content.videoRenderer.lengthText
-                .simpleText,
+                ?.simpleText,
           };
         });
 
diff --git a/enjoy/src/renderer/components/medias/media-current-recording.tsx b/enjoy/src/renderer/components/medias/media-current-recording.tsx
index c451dac5..7ab0297d 100644
--- a/enjoy/src/renderer/components/medias/media-current-recording.tsx
+++ b/enjoy/src/renderer/components/medias/media-current-recording.tsx
@@ -616,7 +616,7 @@ export const MediaCurrentRecording = () => {
       </div>
 
       <AlertDialog open={isSharing} onOpenChange={setIsSharing}>
-        <AlertDialogContent>
+        <AlertDialogContent aria-describedby={undefined}>
           <AlertDialogHeader>
             <AlertDialogTitle>{t("shareRecording")}</AlertDialogTitle>
             <AlertDialogDescription>
@@ -634,6 +634,7 @@ export const MediaCurrentRecording = () => {
 
       <Sheet open={detailIsOpen} onOpenChange={(open) => setDetailIsOpen(open)}>
         <SheetContent
+          aria-describedby={undefined}
           side="bottom"
           className="rounded-t-2xl shadow-lg max-h-screen overflow-y-scroll"
           displayClose={false}
diff --git a/enjoy/src/renderer/components/medias/media-transcription-read-button.tsx b/enjoy/src/renderer/components/medias/media-transcription-read-button.tsx
index 0807a702..a6095c34 100644
--- a/enjoy/src/renderer/components/medias/media-transcription-read-button.tsx
+++ b/enjoy/src/renderer/components/medias/media-transcription-read-button.tsx
@@ -258,6 +258,7 @@ export const MediaTranscriptionReadButton = (props: {
         }}
       >
         <SheetContent
+          aria-describedby={undefined}
           side="bottom"
           className="rounded-t-2xl shadow-lg max-h-screen overflow-y-scroll"
           displayClose={false}
diff --git a/enjoy/src/renderer/components/messages/assistant-message.tsx b/enjoy/src/renderer/components/messages/assistant-message.tsx
index 4525c033..bb570f50 100644
--- a/enjoy/src/renderer/components/messages/assistant-message.tsx
+++ b/enjoy/src/renderer/components/messages/assistant-message.tsx
@@ -11,6 +11,7 @@ import {
   DropdownMenuTrigger,
   DropdownMenuContent,
   DropdownMenuItem,
+  SheetTitle,
 } from "@renderer/components/ui";
 import {
   SpeechPlayer,
@@ -294,6 +295,7 @@ export const AssistantMessageComponent = (props: {
         onOpenChange={(value) => setShadowing(value)}
       >
         <SheetContent
+          aria-describedby={undefined}
           side="bottom"
           className="h-screen p-0"
           displayClose={false}
@@ -301,6 +303,7 @@ export const AssistantMessageComponent = (props: {
           onInteractOutside={(event) => event.preventDefault()}
         >
           <SheetHeader className="flex items-center justify-center h-14">
+            <SheetTitle className="sr-only">{t("shadow")}</SheetTitle>
             <SheetClose>
               <ChevronDownIcon />
             </SheetClose>
diff --git a/enjoy/src/renderer/components/misc/bandu-login-form.tsx b/enjoy/src/renderer/components/misc/bandu-login-form.tsx
index 2eba46d1..4be47736 100644
--- a/enjoy/src/renderer/components/misc/bandu-login-form.tsx
+++ b/enjoy/src/renderer/components/misc/bandu-login-form.tsx
@@ -33,7 +33,11 @@ export const BanduLoginButton = () => {
           />
         </Button>
       </SheetTrigger>
-      <SheetContent side="bottom" className="h-screen">
+      <SheetContent
+        side="bottom"
+        className="h-screen"
+        aria-describedby={undefined}
+      >
         <div className="w-full h-full flex">
           <div className="m-auto">{open && <BanduLoginForm />}</div>
         </div>
diff --git a/enjoy/src/renderer/components/misc/github-login-form.tsx b/enjoy/src/renderer/components/misc/github-login-form.tsx
index b8d53392..1b6e432b 100644
--- a/enjoy/src/renderer/components/misc/github-login-form.tsx
+++ b/enjoy/src/renderer/components/misc/github-login-form.tsx
@@ -31,7 +31,11 @@ export const GithubLoginButton = () => {
           />
         </Button>
       </SheetTrigger>
-      <SheetContent side="bottom" className="h-screen">
+      <SheetContent
+        side="bottom"
+        className="h-screen"
+        aria-describedby={undefined}
+      >
         <div className="w-full h-full flex">
           <div className="m-auto">{open && <GithubLoginForm />}</div>
         </div>
diff --git a/enjoy/src/renderer/components/misc/mixin-login-form.tsx b/enjoy/src/renderer/components/misc/mixin-login-form.tsx
index bbbb0512..9048f727 100644
--- a/enjoy/src/renderer/components/misc/mixin-login-form.tsx
+++ b/enjoy/src/renderer/components/misc/mixin-login-form.tsx
@@ -32,7 +32,11 @@ export const MixinLoginButton = () => {
           />
         </Button>
       </SheetTrigger>
-      <SheetContent side="bottom" className="h-screen">
+      <SheetContent
+        side="bottom"
+        className="h-screen"
+        aria-describedby={undefined}
+      >
         <div className="w-full h-full flex">
           <div className="m-auto">{open && <MixinLoginForm />}</div>
         </div>
diff --git a/enjoy/src/renderer/components/stories/story-preview-toolbar.tsx b/enjoy/src/renderer/components/stories/story-preview-toolbar.tsx
index 90870b03..10edea30 100644
--- a/enjoy/src/renderer/components/stories/story-preview-toolbar.tsx
+++ b/enjoy/src/renderer/components/stories/story-preview-toolbar.tsx
@@ -35,7 +35,7 @@ export const StoryPreviewToolbar = (props: {
             <ScanTextIcon className="w-6 h-6" />
           </ToolbarButton>
         </AlertDialogTrigger>
-        <AlertDialogContent>
+        <AlertDialogContent aria-describedby={undefined}>
           <AlertDialogHeader>
             <AlertDialogTitle>{t("aiExtractVocabulary")}</AlertDialogTitle>
 
diff --git a/enjoy/src/renderer/components/stories/story-vocabulary-sheet.tsx b/enjoy/src/renderer/components/stories/story-vocabulary-sheet.tsx
index 2fc91108..fdec90af 100644
--- a/enjoy/src/renderer/components/stories/story-vocabulary-sheet.tsx
+++ b/enjoy/src/renderer/components/stories/story-vocabulary-sheet.tsx
@@ -43,7 +43,11 @@ export const StoryVocabularySheet = (props: {
         if (!value) setVocabularyVisible(null);
       }}
     >
-      <SheetContent side="bottom" className="rounded-t-2xl shadow-lg h-5/6">
+      <SheetContent
+        side="bottom"
+        className="rounded-t-2xl shadow-lg h-5/6"
+        aria-describedby={undefined}
+      >
         <SheetHeader className="flex items-center justify-center mb-2">
           <div className="text-center">
             <span className="font-semibold text-xl capitalize">
diff --git a/enjoy/src/renderer/components/transcriptions/transcription-edit-button.tsx b/enjoy/src/renderer/components/transcriptions/transcription-edit-button.tsx
index f7e5353f..c32eec80 100644
--- a/enjoy/src/renderer/components/transcriptions/transcription-edit-button.tsx
+++ b/enjoy/src/renderer/components/transcriptions/transcription-edit-button.tsx
@@ -75,7 +75,10 @@ export const TranscriptionEditButton = (props: {
           </Button>
         )}
       </DialogTrigger>
-      <DialogContent className="max-w-screen-sm xl:max-w-screen-md">
+      <DialogContent
+        className="max-w-screen-sm xl:max-w-screen-md"
+        aria-describedby={undefined}
+      >
         <DialogHeader>
           <DialogTitle>{t("editTranscription")}</DialogTitle>
         </DialogHeader>
diff --git a/enjoy/src/renderer/components/videos/ted-talks-segment.tsx b/enjoy/src/renderer/components/videos/ted-talks-segment.tsx
index c84c6712..81c4468a 100644
--- a/enjoy/src/renderer/components/videos/ted-talks-segment.tsx
+++ b/enjoy/src/renderer/components/videos/ted-talks-segment.tsx
@@ -177,7 +177,7 @@ export const TedTalksSegment = () => {
           if (!value) setSelectedTalk(null);
         }}
       >
-        <DialogContent>
+        <DialogContent aria-describedby={undefined}>
           <DialogHeader>
             <DialogTitle>{selectedTalk?.title}</DialogTitle>
           </DialogHeader>
diff --git a/enjoy/src/renderer/components/videos/videos-component.tsx b/enjoy/src/renderer/components/videos/videos-component.tsx
index f9762a6e..c0dd818f 100644
--- a/enjoy/src/renderer/components/videos/videos-component.tsx
+++ b/enjoy/src/renderer/components/videos/videos-component.tsx
@@ -252,7 +252,7 @@ export const VideosComponent = () => {
           setEditing(null);
         }}
       >
-        <DialogContent>
+        <DialogContent aria-describedby={undefined}>
           <DialogHeader>
             <DialogTitle>{t("editResource")}</DialogTitle>
           </DialogHeader>
@@ -272,7 +272,7 @@ export const VideosComponent = () => {
           setDeleting(null);
         }}
       >
-        <AlertDialogContent>
+        <AlertDialogContent aria-describedby={undefined}>
           <AlertDialogHeader>
             <AlertDialogTitle>{t("deleteResource")}</AlertDialogTitle>
             <AlertDialogDescription>
diff --git a/enjoy/src/renderer/components/videos/youtube-videos-segment.tsx b/enjoy/src/renderer/components/videos/youtube-videos-segment.tsx
index 4bda8136..a5f26ceb 100755
--- a/enjoy/src/renderer/components/videos/youtube-videos-segment.tsx
+++ b/enjoy/src/renderer/components/videos/youtube-videos-segment.tsx
@@ -119,7 +119,7 @@ export const YoutubeVideosSegment = (props: { channel: string }) => {
           if (!value) setSelectedVideo(null);
         }}
       >
-        <DialogContent>
+        <DialogContent aria-describedby={undefined}>
           <DialogHeader>
             <DialogTitle>{t("downloadVideo")}</DialogTitle>
           </DialogHeader>
diff --git a/enjoy/src/renderer/pages/conversation.tsx b/enjoy/src/renderer/pages/conversation.tsx
index f0073a39..ba5f35f5 100644
--- a/enjoy/src/renderer/pages/conversation.tsx
+++ b/enjoy/src/renderer/pages/conversation.tsx
@@ -249,7 +249,7 @@ export default () => {
               </div>
             </SheetTrigger>
 
-            <SheetContent className="p-0">
+            <SheetContent className="p-0" aria-describedby={undefined}>
               <SheetHeader>
                 <SheetTitle className="sr-only">
                   {t("editConversation")}
diff --git a/enjoy/src/renderer/pages/conversations.tsx b/enjoy/src/renderer/pages/conversations.tsx
index 5cc6a65f..45944bff 100644
--- a/enjoy/src/renderer/pages/conversations.tsx
+++ b/enjoy/src/renderer/pages/conversations.tsx
@@ -235,7 +235,7 @@ export default () => {
               </Button>
             </DialogTrigger>
 
-            <DialogContent>
+            <DialogContent aria-describedby={undefined}>
               <DialogHeader>
                 <DialogTitle>{t("selectAiRole")}</DialogTitle>
               </DialogHeader>
@@ -302,7 +302,7 @@ export default () => {
           </Dialog>
 
           <Sheet open={creating} onOpenChange={(value) => setCreating(value)}>
-            <SheetContent className="p-0">
+            <SheetContent className="p-0" aria-describedby={undefined}>
               <SheetHeader>
                 <SheetTitle className="sr-only">
                   {t("startConversation")}
diff --git a/enjoy/src/renderer/pages/pronunciation-assessments/index.tsx b/enjoy/src/renderer/pages/pronunciation-assessments/index.tsx
index 1b5e2c4c..8329b545 100644
--- a/enjoy/src/renderer/pages/pronunciation-assessments/index.tsx
+++ b/enjoy/src/renderer/pages/pronunciation-assessments/index.tsx
@@ -166,6 +166,7 @@ export default () => {
         }}
       >
         <SheetContent
+          aria-describedby={undefined}
           side="bottom"
           className="rounded-t-2xl shadow-lg max-h-screen overflow-y-scroll"
           displayClose={false}
@@ -190,7 +191,7 @@ export default () => {
           if (!value) setDeleting(null);
         }}
       >
-        <AlertDialogContent>
+        <AlertDialogContent aria-describedby={undefined}>
           <AlertDialogHeader>
             <AlertDialogTitle>{t("delete")}</AlertDialogTitle>
             <AlertDialogDescription>