Skip to content

[DevTools] Recursively compute the bounding rect of the roots - #34629

Merged
sebmarkbage merged 1 commit into
react:mainfrom
sebmarkbage:rootscroll
Sep 28, 2025
Merged

[DevTools] Recursively compute the bounding rect of the roots#34629
sebmarkbage merged 1 commit into
react:mainfrom
sebmarkbage:rootscroll

Conversation

@sebmarkbage

Copy link
Copy Markdown
Contributor

It's possible for the children to overflow the bounding rect of the root in general when they overflow in the DOM. However even when it doesn't overflow in the DOM, the bounding rect of the root can shrink while the content is suspended. In fact, it's very likely.

Originally I thought we didn't need to consider this recursively because document scrolling takes absolute positioned content into account but because we're using nested overflow scrolling, we have to manually compute this.

@github-actionsgithub-actionsBot added the React Core Team Opened by a member of the React Core Team label Sep 28, 2025
@sebmarkbage
sebmarkbage merged commit f78b234 into react:mainSep 28, 2025
247 checks passed
EugeneChoi4 pushed a commit to EugeneChoi4/react that referenced this pull request Sep 29, 2025
…34629)
It's possible for the children to overflow the bounding rect of the root
in general when they overflow in the DOM. However even when it doesn't
overflow in the DOM, the bounding rect of the root can shrink while the
content is suspended. In fact, it's very likely.
Originally I thought we didn't need to consider this recursively because
document scrolling takes absolute positioned content into account but
because we're using nested overflow scrolling, we have to manually
compute this.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedReact Core TeamOpened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@sebmarkbage@eps1lon