Bladeren bron

Merge pull request #8708 from overleaf/ae-loading-threads-fix

Only dispatch the loaded_threads event once

GitOrigin-RevId: d9b8bf4a84d115fcc00edbf1901565ddb407c8be
Alf Eaton 4 jaren geleden
bovenliggende
commit
0b942bd8a8
1 gewijzigde bestanden met toevoegingen van 1 en 1 verwijderingen
  1. 1 1
      services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js

+ 1 - 1
services/web/frontend/js/ide/review-panel/controllers/ReviewPanelController.js

@@ -491,7 +491,7 @@ export default App.controller(
       if (!window.isRestrictedTokenMember) {
       if (!window.isRestrictedTokenMember) {
         if (rangesTracker.comments.length > 0) {
         if (rangesTracker.comments.length > 0) {
           ensureThreadsAreLoaded()
           ensureThreadsAreLoaded()
-        } else {
+        } else if (ide.$scope.loadingThreads === true) {
           // ensure that tracked changes are highlighted even if no comments are loaded
           // ensure that tracked changes are highlighted even if no comments are loaded
           ide.$scope.loadingThreads = false
           ide.$scope.loadingThreads = false
           dispatchReviewPanelEvent('loaded_threads')
           dispatchReviewPanelEvent('loaded_threads')