MDL-69883 message: fix lazy loading of conversations.

Set max-height on each container section rather than the container itself, so that each can respond to the scrollBottom event within the lazy loading module (which handles the overflow itself).
moodle · Feb 24, 2021 · 7aee6e2 · 7aee6e2
1 parent af00404
commit 7aee6e2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/theme/boost/scss/moodle/message.scss b/theme/boost/scss/moodle/message.scss
@@ -659,8 +659,9 @@ $message-day-color: color-yiq($message-app-bg) !default;
         height: 100%;
         margin-top: 0;
         .conversationcontainer {
-            max-height: calc(100vh - 50px);
-            overflow: auto;
+            .section {
+                max-height: calc(100vh - 50px);
+            }
         }
         div[role="main"] {
             height: 100%;

diff --git a/theme/boost/style/moodle.css b/theme/boost/style/moodle.css
@@ -15468,9 +15468,8 @@ a.ygtvspacer:hover {
 #page-message-index #region-main {
   height: 100%;
   margin-top: 0; }
-  #page-message-index #region-main .conversationcontainer {
-    max-height: calc(100vh - 50px);
-    overflow: auto; }
+  #page-message-index #region-main .conversationcontainer .section {
+    max-height: calc(100vh - 50px); }
   #page-message-index #region-main div[role="main"] {
     height: 100%; }
     #page-message-index #region-main div[role="main"] #maincontent {

diff --git a/theme/classic/style/moodle.css b/theme/classic/style/moodle.css
@@ -15684,9 +15684,8 @@ a.ygtvspacer:hover {
 #page-message-index #region-main {
   height: 100%;
   margin-top: 0; }
-  #page-message-index #region-main .conversationcontainer {
-    max-height: calc(100vh - 50px);
-    overflow: auto; }
+  #page-message-index #region-main .conversationcontainer .section {
+    max-height: calc(100vh - 50px); }
   #page-message-index #region-main div[role="main"] {
     height: 100%; }
     #page-message-index #region-main div[role="main"] #maincontent {