fix: race condition where `navigator.update` was called when it should not be

2 years ago · fa643eb808
parent 9f9a835fcb
commit fa643eb808
1 changed files with 3 additions and 3 deletions
--- a/public/src/modules/navigator.js
+++ b/public/src/modules/navigator.js
@ -581,9 +581,9 @@ define('navigator', ['forum/pagination', 'components', 'hooks', 'alerts'], funct
 		function animateScroll() {
 			function reenableScroll() {
 				// Re-enable onScroll behaviour
-				$(window).on('scroll', navigator.delayedUpdate);
-				const scrollToRect = scrollTo.get(0).getBoundingClientRect();
-				navigator.update(scrollToRect.top);
+				setTimeout(() => { // fixes race condition from jQuery — onAnimateComplete called too quickly
+					$(window).on('scroll', navigator.delayedUpdate);
+				}, 50);
 			}
 			function onAnimateComplete() {
 				if (done) {