use type, remove console

jeffvli · Jan 23, 2024 · 5e9ef9f · 5e9ef9f
1 parent b3a9e7c
commit 5e9ef9f
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 4 deletions.
diff --git a/src/main/features/core/settings/index.ts b/src/main/features/core/settings/index.ts
@@ -1,5 +1,6 @@
 import { ipcMain, nativeTheme, safeStorage } from 'electron';
 import Store from 'electron-store';
+import type { TitleTheme } from '/@/renderer/types';
 
 export const store = new Store();
 
@@ -49,7 +50,7 @@ ipcMain.handle('password-set', (_event, password: string, server: string) => {
     return false;
 });
 
-ipcMain.on('theme-set', (_event, theme: 'dark' | 'light' | 'system') => {
+ipcMain.on('theme-set', (_event, theme: TitleTheme) => {
     store.set('theme', theme);
     nativeTheme.themeSource = theme;
 });
diff --git a/src/main/main.ts b/src/main/main.ts
@@ -35,6 +35,7 @@ import { store } from './features/core/settings/index';
 import MenuBuilder from './menu';
 import { hotkeyToElectronAccelerator, isLinux, isMacOS, isWindows, resolveHtmlPath } from './utils';
 import './features';
+import type { TitleTheme } from '/@/renderer/types';
 
 declare module 'node-mpv';
 
@@ -416,7 +417,7 @@ const createWindow = async () => {
         new AppUpdater();
     }
 
-    const theme = store.get('theme') as 'dark' | 'light' | 'system' | undefined;
+    const theme = store.get('theme') as TitleTheme | undefined;
     nativeTheme.themeSource = theme || 'dark';
 };
 

diff --git a/src/main/preload/local-settings.ts b/src/main/preload/local-settings.ts
@@ -1,5 +1,6 @@
 import { IpcRendererEvent, ipcRenderer, webFrame } from 'electron';
 import Store from 'electron-store';
+import type { TitleTheme } from '/@/renderer/types';
 
 const store = new Store();
 
@@ -43,7 +44,7 @@ const fontError = (cb: (event: IpcRendererEvent, file: string) => void) => {
     ipcRenderer.on('custom-font-error', cb);
 };
 
-const themeSet = (theme: 'dark' | 'light' | 'system'): void => {
+const themeSet = (theme: TitleTheme): void => {
     ipcRenderer.send('theme-set', theme);
 };
 

diff --git a/src/renderer/features/settings/components/general/theme-settings.tsx b/src/renderer/features/settings/components/general/theme-settings.tsx
@@ -62,7 +62,6 @@ export const ThemeSettings = () => {
                             },
                         });
                         if (localSettings) {
-                            console.log(theme);
                             localSettings.themeSet(
                                 theme === AppTheme.DEFAULT_DARK ? 'dark' : 'light',
                             );

diff --git a/src/renderer/types.ts b/src/renderer/types.ts
@@ -209,3 +209,5 @@ export enum FontType {
     CUSTOM = 'custom',
     SYSTEM = 'system',
 }
+
+export type TitleTheme = 'dark' | 'light' | 'system';