show less information by default in download modal (#3890)

* client(video/download): collapse advanced informaton by default
* fix(client/video-dl): update metadata upon resolution change

closes #3810

Co-authored-by: Rigel Kent <par@rigelk.eu>

client/src/app/shared/shared-video-miniature/video-download.component.html

@@ -17,22 +17,30 @@
   </div>
 
   <div class="modal-body">
-    <div class="form-group">
     <div class="alert alert-warning" *ngIf="isConfidentialVideo()" i18n>
       The following link contains a private token and should not be shared with anyone.
     </div>
 
+    <ng-container *ngIf="type === 'subtitles'">
       <div class="input-group input-group-sm">
-        <div class="input-group-prepend peertube-select-container">
+        <input #urlInput (click)="urlInput.select()" type="text" class="form-control input-sm readonly" readonly [value]="getLink()" />
-          <select *ngIf="type === 'video'" [(ngModel)]="resolutionId" (ngModelChange)="onResolutionIdChange()">
+        <div class="input-group-append" *ngIf="!isConfidentialVideo()">
-            <option *ngFor="let file of getVideoFiles()" [value]="file.resolution.id">{{ file.resolution.label }}</option>
+          <button [cdkCopyToClipboard]="urlInput.value" (click)="activateCopiedMessage()" type="button" class="btn btn-outline-secondary">
-          </select>
+            <span class="glyphicon glyphicon-duplicate"></span>
-
+          </button>
-          <select *ngIf="type === 'subtitles'" [(ngModel)]="subtitleLanguageId">
-            <option *ngFor="let caption of videoCaptions" [value]="caption.language.id">{{ caption.language.label }}</option>
-          </select>
         </div>
+      </div>
+    </ng-container>
 
+    <ng-container *ngIf="type === 'video'">
+      <div ngbNav #resolutionNav="ngbNav" class="nav-tabs" [activeId]="resolutionId" (activeIdChange)="onResolutionIdChange($event)">
+
+        <ng-container *ngFor="let file of getVideoFiles()" [ngbNavItem]="file.resolution.id">
+          <a ngbNavLink i18n>{{ file.resolution.label }}</a>
+  
+          <ng-template ngbNavContent>
+            <div class="nav-content">
+              <div class="input-group input-group-sm">
                 <input #urlInput (click)="urlInput.select()" type="text" class="form-control input-sm readonly" readonly [value]="getLink()" />
                 <div class="input-group-append" *ngIf="!isConfidentialVideo()">
                   <button [cdkCopyToClipboard]="urlInput.value" (click)="activateCopiedMessage()" type="button" class="btn btn-outline-secondary">
@@ -41,10 +49,14 @@
                 </div>
               </div>
             </div>
+          </ng-template>
+        </ng-container>
+      </div>
+      <div [ngbNavOutlet]="resolutionNav"></div>
   
-    <ng-container *ngIf="type === 'video' && videoFile?.metadata">
+      <div class="advanced-filters collapse-transition" [ngbCollapse]="isAdvancedCustomizationCollapsed">
-      <div ngbNav #nav="ngbNav" class="nav-tabs">
+        <ng-container *ngIf="videoFile?.metadata">
-
+          <div ngbNav #nav="ngbNav" class="nav-tabs nav-metadata">
             <ng-container ngbNavItem>
               <a ngbNavLink i18n>Format</a>
               <ng-template ngbNavContent>
@@ -55,7 +67,6 @@
                   </div>
                 </div>
               </ng-template>
-        </ng-container>
   
               <ng-container ngbNavItem [disabled]="videoFileMetadataVideoStream === undefined">
                 <a ngbNavLink i18n>Video stream</a>
@@ -80,22 +91,42 @@
                   </div>
                 </ng-template>
               </ng-container>
-      </div>
   
-      <div [ngbNavOutlet]="nav"></div>
             </ng-container>
-
+          </div>
-    <div class="download-type" *ngIf="type === 'video'">
+          <div [ngbNavOutlet]="nav"></div>
+          <div class="download-type">
             <div class="peertube-radio-container">
               <input type="radio" name="download" id="download-direct" [(ngModel)]="downloadType" value="direct">
               <label i18n for="download-direct">Direct download</label>
             </div>
-
             <div class="peertube-radio-container">
               <input type="radio" name="download" id="download-torrent" [(ngModel)]="downloadType" value="torrent">
               <label i18n for="download-torrent">Torrent (.torrent file)</label>
             </div>
           </div>
+        </ng-container>
+      </div>
+  
+      <div (click)="isAdvancedCustomizationCollapsed = !isAdvancedCustomizationCollapsed" role="button" class="advanced-filters-button"
+      [attr.aria-expanded]="!isAdvancedCustomizationCollapsed" aria-controls="collapseBasic">
+        <ng-container *ngIf="isAdvancedCustomizationCollapsed">
+          <span class="glyphicon glyphicon-menu-down"></span>
+  
+          <ng-container i18n>
+            Advanced
+          </ng-container>
+        </ng-container>
+  
+        <ng-container *ngIf="!isAdvancedCustomizationCollapsed">
+          <span class="glyphicon glyphicon-menu-up"></span>
+  
+          <ng-container i18n>
+            Simple
+          </ng-container>
+        </ng-container>
+      </div>
+    </ng-container>
   </div>
 
   <div class="modal-footer inputs">

client/src/app/shared/shared-video-miniature/video-download.component.scss

@@ -1,6 +1,28 @@
 @import 'variables';
 @import 'mixins';
 
+.nav-content {
+  margin-top: 30px;
+}
+
+.advanced-filters-button {
+  display: flex;
+  justify-content: center;
+  align-items: center;
+  margin-top: 20px;
+  font-size: 16px;
+  font-weight: 600;
+  cursor: pointer;
+
+  .nav-tabs {
+    margin-top: 10x;
+  }
+  
+  .glyphicon {
+    margin-right: 5px;
+  }
+}
+
 .peertube-select-container {
   @include peertube-select-container(85px);
 
@@ -15,12 +37,21 @@
   }
 }
 
+.action-button-cancel {
+  @include peertube-button-link;
+}
+
+.action-button-submit {
+  @include peertube-button-link;
+  @include orange-button;
+}
+
 #dropdownDownloadType {
   cursor: pointer;
 }
 
 .download-type {
-  margin-top: 30px;
+  margin-top: 20px;
 
   .peertube-radio-container {
     @include peertube-radio-container;
@@ -30,6 +61,10 @@
   }
 }
 
+.nav-metadata {
+  margin-top: 20px;
+}
+
 .file-metadata {
   padding: 1rem;
 }

client/src/app/shared/shared-video-miniature/video-download.component.ts

@@ -30,6 +30,8 @@ export class VideoDownloadComponent {
   videoCaptions: VideoCaption[]
   activeModal: NgbModalRef
 
+  isAdvancedCustomizationCollapsed = true
+
   type: DownloadType = 'video'
 
   private bytesPipe: BytesPipe
@@ -65,8 +67,7 @@ export class VideoDownloadComponent {
 
     this.activeModal = this.modalService.open(this.modal, { centered: true })
 
-    this.resolutionId = this.getVideoFiles()[0].resolution.id
+    this.onResolutionIdChange(this.getVideoFiles()[0].resolution.id)
-    this.onResolutionIdChange()
 
     if (this.videoCaptions) this.subtitleLanguageId = this.videoCaptions[0].language.id
 
@@ -91,12 +92,15 @@ export class VideoDownloadComponent {
       : this.getVideoFileLink()
   }
 
-  async onResolutionIdChange () {
+  async onResolutionIdChange (resolutionId: number) {
+    this.resolutionId = resolutionId
     this.videoFile = this.getVideoFile()
-    if (this.videoFile.metadata || !this.videoFile.metadataUrl) return
+
+    if (!this.videoFile.metadata) {
+      if (!this.videoFile.metadataUrl) return
 
       await this.hydrateMetadataFromMetadataUrl(this.videoFile)
-    if (!this.videoFile.metadata) return
+    }
 
     this.videoFileMetadataFormat = this.videoFile
       ? this.getMetadataFormat(this.videoFile.metadata.format)
@@ -110,9 +114,6 @@ export class VideoDownloadComponent {
   }
 
   getVideoFile () {
-    // HTML select send us a string, so convert it to a number
-    this.resolutionId = parseInt(this.resolutionId.toString(), 10)
-
     const file = this.getVideoFiles().find(f => f.resolution.id === this.resolutionId)
     if (!file) {
       console.error('Could not find file with resolution %d.', this.resolutionId)