fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248

Merged
srest2021 merged 31 commits into
sentry-v2from
srest2021/REPLAY-667
Sep 25, 2025
Merged

fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from
srest2021/REPLAY-667

Conversation

@srest2021

@srest2021srest2021 commented Sep 18, 2025•
edited
Loading

Copy link
Copy Markdown
Member

fixes REPLAY-667
fixes REPLAY-558

Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.

We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.

Before:

  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov

After:

  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov

@linear

linearBot commented Sep 18, 2025

Copy link
Copy Markdown

REPLAY-667 Fast forward behavior is not consistent

@srest2021srest2021 changed the title add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear

linearBot commented Sep 18, 2025

Copy link
Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline

srest2021
srest2021 commented Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts Outdated
@srest2021
srest2021 marked this pull request as ready for review September 19, 2025 19:32
@srest2021srest2021 mentioned this pull request Sep 19, 2025
Closed
@billyvg

billyvg commented Sep 23, 2025

Copy link
Copy Markdown
Member

@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.

@srest2021

srest2021 commented Sep 23, 2025

Copy link
Copy Markdown
MemberAuthor

@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏

billyvg
billyvg reviewed Sep 23, 2025
View reviewed changes

@billyvgbillyvg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now

Comment threadpackages/rrweb/src/replay/index.ts Outdated
Comment threadpackages/rrweb/src/replay/index.ts Outdated
Comment threadpackages/rrweb/src/replay/index.ts Outdated
billyvg
billyvg approved these changes Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts Outdated
Comment threadpackages/rrweb/src/replay/index.ts Outdated
@srest2021
srest2021 merged commit 6d3cb32 into sentry-v2Sep 25, 2025
17 checks passed
@srest2021
srest2021 deleted the srest2021/REPLAY-667 branch September 25, 2025 16:32
@srest2021srest2021 mentioned this pull request Sep 25, 2025
Merged
srest2021 added a commit that referenced this pull request Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021 mentioned this pull request Sep 26, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
fix(replay): handle fast forwarding for arbitrary position jumps in replayer - #248
Merged
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Sep 25, 2025
Merged
fix(replay): handle fast forwarding for arbitrary position jumps in replayer#248
srest2021 merged 31 commits into
sentry-v2from 
srest2021/REPLAY-667
Conversation
@srest2021
@srest2021srest2021
commented
Sep 18, 2025•

edited

Loading

Copy link


Copy Markdown

Member
fixes REPLAY-667

fixes REPLAY-558
Previously, fast forwarding through a replay was buggy and inconsistent if you scrub or jump positions. This is because when users scrub or jump to arbitrary timestamps, the cached nextUserInteractionEvent and state become stale and prevent proper re-evaluation of fast forwarding.
We now extend the Replayer class by adding a private function reevaluateFastForward that will handle arbitrary position jumps. reevaluateFastForward finds the current event index using binary search, scans forward to find the next user interaction position, and calculates the gap of inactivity between them. If the gap is large enough, fast forwarding is triggered. We call this function if we jump/scrub to a new position, if the skipInactive config changes, and on starting playback.
Before:
  • Large period of inactivity at start never fast fowards
  • Clicking to a breadcrumb triggers fastforwarding, but scrubbing or jumping to that same area doesn't
  • Scrubbing back while fastforwarding within the same period of inactivity loses the fastforwarding but the badge stays up
Screen.Recording.2025-09-23.at.9.20.44.AM.mov
After:
  • Correctly fast forward at the start
  • Scrubbing or jumping to any position triggers the correct fastforwarding behavior, and the badge is consistent
Screen.Recording.2025-09-23.at.9.27.18.AM.mov
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-667 Fast forward behavior is not consistent
@srest2021srest2021
changed the title
add refreshSkipStatefix(replay): handle fast forwarding for arbitrary position jumps in replayerSep 18, 2025
@linear
linearBot
commented
Sep 18, 2025

Copy link


Copy Markdown

REPLAY-558 [Bug] "Fast forwarding through inactivity" popover on Replay Details is sticky when scrubbing timeline
srest2021
srest2021
commented
Sep 18, 2025
View reviewed changes
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
marked this pull request as ready for review
September 19, 2025 19:32
@srest2021srest2021
mentioned this pull request
Sep 19, 2025
 Closed

@billyvg
billyvg
commented
Sep 23, 2025

Copy link


Copy Markdown

Member
@srest2021 Is it possible to handle this all inside of rrweb rather than requiring changes in sentry ui? The reason I ask is that if this is a bug within rrweb, it'd be nice to upstream a fix.
@srest2021
srest2021
commented
Sep 23, 2025

Copy link


Copy Markdown

MemberAuthor
@billyvg Ok should be all integrated now! There was also one small bug in the frontend fixed here, I'll request you there as well. I'm thinking merge the frontend PR first then this one? Let me know what you think 🙏
billyvg
billyvg
reviewed
Sep 23, 2025
View reviewed changes
@billyvgbillyvg
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to take a closer look tomorrow, but here are a few nits for now
Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

billyvg
billyvg
approved these changes
Sep 24, 2025
View reviewed changes
Comment threadpackages/rrweb/test/replayer-reevaluate-fast-forward.test.ts
Outdated

Comment threadpackages/rrweb/src/replay/index.ts
Outdated

@srest2021
srest2021
merged commit 6d3cb32
into
sentry-v2Sep 25, 2025

17 checks passed

@srest2021
srest2021
deleted the

srest2021/REPLAY-667

branch
September 25, 2025 16:32
@srest2021srest2021
mentioned this pull request
Sep 25, 2025
 Merged

srest2021 added a commit
that referenced
this pull request
Sep 25, 2025
@srest2021
meta(changelog): Prepare release 2.38.0 (#249)
f856de1
Update changelog for commit:
fix(replay): handle fast forwarding for arbitrary position jumps in
replayer (#248)
@srest2021srest2021
mentioned this pull request
Sep 26, 2025
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@billyvgbillyvgbillyvg approved these changes
@chargomechargomeAwaiting requested review from chargome
@mydeamydeaAwaiting requested review from mydea

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@srest2021@billyvg