Skip to content

Allow overriding the horizontal scroll position restore - #54115

Closed
lenaic wants to merge 3 commits into
react:mainfrom
lenaic:export-D84310601
Closed

Allow overriding the horizontal scroll position restore#54115
lenaic wants to merge 3 commits into
react:mainfrom
lenaic:export-D84310601

Conversation

@lenaic

@lenaic lenaic commented Oct 10, 2025

Copy link
Copy Markdown

Summary:
Horizontal scroll view extensions should be able to customize the scrolling behavior for state restores. This adds the restoreScrollTo function that will be called to restore the scroll position to the location provided by the shadow node state.

Changelog: [Internal]

Differential Revision: D84310601

@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 10, 2025
@meta-codesync

meta-codesync Bot commented Oct 10, 2025

Copy link
Copy Markdown

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

Nick Lefever added 3 commits October 9, 2025 18:20
Summary:

This diff adds the possibility to control the use of `Trait::Hidden` on Android platforms through feature flags. The feature flag is Android specific and only accessed on Android platforms.

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D84289662
Summary:

With `Trait::hidden` unmounting and remounting views for shadow nodes holding scroll state, we have to restore the scroll position from the shadow node state when the scroll view is being remounted

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D84291899
Summary:

Horizontal scroll view extensions should be able to  customize the scrolling behavior for state restores. This adds the `restoreScrollTo` function that will be called to restore the scroll position to the location provided by the shadow node state.

Changelog: [Internal]

Differential Revision: D84310601
@meta-codesync meta-codesync Bot closed this in e926ad0 Oct 10, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Oct 10, 2025
@meta-codesync

meta-codesync Bot commented Oct 10, 2025

Copy link
Copy Markdown

This pull request has been merged in e926ad0.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by Nick Lefever in e926ad0

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