From 592150638eb36b43492b9a206e4753c2b04dafb6 Mon Sep 17 00:00:00 2001
From: pengx17 <pengxiao@outlook.com>
Date: Thu, 22 Aug 2024 04:46:25 +0000
Subject: [PATCH] feat(electron): switch to next/previous tab with
 Ctrl+Tab/Ctrl+Shift+Tab (#7943)

fix PD-1569
---
 .../src/main/application-menu/create.ts       | 48 +++++++++++++++++++
 .../src/main/windows-manager/tab-views.ts     | 30 ++++++++++++
 2 files changed, 78 insertions(+)

diff --git a/packages/frontend/electron/src/main/application-menu/create.ts b/packages/frontend/electron/src/main/application-menu/create.ts
index 016dac844c..c85fe1db98 100644
--- a/packages/frontend/electron/src/main/application-menu/create.ts
+++ b/packages/frontend/electron/src/main/application-menu/create.ts
@@ -11,6 +11,8 @@ import {
   showDevTools,
   showMainWindow,
   switchTab,
+  switchToNextTab,
+  switchToPreviousTab,
   undoCloseTab,
 } from '../windows-manager';
 import { applicationMenuSubjects } from './subject';
@@ -152,6 +154,52 @@ export function createApplicationMenu() {
             visible: false,
           };
         }),
+        {
+          label: 'Switch to next tab',
+          accelerator: 'CommandOrControl+Tab',
+          click: () => {
+            switchToNextTab();
+          },
+        },
+        {
+          label: 'Switch to previous tab',
+          accelerator: 'CommandOrControl+Shift+Tab',
+          click: () => {
+            switchToPreviousTab();
+          },
+        },
+        {
+          label: 'Switch to next tab (mac)',
+          accelerator: 'Command+]',
+          visible: false,
+          click: () => {
+            switchToNextTab();
+          },
+        },
+        {
+          label: 'Switch to previous tab (mac)',
+          accelerator: 'Command+[',
+          visible: false,
+          click: () => {
+            switchToPreviousTab();
+          },
+        },
+        {
+          label: 'Switch to next tab (mac 2)',
+          accelerator: 'Alt+Command+]',
+          visible: false,
+          click: () => {
+            switchToNextTab();
+          },
+        },
+        {
+          label: 'Switch to previous tab (mac 2)',
+          accelerator: 'Alt+Command+[',
+          visible: false,
+          click: () => {
+            switchToPreviousTab();
+          },
+        },
       ],
     },
     {
diff --git a/packages/frontend/electron/src/main/windows-manager/tab-views.ts b/packages/frontend/electron/src/main/windows-manager/tab-views.ts
index a5e23a33a2..591d9044e3 100644
--- a/packages/frontend/electron/src/main/windows-manager/tab-views.ts
+++ b/packages/frontend/electron/src/main/windows-manager/tab-views.ts
@@ -233,6 +233,12 @@ export class WebContentViewsManager {
     );
   }
 
+  get activeWorkbenchIndex() {
+    return this.tabViewsMeta.workbenches.findIndex(
+      w => w.id === this.activeWorkbenchId
+    );
+  }
+
   get activeWorkbenchView() {
     return this.activeWorkbenchId
       ? this.webViewsMap$.value.get(this.activeWorkbenchId)
@@ -1023,3 +1029,27 @@ export const switchTab = (n: number) => {
     WebContentViewsManager.instance.showTab(item.id).catch(logger.error);
   }
 };
+
+export const switchToNextTab = () => {
+  const length =
+    WebContentViewsManager.instance.tabViewsMeta.workbenches.length;
+  const activeIndex = WebContentViewsManager.instance.activeWorkbenchIndex;
+  const item = WebContentViewsManager.instance.tabViewsMeta.workbenches.at(
+    activeIndex === length - 1 ? 0 : activeIndex + 1
+  );
+  if (item) {
+    WebContentViewsManager.instance.showTab(item.id).catch(logger.error);
+  }
+};
+
+export const switchToPreviousTab = () => {
+  const length =
+    WebContentViewsManager.instance.tabViewsMeta.workbenches.length;
+  const activeIndex = WebContentViewsManager.instance.activeWorkbenchIndex;
+  const item = WebContentViewsManager.instance.tabViewsMeta.workbenches.at(
+    activeIndex === 0 ? length - 1 : activeIndex - 1
+  );
+  if (item) {
+    WebContentViewsManager.instance.showTab(item.id).catch(logger.error);
+  }
+};