Skip to content

Commit 1953785

Browse files
committed
Disable 'select all' everywhere - it's not what anyone wants
In future, when we have multi-select for events, we could use this to select all those on the View page, but we'd probably do that as a shortcut/button/etc there, rather than in the app or context menus.
1 parent 5e70afd commit 1953785

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/index.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,10 @@ ipcMain.handle('select-application', () => {
596596
});
597597

598598
// Enable the default context menu
599-
registerContextMenu({ showSaveImageAs: true });
599+
registerContextMenu({
600+
showSaveImageAs: true,
601+
showSelectAll: false // Weird (does web-style select-all-text), skip it
602+
});
600603

601604
// Enable custom context menus, for special cases where the UI wants to define the options available
602605
ipcMain.handle('open-context-menu', (_event: {}, options: ContextMenuDefinition) => openContextMenu(options));

src/menu.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ export const getMenu = (browserWindows: Electron.BrowserWindow[]) => {
1919
{ role: 'copy', registerAccelerator: false },
2020
{ role: 'paste', registerAccelerator: false },
2121
{ role: 'pasteAndMatchStyle', registerAccelerator: false },
22-
{ role: 'delete' },
23-
{ role: 'selectAll' }
22+
{ role: 'delete' }
2423
]
2524
},
2625
{

0 commit comments

Comments
 (0)