Repository: smol-ai/GodMode
File: index.js (JavaScript)

Code before:
            window.reload();
          },
        },
        {
          label: 'Quick Open (use this!)',
          accelerator: store.get('quickOpenShortcut', quickOpenDefaultShortcut),
          click: () => {
            window.reload();
          },
        }, {
					label: 'Toggle Fullscreen',
					accelerator: 'CommandorControl+Shift+F',
					type: 'checkbox',
					checked: store.get('isFullscreen', false),
					click: () => {
						const fullscreen = !store.get('isFullscreen', false);
						store.set('isFullscreen', fullscreen);
						if (fullscreen) {
							window.setBounds({x: 0, width: width, height: window.getSize()[1]});
						} else {
							window.setBounds({x: width - 1200, width: 1200, height: window.getSize()[1]});
						}
					},
				}, {
          label: 'Settings',
          accelerator: 'CommandorControl+,',
          click: () => {
            const settingsWindow = new BrowserWindow({
              parent: null,
              modal: false,
              alwaysOnTop: true,
              show: false,
              autoHideMenuBar: true,
              width: 500,
              height: 300,
              webPreferences: {
                preload: path.join(__dirname, 'src', 'settings-preload.js'),
                nodeIntegration: false,
                contextIsolation: true,
              },
            });
            settingsWindow.loadURL('http://localhost:5173/settings.html');
            settingsWindow.openDevTools({
              mode: 'detach',
            });
            settingsWindow.once('ready-to-show', () => {
              mb.hideWindow();
              settingsWindow.show();
            });
          },
        },

Review comment (suggestion):
1. doesnt really need an accelerator right? its not something we expect people to use often
2a. we could move it up and close to name it Change Quick Open Shortcut since it only does one thing right now?
2b. alternative idea (which we can also reject independently), maybe we move the "Super Prompt Enter Key" functionality into the settings menu since that's also not a frequent thing

Code after:
				{
					label: 'Reload',
					accelerator: 'CommandorControl+R',
					click: () => {
						window.reload();
					},
				},
				{
					label: 'Quick Open (use this!)',
					accelerator: store.get('quickOpenShortcut', quickOpenDefaultShortcut),
					click: () => {
						window.reload();
					},
				},
				{
					label: 'Change Quick Open Shortcut',
					click: () => {
						if (settingsWindow && !settingsWindow.isDestroyed()) {
							// If the settings window is already open, just focus it
							settingsWindow.show();
							settingsWindow.focus();
							return;
						}
						settingsWindow = new BrowserWindow({
							show: true,
							width: 380,
							height: 200,
							titleBarStyle: 'hidden',
							minimizable: false,
							fullscreenable: false,
							maximizable: false,
							webPreferences: {
								preload: path.join(
									__dirname,
									'settings',
									'settings-preload.js',
								),
								contextIsolation: true,
							},
						});

						settingsWindow.loadFile(
							path.join(__dirname, 'settings', 'settings.html'),
						);
						if (process.env.NODE_ENV === 'development') {
							// open devtools if in dev mode
							settingsWindow.openDevTools({
								mode: 'detach',
							});
						}