aboutsummaryrefslogtreecommitdiffhomepage
path: root/client/src/app/shared/video-playlist
diff options
context:
space:
mode:
authorRigel Kent <sendmemail@rigelk.eu>2019-12-26 11:52:46 +0100
committerRigel Kent <sendmemail@rigelk.eu>2019-12-26 11:52:52 +0100
commitc06af5012ecc925ca924e6e20db3a1d909b1148e (patch)
tree88a3595bfd0a68e8f5314737a22f9516d9790c9b /client/src/app/shared/video-playlist
parentdef2a70b7e5ee807d7b532df8c9d33d17d24ccbb (diff)
downloadPeerTube-c06af5012ecc925ca924e6e20db3a1d909b1148e.tar.gz
PeerTube-c06af5012ecc925ca924e6e20db3a1d909b1148e.tar.zst
PeerTube-c06af5012ecc925ca924e6e20db3a1d909b1148e.zip
Add playlist search option and search input for add-to-video-playlist dropdown
fixes #2138
Diffstat (limited to 'client/src/app/shared/video-playlist')
-rw-r--r--client/src/app/shared/video-playlist/video-add-to-playlist.component.html4
-rw-r--r--client/src/app/shared/video-playlist/video-add-to-playlist.component.scss10
-rw-r--r--client/src/app/shared/video-playlist/video-add-to-playlist.component.ts20
-rw-r--r--client/src/app/shared/video-playlist/video-playlist.service.ts10
4 files changed, 39 insertions, 5 deletions
diff --git a/client/src/app/shared/video-playlist/video-add-to-playlist.component.html b/client/src/app/shared/video-playlist/video-add-to-playlist.component.html
index 0cc8af345..ba30953b9 100644
--- a/client/src/app/shared/video-playlist/video-add-to-playlist.component.html
+++ b/client/src/app/shared/video-playlist/video-add-to-playlist.component.html
@@ -41,6 +41,10 @@
41 </div> 41 </div>
42 </div> 42 </div>
43 43
44 <div class="input-container">
45 <input type="text" placeholder="Search playlists" [(ngModel)]="videoPlaylistSearch" (ngModelChange)="onVideoPlaylistSearchChanged()" />
46 </div>
47
44 <div class="playlists"> 48 <div class="playlists">
45 <div class="playlist dropdown-item" *ngFor="let playlist of videoPlaylists" (click)="togglePlaylist($event, playlist)"> 49 <div class="playlist dropdown-item" *ngFor="let playlist of videoPlaylists" (click)="togglePlaylist($event, playlist)">
46 <my-peertube-checkbox [inputName]="'in-playlist-' + playlist.id" [(ngModel)]="playlist.inPlaylist" [onPushWorkaround]="true"></my-peertube-checkbox> 50 <my-peertube-checkbox [inputName]="'in-playlist-' + playlist.id" [(ngModel)]="playlist.inPlaylist" [onPushWorkaround]="true"></my-peertube-checkbox>
diff --git a/client/src/app/shared/video-playlist/video-add-to-playlist.component.scss b/client/src/app/shared/video-playlist/video-add-to-playlist.component.scss
index 090b530cf..5f9bb51a7 100644
--- a/client/src/app/shared/video-playlist/video-add-to-playlist.component.scss
+++ b/client/src/app/shared/video-playlist/video-add-to-playlist.component.scss
@@ -53,6 +53,15 @@
53 overflow-y: auto; 53 overflow-y: auto;
54} 54}
55 55
56.input-container {
57 display: flex;
58
59 input {
60 flex-grow: 1;
61 margin: 0 15px 10px 15px;
62 }
63}
64
56.playlist { 65.playlist {
57 display: flex; 66 display: flex;
58 cursor: pointer; 67 cursor: pointer;
@@ -76,7 +85,6 @@
76.new-playlist-button, 85.new-playlist-button,
77.new-playlist-block { 86.new-playlist-block {
78 padding-top: 10px; 87 padding-top: 10px;
79 margin-top: 10px;
80 border-top: 1px solid $separator-border-color; 88 border-top: 1px solid $separator-border-color;
81} 89}
82 90
diff --git a/client/src/app/shared/video-playlist/video-add-to-playlist.component.ts b/client/src/app/shared/video-playlist/video-add-to-playlist.component.ts
index 6380c2e51..25ba8cbca 100644
--- a/client/src/app/shared/video-playlist/video-add-to-playlist.component.ts
+++ b/client/src/app/shared/video-playlist/video-add-to-playlist.component.ts
@@ -1,7 +1,8 @@
1import { ChangeDetectionStrategy, ChangeDetectorRef, Component, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core' 1import { ChangeDetectionStrategy, ChangeDetectorRef, Component, Input, OnChanges, OnInit, SimpleChanges } from '@angular/core'
2import { VideoPlaylistService } from '@app/shared/video-playlist/video-playlist.service' 2import { VideoPlaylistService } from '@app/shared/video-playlist/video-playlist.service'
3import { AuthService, Notifier } from '@app/core' 3import { AuthService, Notifier } from '@app/core'
4import { forkJoin } from 'rxjs' 4import { forkJoin, Subject } from 'rxjs'
5import { debounceTime } from 'rxjs/operators'
5import { Video, VideoPlaylistCreate, VideoPlaylistElementCreate, VideoPlaylistPrivacy } from '@shared/models' 6import { Video, VideoPlaylistCreate, VideoPlaylistElementCreate, VideoPlaylistPrivacy } from '@shared/models'
6import { FormReactive, FormValidatorService, VideoPlaylistValidatorsService } from '@app/shared/forms' 7import { FormReactive, FormValidatorService, VideoPlaylistValidatorsService } from '@app/shared/forms'
7import { I18n } from '@ngx-translate/i18n-polyfill' 8import { I18n } from '@ngx-translate/i18n-polyfill'
@@ -29,6 +30,8 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit,
29 @Input() lazyLoad = false 30 @Input() lazyLoad = false
30 31
31 isNewPlaylistBlockOpened = false 32 isNewPlaylistBlockOpened = false
33 videoPlaylistSearch: string
34 videoPlaylistSearchChanged = new Subject<string>()
32 videoPlaylists: PlaylistSummary[] = [] 35 videoPlaylists: PlaylistSummary[] = []
33 timestampOptions: { 36 timestampOptions: {
34 startTimestampEnabled: boolean 37 startTimestampEnabled: boolean
@@ -58,6 +61,13 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit,
58 this.buildForm({ 61 this.buildForm({
59 displayName: this.videoPlaylistValidatorsService.VIDEO_PLAYLIST_DISPLAY_NAME 62 displayName: this.videoPlaylistValidatorsService.VIDEO_PLAYLIST_DISPLAY_NAME
60 }) 63 })
64
65 this.videoPlaylistSearchChanged
66 .pipe(
67 debounceTime(500))
68 .subscribe(() => {
69 this.load()
70 })
61 } 71 }
62 72
63 ngOnChanges (simpleChanges: SimpleChanges) { 73 ngOnChanges (simpleChanges: SimpleChanges) {
@@ -74,6 +84,7 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit,
74 84
75 reload () { 85 reload () {
76 this.videoPlaylists = [] 86 this.videoPlaylists = []
87 this.videoPlaylistSearch = undefined
77 88
78 this.init() 89 this.init()
79 90
@@ -82,11 +93,12 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit,
82 93
83 load () { 94 load () {
84 forkJoin([ 95 forkJoin([
85 this.videoPlaylistService.listAccountPlaylists(this.user.account, undefined,'-updatedAt'), 96 this.videoPlaylistService.listAccountPlaylists(this.user.account, undefined, '-updatedAt', this.videoPlaylistSearch),
86 this.videoPlaylistService.doesVideoExistInPlaylist(this.video.id) 97 this.videoPlaylistService.doesVideoExistInPlaylist(this.video.id)
87 ]) 98 ])
88 .subscribe( 99 .subscribe(
89 ([ playlistsResult, existResult ]) => { 100 ([ playlistsResult, existResult ]) => {
101 this.videoPlaylists = []
90 for (const playlist of playlistsResult.data) { 102 for (const playlist of playlistsResult.data) {
91 const existingPlaylist = existResult[ this.video.id ].find(p => p.playlistId === playlist.id) 103 const existingPlaylist = existResult[ this.video.id ].find(p => p.playlistId === playlist.id)
92 104
@@ -178,6 +190,10 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit,
178 return `(${start}-${stop})` 190 return `(${start}-${stop})`
179 } 191 }
180 192
193 onVideoPlaylistSearchChanged () {
194 this.videoPlaylistSearchChanged.next()
195 }
196
181 private removeVideoFromPlaylist (playlist: PlaylistSummary) { 197 private removeVideoFromPlaylist (playlist: PlaylistSummary) {
182 if (!playlist.playlistElementId) return 198 if (!playlist.playlistElementId) return
183 199
diff --git a/client/src/app/shared/video-playlist/video-playlist.service.ts b/client/src/app/shared/video-playlist/video-playlist.service.ts
index 2945b4959..5f74dcd4c 100644
--- a/client/src/app/shared/video-playlist/video-playlist.service.ts
+++ b/client/src/app/shared/video-playlist/video-playlist.service.ts
@@ -59,7 +59,12 @@ export class VideoPlaylistService {
59 ) 59 )
60 } 60 }
61 61
62 listAccountPlaylists (account: Account, componentPagination: ComponentPagination, sort: string): Observable<ResultList<VideoPlaylist>> { 62 listAccountPlaylists (
63 account: Account,
64 componentPagination: ComponentPagination,
65 sort: string,
66 search?: string
67 ): Observable<ResultList<VideoPlaylist>> {
63 const url = AccountService.BASE_ACCOUNT_URL + account.nameWithHost + '/video-playlists' 68 const url = AccountService.BASE_ACCOUNT_URL + account.nameWithHost + '/video-playlists'
64 const pagination = componentPagination 69 const pagination = componentPagination
65 ? this.restService.componentPaginationToRestPagination(componentPagination) 70 ? this.restService.componentPaginationToRestPagination(componentPagination)
@@ -67,6 +72,7 @@ export class VideoPlaylistService {
67 72
68 let params = new HttpParams() 73 let params = new HttpParams()
69 params = this.restService.addRestGetParams(params, pagination, sort) 74 params = this.restService.addRestGetParams(params, pagination, sort)
75 if (search) params = this.restService.addObjectParams(params, { search })
70 76
71 return this.authHttp.get<ResultList<VideoPlaylist>>(url, { params }) 77 return this.authHttp.get<ResultList<VideoPlaylist>>(url, { params })
72 .pipe( 78 .pipe(
@@ -213,8 +219,8 @@ export class VideoPlaylistService {
213 219
214 private doVideosExistInPlaylist (videoIds: number[]): Observable<VideoExistInPlaylist> { 220 private doVideosExistInPlaylist (videoIds: number[]): Observable<VideoExistInPlaylist> {
215 const url = VideoPlaylistService.MY_VIDEO_PLAYLIST_URL + 'videos-exist' 221 const url = VideoPlaylistService.MY_VIDEO_PLAYLIST_URL + 'videos-exist'
216 let params = new HttpParams()
217 222
223 let params = new HttpParams()
218 params = this.restService.addObjectParams(params, { videoIds }) 224 params = this.restService.addObjectParams(params, { videoIds })
219 225
220 return this.authHttp.get<VideoExistInPlaylist>(url, { params }) 226 return this.authHttp.get<VideoExistInPlaylist>(url, { params })