fallback to old notification method, don't spam if old way of creating

notification fails, try to use favicon

src/services/desktop_notification_utils/desktop_notification_utils.js

@@ -1,8 +1,18 @@
-import { showDesktopNotification as swDesktopNotification } from '../sw/sw.js'
+import { showDesktopNotification as swDesktopNotification, isSWSupported } from '../sw/sw.js'
+const state = { failCreateNotif: false }
 
 export const showDesktopNotification = (rootState, desktopNotificationOpts) => {
   if (!('Notification' in window && window.Notification.permission === 'granted')) return
   if (rootState.statuses.notifications.desktopNotificationSilence) { return }
 
+  if (isSWSupported()) {
     swDesktopNotification(desktopNotificationOpts)
+  } else if (!state.failCreateNotif) {
+    try {
+      const desktopNotification = new window.Notification(desktopNotificationOpts.title, desktopNotificationOpts)
+      setTimeout(desktopNotification.close.bind(desktopNotification), 5000)
+    } catch {
+      state.failCreateNotif = true
+    }
+  }
 }

src/services/notification_utils/notification_utils.js

@@ -76,8 +76,12 @@ export const unseenNotificationsFromStore = store =>
   filter(filteredNotificationsFromStore(store), ({ seen }) => !seen)
 
 export const prepareNotificationObject = (notification, i18n) => {
+  const nodes = document.querySelectorAll('link[rel="icon"]')
+  const icon = nodes[0].href
+
   const notifObj = {
-    tag: notification.id
+    tag: notification.id,
+    icon
   }
   const status = notification.status
   const title = notification.from_profile.name

src/services/sw/sw.js

@@ -10,7 +10,7 @@ function urlBase64ToUint8Array (base64String) {
   return Uint8Array.from([...rawData].map((char) => char.charCodeAt(0)))
 }
 
-function isSWSupported () {
+export function isSWSupported () {
   return 'serviceWorker' in navigator
 }
 

src/sw.js

@@ -59,16 +59,19 @@ self.addEventListener('message', async (event) => {
   console.log(event)
 
   if (type === 'desktopNotification') {
-    const { title, body, icon, id } = content
+    const { title, ...rest } = content
-    if (state.notificationIds.has(id)) return
+    const { tag } = rest
-    state.notificationIds.add(id)
+    if (state.notificationIds.has(tag)) return
-    setTimeout(() => state.notificationIds.delete(id), 10000)
+    state.notificationIds.add(tag)
-    self.registration.showNotification('SWTEST:  ' + title, { body, icon })
+    setTimeout(() => state.notificationIds.delete(tag), 10000)
+    self.registration.showNotification(title, rest)
   }
 
   if (type === 'updateFocus') {
     state.lastFocused = event.source.id
-    console.log(state)
+
+    const notifications = await self.registration.getNotifications()
+    notifications.forEach(n => n.close())
   }
 })