Don't play video if user paused it during loading

[github/Chocobozzz/PeerTube.git] / client / src / assets / player / peertube-videojs-plugin.ts

diff --git a/client/src/assets/player/peertube-videojs-plugin.ts b/client/src/assets/player/peertube-videojs-plugin.ts
index 0a7651a81f3664152921312176bfd45ad2b6fd43..5ccfdce36b8ad127dac80119206a6bff6bcee690 100644 (file)
--- a/client/src/assets/player/peertube-videojs-plugin.ts
+++ b/client/src/assets/player/peertube-videojs-plugin.ts
@@ -56,7 +56,8 @@ class ResolutionMenuItem extends MenuItem {
   }
 
   handleClick (event) {
-    MenuItem.prototype.handleClick.call(this, event)
+    super.handleClick(event)
+
     this.player_.peertube().updateResolution(this.options_.id)
   }
 }
@@ -279,14 +280,14 @@ class PeerTubePlugin extends Plugin {
         if (err) return this.handleError(err)
 
         this.renderer = renderer
-        this.player.play().then(done)
+        if (!this.player.paused()) this.player.play().then(done)
       })
     })
 
     this.torrent.on('error', err => this.handleError(err))
     this.torrent.on('warning', (err: any) => {
       // We don't support HTTP tracker but we don't care -> we use the web socket tracker
-      if (err.message.indexOf('Unsupported tracker protocol: http') !== -1) return
+      if (err.message.indexOf('Unsupported tracker protocol') !== -1) return
       // Users don't care about issues with WebRTC, but developers do so log it in the console
       if (err.message.indexOf('Ice connection failed') !== -1) {
         console.error(err)
@@ -304,6 +305,9 @@ class PeerTubePlugin extends Plugin {
     const currentTime = this.player.currentTime()
     const isPaused = this.player.paused()
 
+    // Remove poster to have black background
+    this.playerElement.poster = ''
+
     // Hide bigPlayButton
     if (!isPaused) {
       this.player.bigPlayButton.hide()