Revert "SDA-4604 Limiting window activation change to macOS for bring…

…-to-front feature only (finos#2176) (finos#2177)" This reverts commit e543476.
sbenmoussati · Jul 19, 2024 · c75a130 · c75a130
1 parent e543476
commit c75a130
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/spec/mainApiHandler.spec.ts b/spec/mainApiHandler.spec.ts
@@ -340,7 +340,7 @@ describe('main api handler', () => {
         reason: 'notification',
         windowName: 'notification',
       };
-      const expectedValue = ['notification', false];
+      const expectedValue = ['notification'];
       ipcMain.send(apiName.symphonyApi, value);
       expect(spy).toBeCalledWith(...expectedValue);
     });

diff --git a/src/app/main-api-handler.ts b/src/app/main-api-handler.ts
@@ -207,7 +207,7 @@ ipcMain.on(
         if (typeof arg.reason === 'string' && arg.reason === 'notification') {
           const { bringToFront } = config.getConfigFields(['bringToFront']);
           if (bringToFront === CloudConfigDataTypes.ENABLED) {
-            activate(arg.windowName, false);
+            activate(arg.windowName);
           }
         }
         break;

diff --git a/src/app/window-actions.ts b/src/app/window-actions.ts
@@ -187,7 +187,9 @@ export const activate = (
         // Bring the window to the top without focusing
         // Flash task bar icon in Windows for windows
         if (!shouldFocus) {
-          return isMac || isLinux ? window.show() : window.flashFrame(true);
+          return isMac || isLinux
+            ? window.showInactive()
+            : window.flashFrame(true);
         }
 
         // Note: On window just focusing will preserve window snapped state