fix: solve the smooth scrolling scene, the y value obtained is the da…

…ta 100ms ago
xujiujiu · Apr 7, 2024 · 67c8d38 · 67c8d38
1 parent 39f3e4d
commit 67c8d38
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/packages/rrweb/src/record/observer.ts b/packages/rrweb/src/record/observer.ts
@@ -374,10 +374,7 @@ export function initScrollObserver({
           });
         }
       }),
-      sampling.scroll || 100,
-      {
-        trailing: true
-      }
+      sampling.scroll || 100
     ),
   );
   return on('scroll', updatePosition, doc);

diff --git a/packages/rrweb/src/utils.ts b/packages/rrweb/src/utils.ts
@@ -88,10 +88,19 @@ export function throttle<T>(
       previous = now;
       func.apply(context, args);
     } else if (!timeout && options.trailing !== false) {
+      const argumentsList = args.map((arg: Omit<T, 'target'>) => {
+        return {
+          ...arg,
+          target: arg.target || null
+        }
+      })
       timeout = setTimeout(() => {
         previous = options.leading === false ? 0 : Date.now();
-        timeout = null;
-        func.apply(context, args);
+        if (timeout) {
+          clearTimeout(timeout)
+          timeout = null;
+        }
+        func.apply(context, argumentsList);
       }, remaining);
     }
   };