fix zoom level shortcuts

RocketChat · Sep 29, 2023 · be9ee04 · be9ee04
1 parent df33261
commit be9ee04
Showing 1 changed file with 19 additions and 33 deletions.
diff --git a/src/ui/main/menuBar.ts b/src/ui/main/menuBar.ts
@@ -178,6 +178,22 @@ const selectViewDeps = createStructuredSelector<
   rootWindowState: ({ rootWindowState }) => rootWindowState,
 });
 
+const getCurrentViewWebcontents = async () => {
+  const browserWindow = await getRootWindow();
+
+  if (!browserWindow.isVisible()) {
+    browserWindow.showInactive();
+  }
+  browserWindow.focus();
+  const currentView = select(({ currentView }) => currentView);
+  const url = typeof currentView === 'object' ? currentView.url : null;
+  if (!url) {
+    return null;
+  }
+  console.log('zoom url', url);
+  return getWebContentsByServerUrl(url);
+};
+
 const createViewMenu = createSelector(
   selectViewDeps,
   ({
@@ -376,17 +392,7 @@ const createViewMenu = createSelector(
         label: t('menus.resetZoom'),
         accelerator: 'CommandOrControl+0',
         click: async () => {
-          const browserWindow = await getRootWindow();
-
-          if (!browserWindow.isVisible()) {
-            browserWindow.showInactive();
-          }
-          browserWindow.focus();
-          const url = typeof currentView === 'object' ? currentView.url : null;
-          if (!url) {
-            return;
-          }
-          const guestWebContents = getWebContentsByServerUrl(url);
+          const guestWebContents = await getCurrentViewWebcontents();
           guestWebContents?.setZoomLevel(0);
         },
       },
@@ -395,16 +401,7 @@ const createViewMenu = createSelector(
         label: t('menus.zoomIn'),
         accelerator: 'CommandOrControl+Plus',
         click: async () => {
-          const browserWindow = await getRootWindow();
-          if (!browserWindow.isVisible()) {
-            browserWindow.showInactive();
-          }
-          browserWindow.focus();
-          const url = typeof currentView === 'object' ? currentView.url : null;
-          if (!url) {
-            return;
-          }
-          const guestWebContents = getWebContentsByServerUrl(url);
+          const guestWebContents = await getCurrentViewWebcontents();
           if (!guestWebContents) {
             return;
           }
@@ -421,21 +418,10 @@ const createViewMenu = createSelector(
         label: t('menus.zoomOut'),
         accelerator: 'CommandOrControl+-',
         click: async () => {
-          const browserWindow = await getRootWindow();
-          if (!browserWindow.isVisible()) {
-            browserWindow.showInactive();
-          }
-          browserWindow.focus();
-          const url = typeof currentView === 'object' ? currentView.url : null;
-          if (!url) {
-            return;
-          }
-
-          const guestWebContents = getWebContentsByServerUrl(url);
+          const guestWebContents = await getCurrentViewWebcontents();
           if (!guestWebContents) {
             return;
           }
-
           const zoomLevel = guestWebContents.getZoomLevel();
           if (zoomLevel <= -9) {
             return;