[FEATURE] Add browser detection to disallow splash screen use for Saf…

…ari and Firefox (not working currently) #354
vicwomg · Aug 28, 2024 · 865ed56 · 865ed56
1 parent b580fee
commit 865ed56
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/pikaraoke/templates/splash.html b/pikaraoke/templates/splash.html
@@ -13,6 +13,9 @@
       video.readyState > 2
     );
 
+  const isFirefox = typeof InstallTrigger !== "undefined";
+  const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+
   var mouseTimer = null,
     cursorVisible = true;
 
@@ -270,6 +273,12 @@
     let params = new URLSearchParams(window.location.search);
     let dismissConfirmation = params.get("confirm");
 
+    if (isFirefox || isSafari) {
+      confirm(
+        "This browser is not supported. You may experience streaming issues. Please use Chrome for best results."
+      );
+    }
+
     if (dismissConfirmation === "false") {
       let modal = document.getElementById("permissions-modal");
       modal.classList.remove("is-active");
@@ -303,7 +312,9 @@
 
 <div id="ap-container">
   <div class="is-size-5 stroke">
-    <div id="hostap">{% for line in hostap_info %}{{ line }}<br />{% endfor %}</div>
+    <div id="hostap">
+      {% for line in hostap_info %}{{ line }}<br />{% endfor %}
+    </div>
   </div>
 </div>
 

diff --git a/pikaraoke/translations/es_VE/messages.po b/pikaraoke/translations/es_VE/messages.po
@@ -559,4 +559,4 @@ msgstr "A continuación:"
 #. to the queue.)
 #: templates/splash.html:10
 msgid "Next singer:"
-msgstr "Próximo cantante:"
+msgstr "Próximo cantante:"