Fix play on iOS (grumph)
authorChocobozzz <me@florianbigard.com>
Tue, 22 May 2018 07:16:05 +0000 (09:16 +0200)
committerChocobozzz <me@florianbigard.com>
Tue, 22 May 2018 07:17:21 +0000 (09:17 +0200)
client/src/app/videos/+video-watch/video-watch.component.ts
client/src/assets/player/peertube-videojs-plugin.ts
client/src/environments/environment.hmr.ts
client/src/standalone/videos/embed.html

index 127ae919d20cb643c4955aac318b87c5e7fb6044..35a7c04ccad1d910988696f55d597cba382e0446 100644 (file)
@@ -340,6 +340,7 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
     const playerElementWrapper = this.elementRef.nativeElement.querySelector('#video-element-wrapper')
     this.playerElement = document.createElement('video')
     this.playerElement.className = 'video-js vjs-peertube-skin'
+    this.playerElement.setAttribute('playsinline', 'true')
     playerElementWrapper.appendChild(this.playerElement)
 
     const videojsOptions = getVideojsOptions({
index ac04421a7f3e322dd7c2b08d0746f835fa33b8e2..5789641fefff4d576604f3e16bdff86267812752 100644 (file)
@@ -63,7 +63,8 @@ class PeerTubePlugin extends Plugin {
   constructor (player: videojs.Player, options: PeertubePluginOptions) {
     super(player, options)
 
-    this.autoplay = options.autoplay
+    // Disable auto play on iOS
+    this.autoplay = options.autoplay && this.isIOS() === false
 
     this.startTime = options.startTime
     this.videoFiles = options.videoFiles
@@ -180,6 +181,7 @@ class PeerTubePlugin extends Plugin {
         oldTorrent.removePeer(oldTorrent['ws'])
       }
 
+      // Render the video in a few seconds? (on resolution change for example, we wait some seconds of the new video resolution)
       this.addTorrentDelay = setTimeout(() => {
         this.flushVideoFile(previousVideoFile)
 
@@ -337,6 +339,12 @@ class PeerTubePlugin extends Plugin {
         this.tryToPlay()
       })
     } else {
+      // Don't try on iOS that does not support MediaSource
+      if (this.isIOS()) {
+        this.currentVideoFile = this.videoFiles[0]
+        return this.fallbackToHttp(undefined, false)
+      }
+
       // Proxy first play
       const oldPlay = this.player.play.bind(this.player)
       this.player.play = () => {
@@ -439,7 +447,7 @@ class PeerTubePlugin extends Plugin {
     return fetch(this.videoViewUrl, { method: 'POST' })
   }
 
-  private fallbackToHttp (done: Function) {
+  private fallbackToHttp (done?: Function, play = true) {
     this.flushVideoFile(this.currentVideoFile, true)
     this.torrent = null
 
@@ -449,9 +457,9 @@ class PeerTubePlugin extends Plugin {
     const httpUrl = this.currentVideoFile.fileUrl
     this.player.src = this.savePlayerSrcFunction
     this.player.src(httpUrl)
-    this.player.play()
+    if (play) this.tryToPlay()
 
-    return done()
+    if (done) return done()
   }
 
   private handleError (err: Error | string) {
@@ -466,6 +474,10 @@ class PeerTubePlugin extends Plugin {
     this.player.removeClass('vjs-error-display-enabled')
   }
 
+  private isIOS () {
+    return !!navigator.platform && /iPad|iPhone|iPod/.test(navigator.platform)
+  }
+
   private alterInactivity () {
     let saveInactivityTimeout: number
 
index 20e2b8fcd1468d2978bd549e86b28e46e18a040a..78cc8a992587b175a081f4701bcc702d7efb901e 100644 (file)
@@ -1,5 +1,5 @@
 export const environment = {
   production: false,
   hmr: true,
-  apiUrl: 'http://localhost:9000'
+  apiUrl: 'http://192.168.1.42:9000'
 }
index b60b03a2279327d3884450fde9f4c8dc5a350fc3..2d1b48b3881141291fc2c73bcd4e99102ffe334f 100644 (file)
@@ -17,7 +17,7 @@
       <div id="error-content"></div>
     </div>
 
-    <video id="video-container" class="video-js vjs-peertube-skin">
+    <video playsinline="true" id="video-container" class="video-js vjs-peertube-skin">
     </video>
 
   </body>