From 03a56f239267b0c6c18d42656cd5a7abcf17a22b Mon Sep 17 00:00:00 2001 From: Alf Eaton Date: Mon, 4 Jul 2022 15:42:56 +0100 Subject: [PATCH] Merge pull request #8708 from overleaf/ae-loading-threads-fix Only dispatch the loaded_threads event once GitOrigin-RevId: d9b8bf4a84d115fcc00edbf1901565ddb407c8be --- .../js/ide/review-panel/controllers/ReviewPanelController.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js b/services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js index 74383832c1..beac41cc47 100644 --- a/services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js +++ b/services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js @@ -491,7 +491,7 @@ export default App.controller( if (!window.isRestrictedTokenMember) { if (rangesTracker.comments.length > 0) { ensureThreadsAreLoaded() - } else { + } else if (ide.$scope.loadingThreads === true) { // ensure that tracked changes are highlighted even if no comments are loaded ide.$scope.loadingThreads = false dispatchReviewPanelEvent('loaded_threads')