From b1ab4dd667bbf6e05a8a05be88c99c24a1a65eeb Mon Sep 17 00:00:00 2001
From: friarzen <friarzen@gmail.com>
Date: Mon, 19 Mar 2018 00:59:28 +0000
Subject: [PATCH] Fix append scrolling -- needs more testing

---
 .../web/webclient/static/webclient/js/webclient_gui.js | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/evennia/web/webclient/static/webclient/js/webclient_gui.js b/evennia/web/webclient/static/webclient/js/webclient_gui.js
index d07239039f..4189e026fb 100644
--- a/evennia/web/webclient/static/webclient/js/webclient_gui.js
+++ b/evennia/web/webclient/static/webclient/js/webclient_gui.js
@@ -257,14 +257,15 @@ function onText(args, kwargs) {
             if( SplitHandler ) {
                 for ( var key in SplitHandler.split_panes) {
                     var pane = SplitHandler.split_panes[key];
-		    console.log(pane);
                     // is this message type mapped to this pane?
                     if ( (pane['types'].length > 0) && pane['types'].includes(msgtype) ) {
                         // yes, so append/replace this pane's inner div with this message
                         if ( pane['update_method'] == 'replace' ) {
                             $('#'+key).html(args[0])
                         } else {
-                            $('#'+key).append(args[0]).animate({ scrollTop: document.getElementById("#"+key).scrollHeight }, 0);
+                            $('#'+key).append(args[0]);
+                            var scrollHeight = $('#'+key).parent().prop("scrollHeight");
+                            $('#'+key).parent().animate({ scrollTop: scrollHeight }, 0);
                         }
                         // record sending this message to a pane, no need to update the default div
                         use_default_pane = false;
@@ -279,9 +280,8 @@ function onText(args, kwargs) {
         var mwin = $("#messagewindow");
         var cls = kwargs == null ? 'out' : kwargs['cls'];
         mwin.append("<div class='" + cls + "'>" + args[0] + "</div>");
-        mwin.animate({
-            scrollTop: document.getElementById("messagewindow").scrollHeight
-        }, 0);
+        var scrollHeight = mwin.parent().parent().prop("scrollHeight");
+        mwin.parent().parent().animate({ scrollTop: scrollHeight }, 0);
 
         onNewLine(args[0], null);
     }