From 03870964f42ed3bf7be1cc5c29995eb612557e41 Mon Sep 17 00:00:00 2001
From: Nicolas Constant <github@nicolas-constant.com>
Date: Thu, 11 Jul 2019 19:37:43 -0400
Subject: [PATCH] fixed maximum height in image display

---
 src/app/components/media-viewer/media-viewer.component.html | 6 +++---
 src/app/components/media-viewer/media-viewer.component.scss | 3 ++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/app/components/media-viewer/media-viewer.component.html b/src/app/components/media-viewer/media-viewer.component.html
index fbe6df09..2c3e29b5 100644
--- a/src/app/components/media-viewer/media-viewer.component.html
+++ b/src/app/components/media-viewer/media-viewer.component.html
@@ -16,9 +16,9 @@
     </button>
 
     <div *ngFor="let att of attachments" class="media-viewer-canvas__attachement"
-        [ngClass]="{ 'collapsed': currentIndex !== att.index }">
-        <a href="{{att.url}}" target="_blank" title="open image">
-            <img *ngIf="att.type === 'image'" src="{{att.url}}" class="media-viewer-canvas__image" />
+        [class.collapsed]="currentIndex !== att.index">
+        <a href="{{att.url}}" target="_blank" title="{{att.description}}">
+            <img *ngIf="att.type === 'image'" src="{{att.url}}" class="media-viewer-canvas__image"   [class.collapsed]="currentIndex !== att.index" />
         </a>
 
         <video *ngIf="att.type === 'gifv'" class="media-viewer-canvas__image" role="application" loop autoplay>
diff --git a/src/app/components/media-viewer/media-viewer.component.scss b/src/app/components/media-viewer/media-viewer.component.scss
index 2239266a..a4f0e689 100644
--- a/src/app/components/media-viewer/media-viewer.component.scss
+++ b/src/app/components/media-viewer/media-viewer.component.scss
@@ -76,7 +76,7 @@
             max-width: 95%;
         }   
 
-        max-height: calc(100% - 120px);
+        max-height: 90vh;
         margin-top: 50vh;
         margin-left: 50vw;
         transform: translate(-50%, -50%);
@@ -104,4 +104,5 @@
 
 .collapsed {
     height: 0;
+    max-height: 0;
 }