Import ContextMenuItem from contracts in desktop app

Remove the duplicate DesktopContextMenuItem interface and use the shared ContextMenuItem type from @t3tools/contracts instead.

Julius Marminge committed 8 months ago

Commit 9e2e73c1 parent c530e5d1 file · +2 −7Browse files

Unified · Split

 import { app, BrowserWindow, dialog, ipcMain, Menu, nativeImage, protocol, shell } from "electron";
 import type { MenuItemConstructorOptions } from "electron";
+import type { ContextMenuItem } from "@t3tools/contracts";
 import { showDesktopConfirmDialog } from "./confirmDialog";
 import { fixPath } from "./fixPath";
 let isQuitting = false;
 let desktopProtocolRegistered = false;
-interface DesktopContextMenuItem {
-  id: string;
-  label: string;
-  destructive?: boolean;
-}
 let destructiveMenuIconCache: Electron.NativeImage | null | undefined;
 function getDestructiveMenuIcon(): Electron.NativeImage | undefined {
     CONTEXT_MENU_CHANNEL,
     async (
       _event,
-      items: DesktopContextMenuItem[],
+      items: ContextMenuItem[],
       position?: { x: number; y: number },
     ) => {
       const normalizedItems = items

Keyboard shortcuts