aboutsummaryrefslogtreecommitdiffhomepage
path: root/client/src/assets
diff options
context:
space:
mode:
authorChocobozzz <me@florianbigard.com>2021-04-21 09:46:55 +0200
committerChocobozzz <me@florianbigard.com>2021-04-21 15:18:22 +0200
commite8bb5b6b3a1e4c2aeab368f01cc5092d8478b893 (patch)
tree5cb81a909d6789f8803d35e902f6fc7fa822284a /client/src/assets
parentb7a27f2860982db2322048441d85b7143862d3b7 (diff)
downloadPeerTube-e8bb5b6b3a1e4c2aeab368f01cc5092d8478b893.tar.gz
PeerTube-e8bb5b6b3a1e4c2aeab368f01cc5092d8478b893.tar.zst
PeerTube-e8bb5b6b3a1e4c2aeab368f01cc5092d8478b893.zip
Fix context menu when watching a playlist
Diffstat (limited to 'client/src/assets')
-rw-r--r--client/src/assets/player/peertube-player-manager.ts110
-rw-r--r--client/src/assets/player/peertube-videojs-typings.ts9
-rw-r--r--client/src/assets/player/utils.ts6
3 files changed, 89 insertions, 36 deletions
diff --git a/client/src/assets/player/peertube-player-manager.ts b/client/src/assets/player/peertube-player-manager.ts
index ed82e0496..689e70fb5 100644
--- a/client/src/assets/player/peertube-player-manager.ts
+++ b/client/src/assets/player/peertube-player-manager.ts
@@ -35,7 +35,7 @@ import {
35 VideoJSPluginOptions 35 VideoJSPluginOptions
36} from './peertube-videojs-typings' 36} from './peertube-videojs-typings'
37import { TranslationsManager } from './translations-manager' 37import { TranslationsManager } from './translations-manager'
38import { buildVideoOrPlaylistEmbed, buildVideoLink, getRtcConfig, isSafari, isIOS } from './utils' 38import { buildVideoOrPlaylistEmbed, buildVideoLink, getRtcConfig, isSafari, isIOS, buildPlaylistLink } from './utils'
39import { copyToClipboard } from '../../root-helpers/utils' 39import { copyToClipboard } from '../../root-helpers/utils'
40 40
41// Change 'Playback Rate' to 'Speed' (smaller for our settings menu) 41// Change 'Playback Rate' to 'Speed' (smaller for our settings menu)
@@ -87,6 +87,8 @@ export interface CommonOptions extends CustomizationOptions {
87 hasPreviousVideo?: () => boolean 87 hasPreviousVideo?: () => boolean
88 88
89 playlist?: PlaylistPluginOptions 89 playlist?: PlaylistPluginOptions
90 playlistEmbedUrl?: string
91 playlistEmbedTitle?: string
90 92
91 videoDuration: number 93 videoDuration: number
92 enableHotkeys: boolean 94 enableHotkeys: boolean
@@ -167,7 +169,13 @@ export class PeertubePlayerManager {
167 PeertubePlayerManager.alreadyPlayed = true 169 PeertubePlayerManager.alreadyPlayed = true
168 }) 170 })
169 171
170 self.addContextMenu(mode, player, options.common.embedUrl, options.common.embedTitle) 172 self.addContextMenu({
173 mode,
174 player,
175 videoEmbedUrl: options.common.embedUrl,
176 videoEmbedTitle: options.common.embedTitle,
177 playlist: options.common.playlist
178 })
171 179
172 player.bezels() 180 player.bezels()
173 181
@@ -205,7 +213,13 @@ export class PeertubePlayerManager {
205 videojs(newVideoElement, videojsOptions, function (this: videojs.Player) { 213 videojs(newVideoElement, videojsOptions, function (this: videojs.Player) {
206 const player = this 214 const player = this
207 215
208 self.addContextMenu(mode, player, options.common.embedUrl, options.common.embedTitle) 216 self.addContextMenu({
217 mode,
218 player,
219 videoEmbedUrl: options.common.embedUrl,
220 videoEmbedTitle: options.common.embedTitle,
221 playlist: options.common.playlist
222 })
209 223
210 PeertubePlayerManager.onPlayerChange(player) 224 PeertubePlayerManager.onPlayerChange(player)
211 }) 225 })
@@ -239,7 +253,7 @@ export class PeertubePlayerManager {
239 } 253 }
240 } 254 }
241 255
242 if (commonOptions.playlist) { 256 if (commonOptions.playlist?.createComponent === true) {
243 plugins.playlist = commonOptions.playlist 257 plugins.playlist = commonOptions.playlist
244 } 258 }
245 259
@@ -497,37 +511,71 @@ export class PeertubePlayerManager {
497 return children 511 return children
498 } 512 }
499 513
500 private static addContextMenu (mode: PlayerMode, player: videojs.Player, videoEmbedUrl: string, videoEmbedTitle: string) { 514 private static addContextMenu (options: {
515 mode: PlayerMode
516 player: videojs.Player
517 videoEmbedUrl: string
518 videoEmbedTitle: string
519 playlist?: PlaylistPluginOptions
520 }) {
521 const { mode, player, videoEmbedUrl, videoEmbedTitle, playlist } = options
522
501 const content = () => { 523 const content = () => {
502 const isLoopEnabled = player.options_['loop'] 524 let items: { icon?: string, label: string, listener: Function }[] = []
503 const items = [ 525
504 { 526 if (!playlist) {
505 icon: 'repeat', 527 const isLoopEnabled = player.options_['loop']
506 label: player.localize('Play in loop') + (isLoopEnabled ? '<span class="vjs-icon-tick-white"></span>' : ''), 528 items = items.concat([
507 listener: function () { 529 {
508 player.options_['loop'] = !isLoopEnabled 530 icon: 'repeat',
509 } 531 label: player.localize('Play in loop') + (isLoopEnabled ? '<span class="vjs-icon-tick-white"></span>' : ''),
510 }, 532 listener: function () {
511 { 533 player.options_['loop'] = !isLoopEnabled
512 label: player.localize('Copy the video URL'), 534 }
513 listener: function () { 535 },
514 copyToClipboard(buildVideoLink()) 536 {
515 } 537 label: player.localize('Copy the video URL'),
516 }, 538 listener: function () {
517 { 539 copyToClipboard(buildVideoLink())
518 label: player.localize('Copy the video URL at the current time'), 540 }
519 listener: function (this: videojs.Player) { 541 },
520 copyToClipboard(buildVideoLink({ startTime: this.currentTime() })) 542 {
543 label: player.localize('Copy the video URL at the current time'),
544 listener: function (this: videojs.Player) {
545 copyToClipboard(buildVideoLink({ startTime: this.currentTime() }))
546 }
521 } 547 }
522 }, 548 ])
523 { 549 } else {
524 icon: 'code', 550 items = items.concat([
525 label: player.localize('Copy embed code'), 551 {
526 listener: () => { 552 label: player.localize('Copy the playlist URL'),
527 copyToClipboard(buildVideoOrPlaylistEmbed(videoEmbedUrl, videoEmbedTitle)) 553 listener: function () {
554 copyToClipboard(buildPlaylistLink())
555 }
556 },
557 {
558 label: player.localize('Copy the playlist URL at current video position'),
559 listener: function (this: videojs.Player) {
560 copyToClipboard(buildPlaylistLink({ playlistPosition: playlist.getCurrentPosition() }))
561 }
562 },
563 {
564 label: player.localize('Copy the playlist embed code'),
565 listener: function (this: videojs.Player) {
566 copyToClipboard(buildVideoOrPlaylistEmbed(playlist.embedUrl, playlist.embedTitle))
567 }
528 } 568 }
569 ])
570 }
571
572 items = items.concat({
573 icon: 'code',
574 label: player.localize('Copy video embed code'),
575 listener: () => {
576 copyToClipboard(buildVideoOrPlaylistEmbed(videoEmbedUrl, videoEmbedTitle))
529 } 577 }
530 ] 578 })
531 579
532 if (mode === 'webtorrent') { 580 if (mode === 'webtorrent') {
533 items.push({ 581 items.push({
diff --git a/client/src/assets/player/peertube-videojs-typings.ts b/client/src/assets/player/peertube-videojs-typings.ts
index 4a6c80247..a48ff2cd0 100644
--- a/client/src/assets/player/peertube-videojs-typings.ts
+++ b/client/src/assets/player/peertube-videojs-typings.ts
@@ -113,13 +113,18 @@ type PeerTubePluginOptions = {
113} 113}
114 114
115type PlaylistPluginOptions = { 115type PlaylistPluginOptions = {
116 elements: VideoPlaylistElement[] 116 createComponent: boolean
117
118 elements?: VideoPlaylistElement[]
117 119
118 playlist: VideoPlaylist 120 playlist: VideoPlaylist
119 121
120 getCurrentPosition: () => number 122 getCurrentPosition: () => number
121 123
122 onItemClicked: (element: VideoPlaylistElement) => void 124 embedUrl: string
125 embedTitle: string
126
127 onItemClicked?: (element: VideoPlaylistElement) => void
123} 128}
124 129
125type NextPreviousVideoButtonOptions = { 130type NextPreviousVideoButtonOptions = {
diff --git a/client/src/assets/player/utils.ts b/client/src/assets/player/utils.ts
index d7451fa1d..136b69b4f 100644
--- a/client/src/assets/player/utils.ts
+++ b/client/src/assets/player/utils.ts
@@ -94,9 +94,8 @@ function buildVideoLink (options: {
94 94
95function buildPlaylistLink (options: { 95function buildPlaylistLink (options: {
96 baseUrl?: string 96 baseUrl?: string
97 97 playlistPosition?: number
98 playlistPosition: number 98} = {}) {
99}) {
100 const { baseUrl } = options 99 const { baseUrl } = options
101 100
102 const url = baseUrl 101 const url = baseUrl
@@ -106,6 +105,7 @@ function buildPlaylistLink (options: {
106 const params = generateParams(window.location.search) 105 const params = generateParams(window.location.search)
107 106
108 if (options.playlistPosition) params.set('playlistPosition', '' + options.playlistPosition) 107 if (options.playlistPosition) params.set('playlistPosition', '' + options.playlistPosition)
108 else params.delete('playlistPosition')
109 109
110 return buildUrl(url, params) 110 return buildUrl(url, params)
111} 111}