Merge pull request danskernesdigitalebibliotek#210 from itk-dev/featu…

…re/loan-list-testfeedback Feature/Loan list additions
reload · Mar 27, 2023 · ceaa97a · ceaa97a
2 parents 44ae1cc + 69e35e6
commit ceaa97a
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/src/stories/Library/Modals/modal-details/modal-details.scss b/src/stories/Library/Modals/modal-details/modal-details.scss
@@ -12,12 +12,12 @@ $MODAL_DETAILS_CONTAINER_WIDTH: 1000px;
 
 .modal-details__container {
   width: 100%;
-  overflow: scroll;
+  overflow: auto;
   max-width: 100%;
   padding-bottom: $s-2xl;
 
   @include breakpoint-s {
-    margin: 0 45px;
+    margin: 0;
     padding-bottom: 0;
   }
 }
@@ -56,8 +56,8 @@ $MODAL_DETAILS_CONTAINER_WIDTH: 1000px;
   margin-bottom: 16px;
 }
 
-/* 
-  We disable stylelint here since -webkit-box 
+/*
+  We disable stylelint here since -webkit-box
   uses ellipsis for truncating which we need.
   https://caniuse.com/?search=line-clamp
 */

diff --git a/src/stories/Library/Modals/modal-loan/modal-loan.scss b/src/stories/Library/Modals/modal-loan/modal-loan.scss
@@ -36,6 +36,10 @@ $MODAL_LOAN_CONTAINER_WIDTH: 1000px;
   grid-row-gap: 8px;
 }
 
+.modal-loan__list-materials--bottom-buttons-visible {
+  padding-bottom: #{$s-75}px;
+}
+
 .modal-loan__buttons {
   display: flex;
   grid-gap: 24px;

diff --git a/src/stories/Library/Modals/modal.scss b/src/stories/Library/Modals/modal.scss
@@ -20,7 +20,7 @@
   display: none;
   transition: opacity 0.3s;
   height: 100vh;
-  overflow: scroll;
+  overflow: auto;
 
   &__screen-reader-description {
     @include hide-visually;

diff --git a/src/stories/Library/spacing/spacing.scss b/src/stories/Library/spacing/spacing.scss
@@ -7,6 +7,7 @@ $s-32: 32;
 $s-35: 35;
 $s-48: 48;
 $s-64: 64;
+$s-75: 75;
 $s-80: 80;
 $s-96: 96;
 $s-112: 112;