From 531ab5b6279707b2d0d01d3153e62c4bdfca8480 Mon Sep 17 00:00:00 2001 From: Chocobozzz Date: Tue, 23 Jan 2018 15:01:54 +0100 Subject: Don't play video if user paused it during loading --- client/src/assets/player/peertube-videojs-plugin.ts | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/client/src/assets/player/peertube-videojs-plugin.ts b/client/src/assets/player/peertube-videojs-plugin.ts index 0a7651a81..5ccfdce36 100644 --- 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() -- cgit v1.2.3