Skip to content

Enable scroll state restore for Trait::hidden - #54113

Closed
lenaic wants to merge 2 commits into
react:mainfrom
lenaic:export-D84291899
Closed

Enable scroll state restore for Trait::hidden#54113
lenaic wants to merge 2 commits into
react:mainfrom
lenaic:export-D84291899

Conversation

@lenaic

@lenaic lenaic commented Oct 9, 2025

Copy link
Copy Markdown

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]

Differential Revision: D84291899

Nick Lefever added 2 commits October 9, 2025 14:08
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]

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]

Differential Revision: D84291899
@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 9, 2025
@meta-codesync

meta-codesync Bot commented Oct 9, 2025

Copy link
Copy Markdown

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

lenaic pushed a commit to lenaic/react-native-macos that referenced this pull request Oct 10, 2025
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
@meta-codesync meta-codesync Bot closed this in 78bdf58 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 78bdf58.

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.

2 participants