Skip to content

Do not scroll to child inside nested scroll view#54044

Closed
rozele wants to merge 1 commit intofacebook:mainfrom
rozele:export-D83804043
Closed

Do not scroll to child inside nested scroll view#54044
rozele wants to merge 1 commit intofacebook:mainfrom
rozele:export-D83804043

Conversation

@rozele
Copy link
Contributor

@rozele rozele commented Oct 3, 2025

Summary:
In rare cases, there may be a vertical ScrollView nested inside another vertical ScrollView. In these cases, when the requestChildFocus method is called, we may attempt to bring the entire focused child into view. However, we really only want to bring the entire nested ScrollView into view.

Changelog

[Android][Fixed] Focused scroll into view behaviors for nested vertical scroll views

Differential Revision: D83804043

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 3, 2025
@meta-codesync
Copy link

meta-codesync bot commented Oct 3, 2025

@rozele has exported this pull request. If you are a Meta employee, you can view the originating Diff in D83804043.

Summary:

In rare cases, there may be a vertical ScrollView nested inside another vertical ScrollView. In these cases, when the requestChildFocus method is called, we may attempt to bring the entire focused child into view. However, we really only want to bring the entire nested ScrollView into view.

## Changelog

[Android][Fixed] Focused scroll into view behaviors for nested vertical scroll views

Differential Revision: D83804043
@meta-codesync meta-codesync bot closed this in 26502c6 Oct 3, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Oct 3, 2025
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 26502c6.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @rozele in 26502c6

When will my fix make it into a release? | How to file a pick request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants