From 7562886cef311bf3713f044fb15ddf4178bbd1da Mon Sep 17 00:00:00 2001 From: ryanreh99 Date: Sun, 29 Nov 2020 03:07:47 +0530 Subject: [PATCH] ui: Hide loading indicators for non-existant narrows. We were still displaying the loading spinner even after displaying the error text, which was confusing as we do not try to fetch again. This fixes it. --- static/js/message_fetch.js | 1 + 1 file changed, 1 insertion(+) diff --git a/static/js/message_fetch.js b/static/js/message_fetch.js index d0b089c24d..bc1951a0e4 100644 --- a/static/js/message_fetch.js +++ b/static/js/message_fetch.js @@ -248,6 +248,7 @@ exports.load_messages = function (opts) { // retry or display a connection error. // // FIXME: Warn the user when this has happened? + message_scroll.hide_indicators(); const data = { messages: [], };