Skip to content

Conversation

@eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Nov 11, 2025

We explicitly defer unmounting the root until all removal operations have been processed. This requires flushing pending operations before we unmount the next root.

All the other unmount operations already rely on child-first ordering so I don't think we need to special case the root.

This change is required to batch operations when updating the filter where we unmount+mount all roots.

@meta-cla meta-cla bot added the CLA Signed label Nov 11, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Nov 11, 2025
@eps1lon eps1lon marked this pull request as ready for review November 11, 2025 10:16
@eps1lon eps1lon requested review from hoxyq and sebmarkbage November 11, 2025 10:16
Copy link
Contributor

@hoxyq hoxyq left a comment

Choose a reason for hiding this comment

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

If I read the code correctly, the root removal was still sent as regular removal? But the code for some reason was structured in a way that it is being handled differently from regular removals. Weird.

@eps1lon eps1lon merged commit 8c15edd into facebook:main Nov 11, 2025
247 checks passed
@eps1lon eps1lon deleted the sebbie/11-11-_devtools_send_root_unmount_as_a_regular_removal_operation branch November 11, 2025 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants