remove "window.Notification" for "WebClient" project only, closes #328

* so the following projects should not be affected: "proton-mail-settings", "proton-contacts", "proton-calendar"
vladimiry · Sep 19, 2020 · aee67aa · aee67aa
1 parent c924300
commit aee67aa
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/electron-preload/webview/primary/configure-provider-app.ts b/src/electron-preload/webview/primary/configure-provider-app.ts
@@ -84,19 +84,21 @@ export function configureProviderApp(
 
     initSpellCheckProvider(logger);
     applyZoomFactor(logger);
-    disableBrowserNotificationFeature(logger);
     registerDocumentKeyDownEventListener(document, logger);
     registerDocumentClickEventListener(document, logger);
 
     const packagedWebClientApp = resolvePackagedWebClientApp(packagedWebClientUrl);
-    const isAngularWebClient = packagedWebClientApp.project === "WebClient";
-    logger.verbose(JSON.stringify({packagedWebClientApp, isAngularWebClient}));
+    const isWebClient = packagedWebClientApp.project === "WebClient";
+    logger.verbose(JSON.stringify({packagedWebClientApp, isWebClient}));
 
-    if (!isAngularWebClient) {
+    if (!isWebClient) {
         logger.info(`Skip configuring AngularWebClient-specific stuff`);
         return;
     }
 
+    // should be called for "WebClient" project only
+    disableBrowserNotificationFeature(logger);
+
     logger.info(`Start configuring AngularWebClient-specific stuff`);
 
     configureAngularApp();