fix: setting window can multi create (#64)

Signed-off-by: Innei <i@innei.in>
RSSNext · Jun 14, 2024 · 396a7d7 · 396a7d7
1 parent fbeb00e
commit 396a7d7
Showing 1 changed file with 19 additions and 2 deletions.
diff --git a/src/main/menu.ts b/src/main/menu.ts
@@ -1,8 +1,15 @@
-import type { MenuItem, MenuItemConstructorOptions } from "electron"
+import type {
+  BrowserWindow,
+  MenuItem,
+  MenuItemConstructorOptions,
+} from "electron"
 import { Menu } from "electron"
 
 import { createWindow } from "./window"
 
+const windows = {
+  settingWindow: null as BrowserWindow | null,
+}
 export const registerAppMenu = () => {
   const menus: Array<MenuItemConstructorOptions | MenuItem> = [
     {
@@ -14,10 +21,20 @@ export const registerAppMenu = () => {
           label: "Settings...",
           accelerator: "CmdOrCtrl+,",
           click: () => {
-            createWindow({
+            if (windows.settingWindow) {
+              windows.settingWindow.show()
+              return
+            }
+            const window = createWindow({
               extraPath: "/settings",
               width: 800,
               height: 600,
+              resizable: false,
+            })
+
+            windows.settingWindow = window
+            window.on("closed", () => {
+              windows.settingWindow = null
             })
           },
         },