From a899b1b46d10d5ad33820ddc3dcce62cf47e527e Mon Sep 17 00:00:00 2001
From: Pavel Djundik <github@xpaw.me>
Date: Sat, 14 Jul 2018 10:27:45 +0300
Subject: [PATCH] Throttle scroll event instead of debouncing

---
 client/components/MessageList.vue | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/client/components/MessageList.vue b/client/components/MessageList.vue
index 6a941334..667eb9c9 100644
--- a/client/components/MessageList.vue
+++ b/client/components/MessageList.vue
@@ -60,7 +60,8 @@
 
 <script>
 require("intersection-observer");
-const debounce = require("lodash/debounce");
+
+import {debounce, throttle} from "lodash";
 
 const constants = require("../js/constants");
 const clipboard = require("../js/clipboard");
@@ -116,7 +117,7 @@ export default {
 	},
 	watch: {
 		"channel.id"() {
-			this.$set(this.link, "scrolledToBottom", true);
+			this.$set(this.channel, "scrolledToBottom", true);
 		},
 		"channel.messages"() {
 			this.keepScrollPosition();
@@ -139,8 +140,8 @@ export default {
 		});
 	},
 	mounted() {
-		this.debouncedResize = debounce(this.handleResize, 50);
-		this.debouncedScroll = debounce(this.handleScroll, 50);
+		this.debouncedResize = debounce(this.handleResize, 500);
+		this.debouncedScroll = throttle(this.handleScroll, 150);
 
 		window.addEventListener("resize", this.debouncedResize, {passive: true});
 		this.$refs.chat.addEventListener("scroll", this.debouncedScroll, {passive: true});
@@ -260,9 +261,7 @@ export default {
 			const el = this.$refs.chat;
 
 			if (el && this.channel.scrolledToBottom) {
-				this.$nextTick(() => {
-					el.scrollTop = el.scrollHeight;
-				});
+				el.scrollTop = el.scrollHeight;
 			}
 		},
 	},