Add video miniature dropdown

3 files changed, 67 insertions, 55 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 19b326206..6029b3648 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
@@ -1,74 +1,76 @@
-<div class="header">
-  <div class="first-row">
-    <div i18n class="title">Save to</div>
+<div class="root">
+  <div class="header">
+    <div class="first-row">
+      <div i18n class="title">Save to</div>
 
-    <div class="options" (click)="displayOptions = !displayOptions">
-      <my-global-icon iconName="cog"></my-global-icon>
+      <div class="options" (click)="displayOptions = !displayOptions">
+        <my-global-icon iconName="cog"></my-global-icon>
 
-      <span i18n>Options</span>
+        <span i18n>Options</span>
+      </div>
     </div>
-  </div>
 
-  <div class="options-row" *ngIf="displayOptions">
-    <div>
-      <my-peertube-checkbox
-        inputName="startAt" [(ngModel)]="timestampOptions.startTimestampEnabled"
-        i18n-labelText labelText="Start at"
-      ></my-peertube-checkbox>
+    <div class="options-row" *ngIf="displayOptions">
+      <div>
+        <my-peertube-checkbox
+          inputName="startAt" [(ngModel)]="timestampOptions.startTimestampEnabled"
+          i18n-labelText labelText="Start at"
+        ></my-peertube-checkbox>
 
-      <my-timestamp-input
-        [timestamp]="timestampOptions.startTimestamp"
-        [maxTimestamp]="video.duration"
-        [disabled]="!timestampOptions.startTimestampEnabled"
-        [(ngModel)]="timestampOptions.startTimestamp"
-      ></my-timestamp-input>
-    </div>
+        <my-timestamp-input
+          [timestamp]="timestampOptions.startTimestamp"
+          [maxTimestamp]="video.duration"
+          [disabled]="!timestampOptions.startTimestampEnabled"
+          [(ngModel)]="timestampOptions.startTimestamp"
+        ></my-timestamp-input>
+      </div>
 
-    <div>
-      <my-peertube-checkbox
-        inputName="stopAt" [(ngModel)]="timestampOptions.stopTimestampEnabled"
-        i18n-labelText labelText="Stop at"
-      ></my-peertube-checkbox>
+      <div>
+        <my-peertube-checkbox
+          inputName="stopAt" [(ngModel)]="timestampOptions.stopTimestampEnabled"
+          i18n-labelText labelText="Stop at"
+        ></my-peertube-checkbox>
 
-      <my-timestamp-input
-        [timestamp]="timestampOptions.stopTimestamp"
-        [maxTimestamp]="video.duration"
-        [disabled]="!timestampOptions.stopTimestampEnabled"
-        [(ngModel)]="timestampOptions.stopTimestamp"
-      ></my-timestamp-input>
+        <my-timestamp-input
+          [timestamp]="timestampOptions.stopTimestamp"
+          [maxTimestamp]="video.duration"
+          [disabled]="!timestampOptions.stopTimestampEnabled"
+          [(ngModel)]="timestampOptions.stopTimestamp"
+        ></my-timestamp-input>
+      </div>
     </div>
   </div>
-</div>
 
-<div class="playlist dropdown-item" *ngFor="let playlist of videoPlaylists" (click)="togglePlaylist($event, playlist)">
-  <my-peertube-checkbox [inputName]="'in-playlist-' + playlist.id" [(ngModel)]="playlist.inPlaylist"></my-peertube-checkbox>
+  <div class="playlist dropdown-item" *ngFor="let playlist of videoPlaylists" (click)="togglePlaylist($event, playlist)">
+    <my-peertube-checkbox [inputName]="'in-playlist-' + playlist.id" [(ngModel)]="playlist.inPlaylist"></my-peertube-checkbox>
 
-  <div class="display-name">
-    {{ playlist.displayName }}
+    <div class="display-name">
+      {{ playlist.displayName }}
 
-    <div *ngIf="playlist.inPlaylist && (playlist.startTimestamp || playlist.stopTimestamp)" class="timestamp-info">
-      {{ formatTimestamp(playlist) }}
+      <div *ngIf="playlist.inPlaylist && (playlist.startTimestamp || playlist.stopTimestamp)" class="timestamp-info">
+        {{ formatTimestamp(playlist) }}
+      </div>
     </div>
   </div>
-</div>
 
-<div class="new-playlist-button dropdown-item" (click)="openCreateBlock($event)" [hidden]="isNewPlaylistBlockOpened">
-  <my-global-icon iconName="add"></my-global-icon>
+  <div class="new-playlist-button dropdown-item" (click)="openCreateBlock($event)" [hidden]="isNewPlaylistBlockOpened">
+    <my-global-icon iconName="add"></my-global-icon>
 
-  Create a new playlist
-</div>
+    Create a new playlist
+  </div>
 
-<form class="new-playlist-block dropdown-item" *ngIf="isNewPlaylistBlockOpened" (ngSubmit)="createPlaylist()" [formGroup]="form">
-  <div class="form-group">
-    <label i18n for="displayName">Display name</label>
-    <input
-      type="text" id="displayName"
-      formControlName="displayName" [ngClass]="{ 'input-error': formErrors['displayName'] }"
-    >
-    <div *ngIf="formErrors['displayName']" class="form-error">
-      {{ formErrors['displayName'] }}
+  <form class="new-playlist-block dropdown-item" *ngIf="isNewPlaylistBlockOpened" (ngSubmit)="createPlaylist()" [formGroup]="form">
+    <div class="form-group">
+      <label i18n for="displayName">Display name</label>
+      <input
+        type="text" id="displayName"
+        formControlName="displayName" [ngClass]="{ 'input-error': formErrors['displayName'] }"
+      >
+      <div *ngIf="formErrors['displayName']" class="form-error">
+        {{ formErrors['displayName'] }}
+      </div>
     </div>
-  </div>
 
-  <input type="submit" i18n-value value="Create" [disabled]="!form.valid">
-</form>
+    <input type="submit" i18n-value value="Create" [disabled]="!form.valid">
+  </form>
+</div>
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 bc0d55912..0424e2ee9 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
@@ -1,6 +1,11 @@
 @import '_variables';
 @import '_mixins';
 
+.root {
+  max-height: 300px;
+  overflow-y: auto;
+}
+
 .header {
   min-width: 240px;
   padding: 6px 24px 10px 24px;
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 705f62404..152f20c85 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
@@ -24,6 +24,7 @@ type PlaylistSummary = {
 export class VideoAddToPlaylistComponent extends FormReactive implements OnInit {
   @Input() video: Video
   @Input() currentVideoTimestamp: number
+  @Input() lazyLoad = false
 
   isNewPlaylistBlockOpened = false
   videoPlaylists: PlaylistSummary[] = []
@@ -57,6 +58,10 @@ export class VideoAddToPlaylistComponent extends FormReactive implements OnInit
       displayName: this.videoPlaylistValidatorsService.VIDEO_PLAYLIST_DISPLAY_NAME
     })
 
+    if (this.lazyLoad !== true) this.load()
+  }
+
+  load () {
     forkJoin([
       this.videoPlaylistService.listAccountPlaylists(this.user.account, '-updatedAt'),
       this.videoPlaylistService.doesVideoExistInPlaylist(this.video.id)