From bd70b6274f2758975312223762b27abcfe7b4726 Mon Sep 17 00:00:00 2001
From: Andres Galante <agalante@gmail.com>
Date: Thu, 16 Nov 2017 08:56:13 -0300
Subject: [PATCH] lets the centered modal have any height

---
 docs/4.0/components/modal.md | 2 +-
 scss/_modal.scss             | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/docs/4.0/components/modal.md b/docs/4.0/components/modal.md
index 214ed01169ea..5a6e000e9268 100644
--- a/docs/4.0/components/modal.md
+++ b/docs/4.0/components/modal.md
@@ -210,7 +210,7 @@ When modals become too long for the user's viewport or device, they scroll indep
 
 ### Vertically centered
 
-Add `.modal-dialog-centered` to `.modal-dialog` to vertically center the modal. **Do not use this with long modals**—it will overflow the viewport and potentially hide parts of your modal.
+Add `.modal-dialog-centered` to `.modal-dialog` to vertically center the modal.
 
 <div id="exampleModalCenter" class="modal fade" tabindex="-1" role="dialog" aria-labelledby="exampleModalCenterTitle" aria-hidden="true">
   <div class="modal-dialog modal-dialog-centered" role="document">
diff --git a/scss/_modal.scss b/scss/_modal.scss
index bd4abc7c6410..edda836636dc 100644
--- a/scss/_modal.scss
+++ b/scss/_modal.scss
@@ -53,9 +53,7 @@
 .modal-dialog-centered {
   display: flex;
   align-items: center;
-  height: 100%;
-  margin-top: 0;
-  margin-bottom: 0;
+  min-height: calc(100% - (#{$modal-dialog-margin} * 2));
 }
 
 // Actual modal
@@ -154,8 +152,7 @@
   }
 
   .modal-dialog-centered {
-    margin-top: 0;
-    margin-bottom: 0;
+    min-height: calc(100% - (#{$modal-dialog-margin-y-sm-up} * 2));
   }
 
   .modal-content {