🤖 fix: Cmd+Q now properly quits app on macOS

The menu structure was not following Electron's standard pattern for macOS,
causing Cmd+Q to not trigger the quit action.

On macOS, the standard pattern is:
- App menu: Contains 'quit' (Cmd+Q)
- File menu: Contains 'close' (Cmd+W)
- Window menu: Contains 'minimize', 'zoom', 'front' (NOT 'close')

Previously we had 'close' in the Window menu which may have interfered with
Electron's accelerator registration. Now we follow the official Electron
documentation pattern, placing 'close' in the File menu on macOS.

Fixes #1100

Author: ibetitsmike

src/desktop/main.ts

@@ -143,6 +143,19 @@ function timestamp(): string {
 
 function createMenu() {
   const template: MenuItemConstructorOptions[] = [
+    {
+      label: "File",
+      submenu:
+        process.platform === "darwin"
+          ? [
+              // On macOS, "Close" (Cmd+W) lives in File menu, not Window menu
+              { role: "close" },
+            ]
+          : [
+              // On Windows/Linux, Quit (Ctrl+Q) lives in File menu
+              { role: "quit" },
+            ],
+    },
     {
       label: "Edit",
       submenu: [
@@ -182,7 +195,17 @@ function createMenu() {
     },
     {
       label: "Window",
-      submenu: [{ role: "minimize" }, { role: "close" }],
+      submenu:
+        process.platform === "darwin"
+          ? [
+              // macOS: Standard window management items
+              // Note: "close" goes in File menu on macOS (see Electron docs)
+              { role: "minimize" },
+              { role: "zoom" },
+              { type: "separator" },
+              { role: "front" },
+            ]
+          : [{ role: "minimize" }, { role: "close" }],
     },
   ];