Skip to content

fix(replay): Fix replayStepper perf issues #92167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented May 22, 2025

This PR fixes an perf regression from #83016 -- we were completely removing these style elements from the tree which causes issues when rebuilding the tree and ultimately we were hitting this timeout too frequently, causing the browser to lockup.

Closes #91331

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 22, 2025
@billyvg billyvg marked this pull request as ready for review May 22, 2025 22:11
@billyvg billyvg requested a review from a team as a code owner May 22, 2025 22:11
@billyvg billyvg merged commit d6e07c6 into master May 22, 2025
43 checks passed
@billyvg billyvg deleted the billy/replay-216-high-page-load-times-and-tab-crashes-when-loading-long branch May 22, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

High page load times and tab crashes when loading long Sentry Replays
2 participants