don't use Notification API on mobile Chrome

linnaea · Oct 3, 2024 · 960d5d9 · 960d5d9
1 parent 543ed4f
commit 960d5d9
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/index.html b/index.html
@@ -418,7 +418,15 @@
                         encodeVideo();
                         document.getElementById('seekControl').style.display = 'none';
                         document.getElementById('stepControl').style.display = 'none';
-                        window.Notification?.requestPermission();
+                        try {
+                            if (window.Notification && window.Notification?.permission !== 'granted') {
+                                new Notification('');
+                                Notification.requestPermission();
+                            }
+                        } catch(e) {
+                            if (!(e instanceof TypeError))
+                                Notification.requestPermission();
+                        }
                         return;
                     case ALL_STEPS.RING:
                         let frameWidth = previewFrame.width - config.cropL - config.cropR;