Moved the button above the spellchecker switch b ecause the spellchecker switch has a subtitle associated with it that I don'tk now how to move so this makes more sense.

Author: coxtrent

app/common/typed-ipc.ts

@@ -5,7 +5,6 @@ export type MainMessage = {
   "clear-app-settings": () => void;
   "configure-spell-checker": () => void;
   "fetch-user-agent": () => string;
-  'zoom-other-tabs': (zoomLevel: number) => void;
   "focus-app": () => void;
   "focus-this-webview": () => void;
   "new-clipboard-key": () => {key: Uint8Array; sig: Uint8Array};
@@ -19,12 +18,12 @@ export type MainMessage = {
   "toggle-app": () => void;
   "toggle-badge-option": (newValue: boolean) => void;
   "toggle-menubar": (showMenubar: boolean) => void;
-  "toggle-one-zoom": (newValue: boolean) => void;
   toggleAutoLauncher: (AutoLaunchValue: boolean) => void;
   "unread-count": (unreadCount: number) => void;
   "update-badge": (messageCount: number) => void;
   "update-menu": (properties: MenuProperties) => void;
   "update-taskbar-icon": (data: string, text: string) => void;
+  "zoom-other-tabs": (zoomLevel: number) => void;
 };
 
 export type MainCall = {

app/main/index.ts

@@ -31,8 +31,6 @@ import {sentryInit} from "./sentry.js";
 import {setAutoLaunch} from "./startup.js";
 import {ipcMain, send} from "./typed-ipc-main.js";
 
-
-
 import "gatemaker/electron-setup"; // eslint-disable-line import/no-unassigned-import
 
 // eslint-disable-next-line @typescript-eslint/naming-convention
@@ -156,6 +154,7 @@ function createMainWindow(): BrowserWindow {
     app.quit();
     return;
   }
+  
   await app.whenReady();
 
   if (process.env.GDK_BACKEND !== GDK_BACKEND) {
@@ -280,13 +279,13 @@ function createMainWindow(): BrowserWindow {
     }
   });
 
-  ipcMain.on('zoom-other-tabs', (event, zoomLevel) => {
-      BrowserWindow.getAllWindows().forEach((window) => {
-          window.webContents.setZoomLevel(zoomLevel);
-      });
+  ipcMain.on("zoom-other-tabs", (event, zoomLevel) => {
+    const windows = BrowserWindow.getAllWindows();
+    for (const window of windows) {
+      window.webContents.setZoomLevel(zoomLevel);
+    }
   });
 
-
   ipcMain.on("fetch-user-agent", (event) => {
     event.returnValue = session
       .fromPartition("persist:webviewsession")

app/renderer/js/components/webview.ts

@@ -1,7 +1,6 @@
 import type {WebContents} from "electron/main";
 import fs from "node:fs";
 import process from "node:process";
-//import { ipcRenderer } from 'electron';
 
 import * as remote from "@electron/remote";
 import {app, dialog} from "@electron/remote";
@@ -159,15 +158,6 @@ export default class WebView {
     this.show();
   }
 
-  private syncZooms(): void {
-    // Sync zoom level with other tabs if useOneZoom is enabled
-    const useOneZoom = ConfigUtil.getConfigItem("useOneZoom", true);
-    if(useOneZoom) {
-      const zoomLevel = this.getWebContents().zoomLevel;
-      ipcRenderer.send('zoom-other-tabs', zoomLevel);
-    }
-  }
-
   zoomIn(): void {
     this.getWebContents().zoomLevel += 0.5;
     this.syncZooms();
@@ -237,6 +227,15 @@ export default class WebView {
     ipcRenderer.send("forward-to", this.webContentsId, channel, ...arguments_);
   }
 
+  private syncZooms(): void {
+    // Sync zoom level with other tabs if useOneZoom is enabled
+    const useOneZoom = ConfigUtil.getConfigItem("useOneZoom", true);
+    if (useOneZoom) {
+      const zoomLevel = this.getWebContents().getZoomLevel();
+      ipcRenderer.send("zoom-other-tabs", zoomLevel);
+    }
+  }
+
   private registerListeners(): void {
     const webContents = this.getWebContents();
 

app/renderer/js/pages/preference/general-section.ts

@@ -126,15 +126,16 @@ export function initGeneralSection({$root}: GeneralSectionProperties): void {
           </div>
           <div class="setting-control"></div>
         </div>
-        <div class="setting-row" id="enable-spellchecker-option">
+        <div class="setting-row" id="one-zoom-option">
           <div class="setting-description">
-            ${t.__("Enable spellchecker (requires restart)")}
+            ${t.__("Use one zoom for all server tabs")}
           </div>
           <div class="setting-control"></div>
         </div>
-        <div class="setting-row" id="one-zoom-option">
+        <div class="setting-row" id="enable-spellchecker-option">
           <div class="setting-description">
-            ${t.__("Use one zoom for all server tabs")}</div>
+            ${t.__("Enable spellchecker (requires restart)")}
+          </div>
           <div class="setting-control"></div>
         </div>
         <div
@@ -277,9 +278,11 @@ export function initGeneralSection({$root}: GeneralSectionProperties): void {
         const newValue = !ConfigUtil.getConfigItem("useOneZoom", false);
         ConfigUtil.setConfigItem("useOneZoom", newValue);
         useOneZoom();
+        if (newValue) {
+          ipcRenderer.send("zoom-other-tabs", 1);
+        }
       },
-    })
-    return;
+    });
   }
 
   function updateMenubarOption(): void {