Fixes #1666. Fixes #1792. Add scrollbars for modals

CHANGES.md

@@ -17,8 +17,10 @@
 
 - #129: Add support for [XEP-0156: Disovering Alternative XMPP Connection Methods](https://xmpp.org/extensions/xep-0156.html). Only XML is supported for now.
 - #1105: Preliminary support for storing persistent data in IndexedDB instead of localStorage
+- #1666: Allow scrolling of the OMEMO fingerprints list
 - #1691: Fix `collection.chatbox is undefined` errors
 - #1772: `_converse.api.contact.add(jid, nick)` fails, says not a function
+- #1792: Fix: modals don't have scrollbars
 - #1796: Don't show "back" arrow navigation (on mobile) in the chat header when in `singleton` mode
 
 ### Breaking changes

sass/_chatrooms.scss

@@ -1,6 +1,5 @@
 #conversejs.converse-embedded,
 #conversejs {
-
     .badge-room-color {
         background-color: var(--chatroom-head-color);
     }
@@ -12,17 +11,6 @@
         }
     }
 
-    #list-chatrooms-modal {
-        .modal-body {
-			max-height: 70vh;
-            padding-bottom: 0; // When the list of chats overflows, we don't want bottom padding
-        }
-        .available-chatrooms {
-            overflow: auto;
-            padding-bottom: 1rem;
-        }
-    }
-
     #room-details-modal {
         .features-list {
             margin-left: 1em;

sass/_core.scss

@@ -327,20 +327,6 @@ body.converse-fullscreen {
         content: close-quote;
     }
 
-    .modal {
-        background-color: rgba(0, 0, 0, 0.4);
-
-        .modal-body {
-            p {
-                padding: 0.25rem 0;
-            }
-        }
-
-        .modal-footer {
-            justify-content: flex-start;
-        }
-    }
-
     .helptext {
         font-size: var(--font-size-tiny);
         color: var(--text-color-lighten-15-percent);

sass/_modal.scss

@@ -1,7 +1,15 @@
 #conversejs {
+
     #converse-modals {
+        .modal {
+            background-color: rgba(0, 0, 0, 0.4);
             .modal-body {
+                overflow-y: auto;
+                max-height: 75vh;
                 margin-bottom: 2em;
+                p {
+                    padding: 0.25rem 0;
+                }
                 .confirm {
                     .form-group {
                         p:first-child {
@@ -11,6 +19,10 @@
                     }
                 }
             }
+            .modal-footer {
+                justify-content: flex-start;
+            }
+        }
 
         .scrollable-container {
             max-height: 50vh;