From e4b108f22c6c1e3723e43fa87acaa4bdc6e552da Mon Sep 17 00:00:00 2001
From: KaniPan <24561326+bluekani@users.noreply.github.com>
Date: Tue, 23 Jun 2026 15:10:26 +0900
Subject: [PATCH] fix: Restore hotkey edit and remove actions (#11374)

---
 .github/workflows/build.yml                                 | 2 +-
 .../src/components/multiHotkeyInput.component.pug           | 4 ++--
 .../src/components/multiHotkeyInput.component.scss          | 1 +
 tabby-settings/src/components/multiHotkeyInput.component.ts | 6 ++++--
 4 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index f78bf49f..4bc3f877 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -298,7 +298,7 @@ jobs:
       if: matrix.build-arch == 'x64'
 
   Windows-Build:
-    runs-on: windows-latest
+    runs-on: windows-2022
     needs: Lint
     strategy:
       matrix:
diff --git a/tabby-settings/src/components/multiHotkeyInput.component.pug b/tabby-settings/src/components/multiHotkeyInput.component.pug
index d0041fae..d8e7de2b 100644
--- a/tabby-settings/src/components/multiHotkeyInput.component.pug
+++ b/tabby-settings/src/components/multiHotkeyInput.component.pug
@@ -1,8 +1,8 @@
 .item(*ngFor='let hotkey of hotkeys; let i = index')
-    .body((click)='editItem(i)')
+    .body((mousedown)='editItem(i, $event)')
         .stroke(*ngFor='let stroke of castAny(hotkey.strokes)')
             span(*ngIf='!hotkey.isDuplicate') {{stroke}}
             span.duplicate(*ngIf='hotkey.isDuplicate') {{stroke}}
-    .remove((click)='removeItem(i)') &times;
+    .remove((mousedown)='removeItem(i, $event)') &times;
 
 .add((click)='addItem()', translate) Add...
diff --git a/tabby-settings/src/components/multiHotkeyInput.component.scss b/tabby-settings/src/components/multiHotkeyInput.component.scss
index a47373c2..6050b67a 100644
--- a/tabby-settings/src/components/multiHotkeyInput.component.scss
+++ b/tabby-settings/src/components/multiHotkeyInput.component.scss
@@ -28,6 +28,7 @@
 .add {
     flex: auto;
     opacity: 0;
+    white-space: nowrap;
 
     &:first-child {
         opacity: 1;
diff --git a/tabby-settings/src/components/multiHotkeyInput.component.ts b/tabby-settings/src/components/multiHotkeyInput.component.ts
index fcff602e..4c11b764 100644
--- a/tabby-settings/src/components/multiHotkeyInput.component.ts
+++ b/tabby-settings/src/components/multiHotkeyInput.component.ts
@@ -22,7 +22,8 @@ export class MultiHotkeyInputComponent {
         this.hotkeys = this.hotkeys.map(hotkey => typeof hotkey.strokes === 'string' ? { ...hotkey, strokes: [hotkey.strokes] } : hotkey)
     }
 
-    editItem (index: number): void {
+    editItem (index: number, event?: MouseEvent): void {
+        if (event && event.button !== 0) { return } // Ignore non-left clicks
         this.ngbModal.open(HotkeyInputModalComponent).result.then((newStrokes: string[]) => {
             if (this.hotkeys[index]) {
                 this.hotkeys[index].strokes = newStrokes
@@ -38,7 +39,8 @@ export class MultiHotkeyInputComponent {
         })
     }
 
-    removeItem (index: number): void {
+    removeItem (index: number, event?: MouseEvent): void {
+        if (event && event.button !== 0) { return } // Ignore non-left clicks
         this.hotkeys = this.hotkeys.filter((_, i) => i !== index)
         this.storeUpdatedHotkeys()
     }