Skip to content

Commit

Permalink
Don't immediately remove notifications from notification trays (#4773)
Browse files Browse the repository at this point in the history
Let the notifications go into browser/OS notification trays so users can click on them from there if they miss the initial notification. Modern browsers generally use OS notifications so the user is in control of the notification at the OS level.
  • Loading branch information
mnoorenberghe authored Apr 14, 2020
1 parent de40f1a commit e99c37a
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions client/app/services/notifications.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,6 @@ function showNotification(title, content) {
body: content,
icon: redashIconUrl,
});
setTimeout(() => {
notification.close();
}, 3000);
notification.onclick = function onClick() {
window.focus();
this.close();
Expand Down

0 comments on commit e99c37a

Please sign in to comment.