Merge pull request #3 from amplitude/skip-inactive-periods

feat: skip through inactive periods instead of fast forward
amplitude · Oct 18, 2023 · 605a78d · 605a78d
2 parents 7e43b85 + dbc22b9
commit 605a78d
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/packages/rrweb/src/replay/index.ts b/packages/rrweb/src/replay/index.ts
@@ -679,10 +679,6 @@ export class Replayer {
             // do not check skip in sync
             return;
           }
-          if (event === this.nextUserInteractionEvent) {
-            this.nextUserInteractionEvent = null;
-            this.backToNormal();
-          }
           if (this.config.skipInactive && !this.nextUserInteractionEvent) {
             for (const _event of this.service.state.context.events) {
               if (_event.timestamp <= event.timestamp) {
@@ -704,14 +700,8 @@ export class Replayer {
               const skipTime =
                 // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                 this.nextUserInteractionEvent.delay! - event.delay!;
-              const payload = {
-                speed: Math.min(
-                  Math.round(skipTime / SKIP_TIME_INTERVAL),
-                  this.config.maxSpeed,
-                ),
-              };
-              this.speedService.send({ type: 'FAST_FORWARD', payload });
-              this.emitter.emit(ReplayerEvents.SkipStart, payload);
+              this.play(this.getCurrentTime() + skipTime)
+              this.nextUserInteractionEvent = null;
             }
           }
         };