From 8ff3f88347cb733ca8a3978ba6eb9763fb3ea294 Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Wed, 5 Sep 2018 11:20:44 +0200
Subject: Fix watch page responsive

---
 client/src/app/menu/menu.component.scss            |  3 +-
 client/src/app/shared/misc/screen.service.ts       |  2 +-
 .../videos/+video-watch/video-watch.component.html | 10 ++---
 .../videos/+video-watch/video-watch.component.scss | 48 +++++++---------------
 .../videos/+video-watch/video-watch.component.ts   |  4 --
 5 files changed, 21 insertions(+), 46 deletions(-)

(limited to 'client/src/app')

diff --git a/client/src/app/menu/menu.component.scss b/client/src/app/menu/menu.component.scss
index c725580d1..a699da12f 100644
--- a/client/src/app/menu/menu.component.scss
+++ b/client/src/app/menu/menu.component.scss
@@ -6,6 +6,7 @@
   height: calc(100vh - #{$header-height});
   padding: 0;
   width: $menu-width;
+  z-index: 10000;
 }
 
 menu {
@@ -16,7 +17,6 @@ menu {
   white-space: nowrap;
   text-overflow: ellipsis;
   overflow: hidden;
-  z-index: 1000;
   color: $menu-color;
   display: flex;
   flex-direction: column;
@@ -230,7 +230,6 @@ menu {
 @media screen and (max-width: 400px) {
   .menu-wrapper {
     width: 100% !important;
-    z-index: 10000;
   }
 
   .top-menu {
diff --git a/client/src/app/shared/misc/screen.service.ts b/client/src/app/shared/misc/screen.service.ts
index 2e01839b2..1cbc96b14 100644
--- a/client/src/app/shared/misc/screen.service.ts
+++ b/client/src/app/shared/misc/screen.service.ts
@@ -11,7 +11,7 @@ export class ScreenService {
   }
 
   isInSmallView () {
-    return this.getWindowInnerWidth() < 600
+    return this.getWindowInnerWidth() < 800
   }
 
   isInMobileView () {
diff --git a/client/src/app/videos/+video-watch/video-watch.component.html b/client/src/app/videos/+video-watch/video-watch.component.html
index 582cf6e37..cafd92dbd 100644
--- a/client/src/app/videos/+video-watch/video-watch.component.html
+++ b/client/src/app/videos/+video-watch/video-watch.component.html
@@ -110,8 +110,9 @@
                 <div
                   class="video-info-likes-dislikes-bar"
                   *ngIf="video.likes !== 0 || video.dislikes !== 0"
+                  [ngbTooltip]="likesBarTooltipText"
                   placement="bottom"
-                  [ngbTooltip]="likesBarTooltipText">
+                >
                   <div class="likes-bar" [ngStyle]="{ 'width.%': video.likesPercent }"></div>
                 </div>
               </div>
@@ -200,13 +201,10 @@
         <my-video-comments [video]="video" [user]="user"></my-video-comments>
       </div>
 
-      <div *ngIf="!isMenuExpanded()" class="w-100-until-1150px"></div>
-      
-      <my-recommended-videos class="col-12 col-lg-3"
-        [inputRecommendation]="{ uuid: video.uuid, tags: video.tags }" [user]="user"></my-recommended-videos>
+      <my-recommended-videos [inputRecommendation]="{ uuid: video.uuid, tags: video.tags }" [user]="user"></my-recommended-videos>
     </div>
   </div>
-  
+
   <div class="privacy-concerns" *ngIf="hasAlreadyAcceptedPrivacyConcern === false">
     <div class="privacy-concerns-text">
       <strong i18n>Friendly Reminder: </strong>
diff --git a/client/src/app/videos/+video-watch/video-watch.component.scss b/client/src/app/videos/+video-watch/video-watch.component.scss
index edd5ae6ba..58c45cde9 100644
--- a/client/src/app/videos/+video-watch/video-watch.component.scss
+++ b/client/src/app/videos/+video-watch/video-watch.component.scss
@@ -99,9 +99,10 @@
       }
 
       .video-info-date-views {
-        font-size: 16px;
-        margin-bottom: 10px;
         flex-grow: 1;
+        margin-bottom: 10px;
+        margin-right: 10px;
+        font-size: 16px;
       }
 
       .video-info-channel {
@@ -164,8 +165,7 @@
       }
 
       .video-actions-rates {
-        margin-top: 20px;
-        margin-bottom: 10px;
+        margin: 20px 0 10px 0;
         align-items: start;
 
         .video-actions {
@@ -361,12 +361,13 @@
 
   /deep/ .other-videos {
     padding-left: 1em;
+    width: 260px;
 
     .title-page {
       margin-top: 0 !important;
     }
 
-    /deep/ .video-miniature {
+    .video-miniature {
       display: flex;
       height: 100%;
       margin-bottom: 20px;
@@ -388,9 +389,16 @@
   width: calc(100% - #{$menu-width});
 }
 
+@media screen and (max-width: $small-view) {
+  .privacy-concerns {
+    margin-left: $menu-width;
+  }
+}
+
 :host-context(.expanded) {
   .privacy-concerns {
     width: 100%;
+    margin-left: 0;
   }
 }
 
@@ -438,34 +446,12 @@
 
 
 @media screen and (max-width: 1600px) {
-  .video-bottom {
-    .video-info {
-      .video-info-first-row {
-        flex-direction: column;
-        margin-bottom: 20px;
-      }
-
-      .video-info-date-views {
-        flex-direction: column;
-        margin-bottom: 30px;
-
-        .video-info-likes-dislikes-bar {
-          margin-top: 0;
-        }
-      }
-
-      .video-attributes .video-attribute {
-        margin-bottom: 5px;
-      }
-    }
+  .video-bottom .video-info .video-attributes .video-attribute {
+    margin-bottom: 5px;
   }
 }
 
 @media screen and (max-width: 1300px) {
-  .video-bottom {
-    flex-direction: column;
-  }
-
   /deep/ .other-videos {
     padding-left: 0 !important;
   }
@@ -480,10 +466,6 @@
   }
 }
 
-@media (max-width: 1150px) {
-  .w-100-until-1150px { width: 100% !important }
-}
-
 @media screen and (max-width: 600px) {
   .video-bottom {
     margin: 20px 0 0 0;
diff --git a/client/src/app/videos/+video-watch/video-watch.component.ts b/client/src/app/videos/+video-watch/video-watch.component.ts
index 95198f0aa..768a08d42 100644
--- a/client/src/app/videos/+video-watch/video-watch.component.ts
+++ b/client/src/app/videos/+video-watch/video-watch.component.ts
@@ -317,10 +317,6 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
     return this.video && this.video.state.id === VideoState.TO_IMPORT
   }
 
-  isMenuExpanded () {
-    return document.getElementById('right-container').classList.contains('expanded')
-  }
-
   hasVideoScheduledPublication () {
     return this.video && this.video.scheduledUpdate !== undefined
   }
-- 
cgit v1.2.3