From 6422847473a7dd1aa6066d2735a66a47a956f5e4 Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Mon, 11 May 2020 17:20:23 +0200
Subject: Fix autoplay on Safari

---
 client/src/assets/player/peertube-player-manager.ts | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/client/src/assets/player/peertube-player-manager.ts b/client/src/assets/player/peertube-player-manager.ts
index 12e460f03..61da6a4cb 100644
--- a/client/src/assets/player/peertube-player-manager.ts
+++ b/client/src/assets/player/peertube-player-manager.ts
@@ -233,7 +233,7 @@ export class PeertubePlayerManager {
         : undefined, // Undefined so the player knows it has to check the local storage
 
       autoplay: autoplay === true
-        ? 'play' // Use 'any' instead of true to get notifier by videojs if autoplay fails
+        ? this.getAutoPlayValue()
         : autoplay,
 
       poster: commonOptions.poster,
@@ -509,6 +509,16 @@ export class PeertubePlayerManager {
       }
     })
   }
+
+  private static getAutoPlayValue () {
+    const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent)
+
+    // We have issues with autoplay and Safari.
+    // any that tries to play using auto mute seems to work
+    if (isSafari) return 'any'
+
+    return 'play'
+  }
 }
 
 // ############################################################################
-- 
cgit v1.2.3