From cf78883c70dca99fc519374d55620d9403d482be Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Thu, 11 Jun 2020 14:33:33 +0200
Subject: Limit thumbnail sizes

---
 .../src/app/shared/video/abstract-video-list.html  | 16 ++++++++-------
 .../src/app/shared/video/abstract-video-list.scss  | 23 +++-------------------
 client/src/app/shared/video/abstract-video-list.ts |  7 ++++++-
 .../shared/video/video-miniature.component.html    | 22 +++++++++++----------
 .../shared/video/video-miniature.component.scss    | 14 +++++++++----
 .../app/shared/video/video-miniature.component.ts  |  8 ++++++++
 6 files changed, 48 insertions(+), 42 deletions(-)

(limited to 'client/src/app/shared/video')

diff --git a/client/src/app/shared/video/abstract-video-list.html b/client/src/app/shared/video/abstract-video-list.html
index 8ce3b25b0..9e551c1c8 100644
--- a/client/src/app/shared/video/abstract-video-list.html
+++ b/client/src/app/shared/video/abstract-video-list.html
@@ -35,13 +35,15 @@
         {{ getCurrentGroupedDateLabel(video) }}
       </div>
 
-      <my-video-miniature
-        [fitWidth]="true"
-        [video]="video" [user]="user" [ownerDisplayType]="ownerDisplayType"
-        [displayVideoActions]="displayVideoActions" [displayOptions]="displayOptions"
-        (videoBlocked)="removeVideoFromArray(video)" (videoRemoved)="removeVideoFromArray(video)"
-      >
-      </my-video-miniature>
+      <div class="video-wrapper">
+        <my-video-miniature
+          [fitWidth]="true"
+          [video]="video" [user]="user" [ownerDisplayType]="ownerDisplayType"
+          [displayVideoActions]="displayVideoActions" [displayOptions]="displayOptions"
+          (videoBlocked)="removeVideoFromArray(video)" (videoRemoved)="removeVideoFromArray(video)"
+        >
+        </my-video-miniature>
+      </div>
     </ng-container>
   </div>
 </div>
diff --git a/client/src/app/shared/video/abstract-video-list.scss b/client/src/app/shared/video/abstract-video-list.scss
index 982204e21..7f23098aa 100644
--- a/client/src/app/shared/video/abstract-video-list.scss
+++ b/client/src/app/shared/video/abstract-video-list.scss
@@ -3,24 +3,6 @@
 @import '_mixins';
 @import '_miniature';
 
-.videos {
-  $column-width: #{$video-thumbnail-width - 25px};
-
-  display: grid;
-  column-gap: calc(10px + .2%);
-  grid-template-columns: repeat(
-    auto-fill,
-    minmax(
-      var(--miniature-min-width, #{$column-width}),
-      1fr
-    )
-  );
-
-  @media screen and (min-width: #{breakpoint(fhd)}) {
-    --miniature-min-width: #{$column-width + 100px};
-  }
-}
-
 .videos-header {
   display: flex;
   justify-content: space-between;
@@ -74,8 +56,8 @@
   }
 }
 
-:host-context(.main-col:not(.expanded)) .margin-content {
-  @include adapt-margin-content-width($fluid: true);
+.margin-content {
+  @include fluid-videos-miniature-layout;
 }
 
 @media screen and (max-width: $mobile-view) {
@@ -83,6 +65,7 @@
     flex-direction: column;
     align-items: center;
     height: auto;
+    margin-bottom: 10px;
 
     .title-page {
       margin-bottom: 10px;
diff --git a/client/src/app/shared/video/abstract-video-list.ts b/client/src/app/shared/video/abstract-video-list.ts
index 69d5c0010..76aa683fc 100644
--- a/client/src/app/shared/video/abstract-video-list.ts
+++ b/client/src/app/shared/video/abstract-video-list.ts
@@ -56,7 +56,7 @@ export abstract class AbstractVideoList implements OnInit, OnDestroy, DisableFor
     date: true,
     views: true,
     by: true,
-    avatar: true,
+    avatar: false,
     privacyLabel: true,
     privacyText: false,
     state: false,
@@ -139,6 +139,11 @@ export abstract class AbstractVideoList implements OnInit, OnDestroy, DisableFor
         if (this.hasDoneFirstQuery) this.reloadVideos()
       }
     )
+
+    // Display avatar in mobile view
+    if (this.screenService.isInMobileView()) {
+      this.displayOptions.avatar = true
+    }
   }
 
   ngOnDestroy () {
diff --git a/client/src/app/shared/video/video-miniature.component.html b/client/src/app/shared/video/video-miniature.component.html
index 575505f63..d37ab478d 100644
--- a/client/src/app/shared/video/video-miniature.component.html
+++ b/client/src/app/shared/video/video-miniature.component.html
@@ -9,32 +9,34 @@
 
   <div class="video-bottom">
     <div class="video-miniature-information">
-      <a
-        tabindex="-1"
-        class="video-miniature-name"
-        [routerLink]="videoLink" [attr.title]="video.name" [ngClass]="{ 'blur-filter': isVideoBlur }"
-      >{{ video.name }}</a>
-
       <div class="d-inline-flex">
-        <avatar-channel *ngIf="displayOptions.avatar" class="mr-1 pt-1" [video]="video" size="sm"></avatar-channel>
+        <div *ngIf="displayOptions.avatar" class="avatar">
+          <img [src]="getAvatarUrl()" alt="Avatar" />
+        </div>
 
         <div class="d-flex flex-column">
+          <a
+            tabindex="-1"
+            class="video-miniature-name"
+            [routerLink]="videoLink" [attr.title]="video.name" [ngClass]="{ 'blur-filter': isVideoBlur }"
+          >{{ video.name }}</a>
+
           <span class="video-miniature-created-at-views">
             <my-date-toggle *ngIf="displayOptions.date" [date]="video.publishedAt"></my-date-toggle>
-    
+
             <span class="views">
               <ng-container *ngIf="displayOptions.date && displayOptions.views"> • </ng-container>
               <ng-container i18n *ngIf="displayOptions.views">{video.views, plural, =1 {1 view} other {{{ video.views | myNumberFormatter }} views}}</ng-container>
             </span>
           </span>
-    
+
           <a tabindex="-1" *ngIf="displayOptions.by && displayOwnerAccount()" class="video-miniature-account" [routerLink]="[ '/accounts', video.byAccount ]">
             {{ video.byAccount }}
           </a>
           <a tabindex="-1" *ngIf="displayOptions.by && displayOwnerVideoChannel()" class="video-miniature-channel" [routerLink]="[ '/video-channels', video.byVideoChannel ]">
             {{ video.byVideoChannel }}
           </a>
-    
+
           <div class="video-info-privacy">
             <ng-container *ngIf="displayOptions.privacyText">{{ video.privacy.label }}</ng-container>
             <ng-container *ngIf="displayOptions.privacyText && displayOptions.state && getStateLabel(video)"> - </ng-container>
diff --git a/client/src/app/shared/video/video-miniature.component.scss b/client/src/app/shared/video/video-miniature.component.scss
index 1e5580b9a..4e7c9fb57 100644
--- a/client/src/app/shared/video/video-miniature.component.scss
+++ b/client/src/app/shared/video/video-miniature.component.scss
@@ -19,16 +19,22 @@ $more-margin-right: 15px;
     .video-miniature-information {
       width: $video-miniature-width - $more-button-width - $more-margin-right;
       line-height: normal;
-      font-size: 13px;
+
+      .avatar {
+        margin: 10px 10px 0 0;
+
+        img {
+          @include avatar(40px);
+        }
+      }
 
       .video-miniature-name {
         @include miniature-name;
-        font-size: 110%;
       }
 
       .video-miniature-created-at-views {
         display: block;
-        font-size: 95%;
+        font-size: 13px;
       }
 
       .video-miniature-account,
@@ -37,7 +43,7 @@ $more-margin-right: 15px;
         @include ellipsis;
 
         display: block;
-        font-size: 95%;
+        font-size: 13px;
         color: pvar(--greyForegroundColor);
 
         &:hover {
diff --git a/client/src/app/shared/video/video-miniature.component.ts b/client/src/app/shared/video/video-miniature.component.ts
index f0b0992e2..ccf90af54 100644
--- a/client/src/app/shared/video/video-miniature.component.ts
+++ b/client/src/app/shared/video/video-miniature.component.ts
@@ -178,6 +178,14 @@ export class VideoMiniatureComponent implements OnInit {
     return ''
   }
 
+  getAvatarUrl () {
+    if (this.ownerDisplayTypeChosen === 'account') {
+      return this.video.accountAvatarUrl
+    }
+
+    return this.video.videoChannelAvatarUrl
+  }
+
   loadActions () {
     if (this.displayVideoActions) this.showActions = true
 
-- 
cgit v1.2.3