Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Fizz] improve Hoistable handling for Elements and Resources inside Suspense Boundaries #28069

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Jan 30, 2024
Merged

[Fizz] improve Hoistable handling for Elements and Resources inside Suspense Boundaries #28069

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e443eb1
Refactor change to convey boundary state is not specific to resources
gnoff Nov 27, 2023
852f521
Uses a simpler approach to trackign hoistables and only writes them e…
gnoff Jan 23, 2024
eed492e
Updates the approach to emit hoistables eagerly but only if the hoist…
gnoff Jan 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Uses a simpler approach to trackign hoistables and only writes them e… 
…ither during the preamble or when a boundary complets. The cost is that during the preamble flush there is an extra traversal of the segments that will write in the preamble. While this is not ideal for perf it does make the tracking significantly easier to follow and overall probalby strikes a better balance of practicality and maintainability over raw performance.
  • Loading branch information
@gnoff
gnoff committed Jan 24, 2024
commit 852f521897f70a5277deb6b32e81daa2efea0833
Loading