SDA-1857 - Add accelerator for zoom in for MacOS (#913)

finos · Mar 12, 2020 · 81ad1c5 · 81ad1c5
1 parent a114fd9
commit 81ad1c5
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/src/app/app-menu.ts b/src/app/app-menu.ts
@@ -41,6 +41,10 @@ const windowsAccelerator = Object.assign({
     zoomOut: 'Ctrl+-',
 });
 
+const macAccelerator = Object.assign({
+    zoomIn: 'CommandOrControl+=',
+});
+
 let {
     minimizeOnClose,
     launchOnStartup,
@@ -490,10 +494,15 @@ export class AppMenu {
      */
     private assignRoleOrLabel({ role, label }: MenuItemConstructorOptions): MenuItemConstructorOptions {
         logger.info(`app-menu: assigning role & label respectively for ${role} & ${label}`);
-        if (isMac || isLinux) {
+        if (isLinux) {
             return label ? { role, label } : { role };
         }
 
+        if (isMac) {
+            return label ? { role, label, accelerator: role ? macAccelerator[ role ] : '' }
+                : { role, accelerator: role ? macAccelerator[ role ] : '' };
+        }
+
         if (isWindowsOS) {
             return label ? { role, label, accelerator: role ? windowsAccelerator[ role ] : '' }
                 : { role, accelerator: role ? windowsAccelerator[ role ] : '' };