Fix issue with scrolling modal and allowMultiple #2423

Semantic-Org · Jan 29, 2018 · aa1d36b · aa1d36b
1 parent 3159fa1
commit aa1d36b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/RELEASE-NOTES.md b/RELEASE-NOTES.md
@@ -4,14 +4,14 @@
 
 **Critical Bugs**
 - **Form** - Fixes issue where radio checkbox would not return correct value from `get values` **Thanks @tincdev** #5713 #6043
+- **Modal** - Fixes issue where an oversized modal would appear behind an existing modal when using `allowMultiple: true` and a second modal that is larger than the sreen height. #2423
 
 **Enhancements**
 - **Button** - Youtube's red color now matches their current brand guidelines **Thanks @hammy2899** #6110
 - **Flag** - Adds missing flag for England **Thanks @zyzniewski** #5944
 - **Reveal** - `ribbon label` can now work with `reveal` #5681
 
 **Bugs**
-- **Form** - Fixe
 - **Dropdown** - Fixed issue where dropdown could incorrectly open upward and leftward opening when using `context` setting due to an incorrect offset calculation. **Thanks @dannyBies** #5974 #5366
 - **Visibility** - Fixes issue where `bottomPassed` and `topPassed` would not fire under some conditions
 - **Dropdown** - Fixes issue where dropdowns might accidentally animate closed two times when quickly tabbing through fields

diff --git a/src/definitions/modules/modal.less b/src/definitions/modules/modal.less
@@ -353,7 +353,9 @@
   position: fixed;
 }
 .modals.dimmer .ui.scrolling.modal {
-  position: static !important;
+  position: relative !important;
+  left: auto !important;
+  top: auto !important;
   margin: @scrollingMargin auto !important;
 }