1 files changed, 18 insertions, 41 deletions

diff --git a/src/webview/notifications.ts b/src/webview/notifications.ts
index e4401ab6e..835869f5c 100644
--- a/src/webview/notifications.ts
+++ b/src/webview/notifications.ts
@@ -31,52 +31,29 @@ export class NotificationsHandler {
 }
 
 export const notificationsClassDefinition = `(() => {
-  class Notification {
-    static permission = 'granted';
-
-    constructor(title = '', options = {}) {
-      this.title = title;
-      this.options = options;
-      try {
-        window.ferdium.displayNotification(title, options)
-          .then(() => {
-            if (typeof (this.onClick) === 'function') {
-              this.onClick();
-            }
-          });
-      } catch(error) {
-                this.options.onClick = null;
-          window.ferdium.displayNotification(title, options)
-            .then(() => {
-              if (typeof (this.onClick) === 'function') {
-                this.onClick();
-              }
-            });
-      }
-    }
-
-    static requestPermission(cb = null) {
-      if (!cb) {
-        return new Promise((resolve) => {
-          resolve(Notification.permission);
-        });
-      }
-
-      if (typeof (cb) === 'function') {
-        return cb(Notification.permission);
-      }
-
-      return Notification.permission;
-    }
+class Notification {
+  static permission = 'granted';
+
+  constructor(title = '', options = {}) {
+    window.ferdium.displayNotification(title, options).then(() => {
+      // TODO: After several tries, we couldn't find a way to trigger the native notification onclick event.
+      // This was needed so that user could go to the specific context when clicking on the notification (it only goes to the service now).
+      // For now, we don't do anything here
+    });
+  }
 
-    onNotify(data) {
-      return data;
+  static requestPermission(cb) {
+    if (typeof cb === 'function') {
+      cb(Notification.permission);
     }
 
-    onClick() {}
+    return Promise.resolve(Notification.permission);
+  }
 
-    close() {}
+  onNotify(data) {
+    return data;
   }
+}
 
   window.Notification = Notification;
 })();`;