Commit 3c62f7c6 authored by Dave Barker's avatar Dave Barker 🐈

Refs #82 - Popup notification links don't dismiss the notification

parent 004f6c9e
Pipeline #83503806 passed with stage
in 1 minute and 2 seconds
......@@ -426,12 +426,18 @@ exports.isOptional = notificationType =>
port.on("notifications.clicked", (message, sender) =>
{
if (message.link)
{
openNotificationLink(message.link);
}
// While clicking on a desktop notification's button dismisses the
// notification, clicking on a popup window notification's link does not.
else
{
if (browserNotificationsSupported)
browser.notifications.clear(message.id);
if (browserNotificationsSupported)
browser.notifications.clear(message.id);
notificationDismissed(message.id, true);
notificationDismissed(message.id, true);
}
});
ext.pages.onLoading.addListener(page =>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment