From b8ca1f920e0b19ade0f9b71e4041f640b11e6f14 Mon Sep 17 00:00:00 2001 From: Matt N Date: Tue, 24 Mar 2020 17:27:14 -0700 Subject: [PATCH] Don't immediately remove notifications from notification trays Let the notifications go into browser/OS notification trays so users can click on them from there if they miss the initial notification. Modern Chrome uses OS notifications so the user is in control of the the notification with the OS. This also aligns with the Electron platform version. Signed-off-by: Matthew Noorenberghe --- src/vector/platform/WebPlatform.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/vector/platform/WebPlatform.js b/src/vector/platform/WebPlatform.js index 4ad96decb8..8db12856f3 100644 --- a/src/vector/platform/WebPlatform.js +++ b/src/vector/platform/WebPlatform.js @@ -91,12 +91,6 @@ export default class WebPlatform extends VectorBasePlatform { global.focus(); notification.close(); }; - - // Chrome only dismisses notifications after 20s, which - // is waaaaay too long - global.setTimeout(function() { - notification.close(); - }, 5 * 1000); } _getVersion(): Promise {