feat: tab list context menu

addons/launcher/locales/zh-CN.json

@@ -1,6 +1,8 @@
 {
-  "Launch#!launcher.1": "启动",
-  "Launcher#!launcher.2": "启动项",
+  "Launcher#!launcher.1": "启动项",
+  "Launch#!launcher.2": "启动",
+  "Open in External#!launcher.3": "在外部打开",
+  "Remove Launcher#!launcher.4": "移除启动项",
   "Persistent Launcher Session#!settings.label.launcher.session.persist": "保留启动项会话",
   "Persistent the history of a launcher after it has been closed#!settings.comments.0.launcher.session.persist": "在启动项关闭后保留其历史记录"
 }

addons/launcher/src/renderer/LauncherList.vue

@@ -80,11 +80,6 @@ function closeTab(tab: TerminalTab) {
   commas.workspace.closeTerminalTab(tab)
 }
 
-function dropLauncher(launcher: Launcher) {
-  const index = launchers.findIndex(item => item.id === launcher.id)
-  removeLauncher(index)
-}
-
 function customizeLauncher(launcher: Launcher, title: string) {
   const index = launchers.findIndex(item => item.id === launcher.id)
   updateLauncher(index, {
@@ -93,25 +88,6 @@ function customizeLauncher(launcher: Launcher, title: string) {
   })
 }
 
-function showLauncherScripts(launcher: Launcher, event: MouseEvent) {
-  const scripts = launcher.scripts ?? []
-  commas.ui.openContextMenu([
-    {
-      label: 'Launch#!launcher.1',
-      command: 'start-launcher',
-      args: [launcher],
-    },
-    {
-      type: 'separator',
-    },
-    ...scripts.map((script, index) => ({
-      label: script.name,
-      command: 'run-script',
-      args: [launcher, index],
-    })),
-  ], event)
-}
-
 function showLauncherMenu(event: MouseEvent) {
   commas.workspace.showTabOptions(event, 'launcher')
 }
@@ -195,6 +171,40 @@ function handleDrop(args: DraggableElementEventPayload<LauncherDraggableElementD
     draggingEdges.set(args.self.data.launcher.id, null)
   }
 }
+
+function openLauncherMenu(launcher: Launcher, tab: TerminalTab | undefined, event: MouseEvent) {
+  const scripts = launcher.scripts ?? []
+  const { updatingItems, deletingItems } = commas.workspace.createTerminalTabContextMenu()
+  commas.ui.openContextMenu([
+    ...commas.ui.withContextMenuSeparator([
+      {
+        label: 'Launch#!launcher.2',
+        command: 'start-launcher',
+        args: [launcher],
+      },
+      {
+        label: 'Open in External#!launcher.3',
+        command: 'start-launcher-externally',
+        args: [launcher],
+      },
+    ], []),
+    ...commas.ui.withContextMenuSeparator(
+      scripts.map((script, index) => ({
+        label: script.name,
+        command: 'run-script',
+        args: [launcher, index],
+      })),
+      [],
+    ),
+    ...(tab ? commas.ui.withContextMenuSeparator(updatingItems, []) : []),
+    ...(tab ? deletingItems : []),
+    {
+      label: 'Remove Launcher#!launcher.4',
+      command: 'remove-launcher',
+      args: [launcher],
+    },
+  ], event)
+}
 </script>
 
 <template>
@@ -221,7 +231,7 @@ function handleDrop(args: DraggableElementEventPayload<LauncherDraggableElementD
           index,
           launcher,
           create: () => openLauncher(launcher),
-          dispose: () => dropLauncher(launcher),
+          dispose: () => removeLauncher(launcher),
         }"
         @dragstart="handleDragStart"
         @drop="handleDragStop"
@@ -253,12 +263,12 @@ function handleDrop(args: DraggableElementEventPayload<LauncherDraggableElementD
               @click="openLauncher(launcher, { tab })"
               @close="closeTab(tab!)"
               @customize="customizeLauncher(launcher, $event)"
+              @contextmenu="openLauncherMenu(launcher, tab, $event)"
             >
               <template #operations>
                 <div
                   class="button launch"
                   @click.stop="startLauncher(launcher)"
-                  @contextmenu="showLauncherScripts(launcher, $event)"
                 >
                   <VisualIcon name="lucide-play" />
                 </div>

addons/launcher/src/renderer/index.ts

@@ -2,7 +2,7 @@ import * as commas from 'commas:api/renderer'
 import { watch } from 'vue'
 import LauncherLink from './LauncherLink.vue'
 import LauncherList from './LauncherList.vue'
-import { getLauncherByTerminalTabCharacter, openLauncher, runLauncherScript, startLauncher, useLauncherCharacters, useLaunchers } from './launcher'
+import { getLauncherByTerminalTabCharacter, openLauncher, removeLauncher, runLauncherScript, startLauncher, startLauncherExternally, useLauncherCharacters, useLaunchers } from './launcher'
 import { clearLauncherSessions, LauncherSessionAddon } from './session'
 
 declare module '../../../../src/typings/terminal' {
@@ -24,15 +24,21 @@ export default () => {
   commas.ipcRenderer.on('start-launcher', (event, launcher) => {
     startLauncher(launcher)
   })
+  commas.ipcRenderer.on('start-launcher-externally', (event, launcher) => {
+    startLauncherExternally(launcher)
+  })
   commas.ipcRenderer.on('run-script', (event, launcher, index) => {
     runLauncherScript(launcher, index)
   })
+  commas.ipcRenderer.on('remove-launcher', (event, launcher) => {
+    removeLauncher(launcher)
+  })
 
   const characters = $(useLauncherCharacters())
 
   commas.app.effect(() => {
     commas.context.provide('terminal.category', {
-      title: 'Launcher#!launcher.2',
+      title: 'Launcher#!launcher.1',
       characters,
       command: 'open-launcher-character',
     })

addons/launcher/src/renderer/launcher.ts

@@ -145,9 +145,10 @@ export function moveLauncher(launcher: Launcher, index: number, edge?: 'start' |
   launchers = updated
 }
 
-export function removeLauncher(index: number) {
-  const launcherTabs = getTerminalTabsByLauncher(launchers[index])
+export function removeLauncher(launcher: Launcher) {
+  const launcherTabs = getTerminalTabsByLauncher(launcher)
   const updated = [...launchers]
+  const index = updated.findIndex(item => item.id === launcher.id)
   updated.splice(index, 1)
   launchers = updated
   for (const tab of launcherTabs) {

api/modules/workspace.ts

@@ -16,6 +16,7 @@ import {
   useTerminalTabGroupSeparating,
   useTerminalTabs,
 } from '../../src/renderer/compositions/terminal'
+import { createTerminalTabContextMenu } from '../../src/renderer/utils/terminal'
 import type { TerminalInfo, TerminalTab, TerminalTabAddons, TerminalTabPane } from '../../src/typings/terminal'
 import type { RendererAPIContext } from '../types'
 
@@ -129,4 +130,5 @@ export {
   useTerminalTabs,
   useTerminalTabGroupSeparating,
   getTerminalTabIndex,
+  createTerminalTabContextMenu,
 }

src/renderer/components/TabList.vue

@@ -20,9 +20,9 @@ import {
   useTerminalTabs,
 } from '../compositions/terminal'
 import type { DraggableElementData, DraggableTabData } from '../utils/draggable'
-import { openContextMenu } from '../utils/frame'
+import { openContextMenu, withContextMenuSeparator } from '../utils/frame'
 import { handleMousePressing } from '../utils/helper'
-import { getShells } from '../utils/terminal'
+import { createTerminalTabContextMenu, getShells } from '../utils/terminal'
 import TabItem from './TabItem.vue'
 import AutoScroll from './basic/AutoScroll.vue'
 import DraggableElement from './basic/DraggableElement.vue'
@@ -167,6 +167,14 @@ function handleGroupSeparating(args: DraggableElementEventPayload<DraggableEleme
 }
 
 const enableSash = false
+
+function openTabItemMenu(event: MouseEvent) {
+  const { updatingItems, deletingItems } = createTerminalTabContextMenu()
+  openContextMenu([
+    ...withContextMenuSeparator(updatingItems, []),
+    ...deletingItems,
+  ], event)
+}
 </script>
 
 <template>
@@ -205,6 +213,7 @@ const enableSash = false
                     :ref="draggable"
                     :tab="tab"
                     @click="activateTerminalTab(tab)"
+                    @contextmenu="openTabItemMenu"
                   />
                 </div>
               </DropTarget>

src/renderer/utils/terminal.ts

@@ -3,6 +3,7 @@ import * as path from 'node:path'
 import { ipcRenderer } from 'electron'
 import { parse } from 'shell-quote'
 import { omitHome, resolveWindowsDisk } from '../../shared/terminal'
+import type { MenuItem } from '../../typings/menu'
 import type { TerminalTab } from '../../typings/terminal'
 import icons from '../assets/icons'
 
@@ -100,3 +101,28 @@ export function getReadableSignal(code: number) {
       ?.[0]
   }
 }
+export function createTerminalTabContextMenu() {
+  let updatingItems: MenuItem[] = [
+    {
+      label: 'Duplicate Tab#!menu.duplicatetab',
+      accelerator: 'CmdOrCtrl+D',
+      command: 'duplicate-tab',
+    },
+    {
+      label: 'Split Tab#!menu.splittab',
+      accelerator: 'CmdOrCtrl+Shift+D',
+      command: 'split-tab',
+    },
+  ]
+  const deletingItems: MenuItem[] = [
+    {
+      label: 'Close Tab#!menu.closetab',
+      accelerator: 'CmdOrCtrl+W',
+      command: 'close-tab',
+    },
+  ]
+  return {
+    updatingItems,
+    deletingItems,
+  }
+}