From 558fa2cee1c9ce39ad2e6ee33a88bcead2bdaa52 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Erik=20V=C3=A4nttinen?= Date: Thu, 21 Mar 2024 11:27:51 +0200 Subject: [PATCH] TMS-1021: Remove image carousel modal max-height --- CHANGELOG.MD | 2 ++ assets/styles/layouts/_image-carousel.scss | 23 +++++++++------------- 2 files changed, 11 insertions(+), 14 deletions(-) diff --git a/CHANGELOG.MD b/CHANGELOG.MD index a3c4ecf..7d93e6c 100644 --- a/CHANGELOG.MD +++ b/CHANGELOG.MD @@ -7,6 +7,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0. ## [Unreleased] +- TMS-1021: Remove image carousel modal max-height + ## [1.2.2] - 2024-03-18 - TMS-1017: Add image alt-text to imported news single-article diff --git a/assets/styles/layouts/_image-carousel.scss b/assets/styles/layouts/_image-carousel.scss index b270a0d..d6ca394 100644 --- a/assets/styles/layouts/_image-carousel.scss +++ b/assets/styles/layouts/_image-carousel.scss @@ -12,7 +12,7 @@ display: inline-block; overflow: hidden; max-width: 70vw; - + img { object-fit: contain; clip-path: circle(closest-side); @@ -40,7 +40,7 @@ &.slick-arrow { background-color: transparent; - border:none; + border: none; .icon { fill: $white; @@ -64,7 +64,7 @@ right: 0; } } - + .image-carousel__item { height: 80vh; display: flex !important; @@ -77,33 +77,28 @@ width: auto; } } - - .img-wrapper{ - flex:2 1 auto; + + .img-wrapper { + flex: 2 1 auto; max-height: 100%; - height:auto; + height: auto; min-width: 0; min-height: 50%; display: flex; align-items: flex-end; - img{ + img { max-width: 100%; - max-height: 100% !important; min-height: 100px; object-fit: contain; } } - + .image-block__meta { flex:1 1 auto; width: 100%; } - - } - - } .slick-track .modal-trigger:focus {