Fix notification bug #67

koel · Dec 20, 2015 · 6457528 · 6457528
1 parent e536ff6
commit 6457528
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/resources/assets/js/services/playback.js b/resources/assets/js/services/playback.js
@@ -94,14 +94,20 @@ export default {
             return;
         }
 
-        var notification = new Notification(`♫ ${song.title}`, {
-            icon: song.album.cover,
-            body: `${song.album.name} – ${song.album.artist.name}`
-        });
-
-        window.setTimeout(() => {
-          notification.close();
-        }, 5000);
+        try {
+            var notification = new Notification(`♫ ${song.title}`, {
+                icon: song.album.cover,
+                body: `${song.album.name} – ${song.album.artist.name}`
+            });
+
+            notification.onclick = () => window.focus();
+
+            // Close the notif after 5 secs.
+            window.setTimeout(() => notification.close(), 5000);
+        } catch (e) {
+            // Notification fails. 
+            // @link https://developer.mozilla.org/en-US/docs/Web/API/ServiceWorkerRegistration/showNotification
+        }
     },
 
     /**