feat(viewer): refactor EventList to react-window and optimize rendering #4
+963
−163
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Replace custom virtualization with react-window VariableSizeList and normalize import for Vite interop - Implement MeasuredRow using ResizeObserver + requestAnimationFrame; de-duplicate size reports and batch measurements - Wire row height changes to list.resetAfterIndex to keep virtualization accurate - Memoize EventRow, EventBody, and CodeBlock; avoid repeated JSON.stringify to reduce GC/CPU - Replace static SAMPLE_TRACE with a realistic, variable-sized generator (small or very large traces) - Add benchmark script (viewer/scripts/benchEventList.mjs) to measure flattening/positioning performance - Minor TS/type adjustments and small helper cleanup Improves performance and responsiveness for very large traces; reduces layout thrash and render overhead.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Improves performance and responsiveness for very large traces; reduces layout thrash and render overhead.