fix(search): fix cannot select auto lyric mode

- resolve #1125
organization · Oct 23, 2024 · bd50958 · bd50958
1 parent 4eeaec7
commit bd50958
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/common/schema/config.ts b/common/schema/config.ts
@@ -133,7 +133,7 @@ export const LyricMapperModeSchema = z.union([
   LyricMapperModeProviderSchema,
 ]);
 export const LyricMapperSchema = z.record(z.object({
-  mode: LyricMapperModeSchema.optional(),
+  mode: LyricMapperModeSchema.optional().nullable(),
   delay: z.number().optional(),
 }).optional());
 

diff --git a/renderer/components/PlayingInfoProvider.tsx b/renderer/components/PlayingInfoProvider.tsx
@@ -83,7 +83,7 @@ const PlayingInfoProvider = (props: { children: JSX.Element }) => {
 
     const mode = mapper[id()]?.mode;
 
-    if (mode === undefined) return 'auto';
+    if (!mode) return 'auto';
     if (mode.type === 'none') return 'none';
     if (mode.type === 'player') return 'player';
 

diff --git a/renderer/lyrics/SideBar.tsx b/renderer/lyrics/SideBar.tsx
@@ -38,7 +38,7 @@ const SideBar = () => {
   });
 
   const onChangeLyricMode = (mode: LyricMode) => {
-    let newMode: LyricMapperMode | undefined = undefined;
+    let newMode: LyricMapperMode | null = null;
     if (mode === 'player') newMode = { type: 'player' };
     if (mode === 'none') newMode = { type: 'none' };