Skip to content

fix: macOS ScrollView resize and content inset behavior#2732

Open
Saadnajmi wants to merge 2 commits intomicrosoft:mainfrom
Saadnajmi:live-resize
Open

fix: macOS ScrollView resize and content inset behavior#2732
Saadnajmi wants to merge 2 commits intomicrosoft:mainfrom
Saadnajmi:live-resize

Conversation

@Saadnajmi
Copy link
Collaborator

@Saadnajmi Saadnajmi commented Oct 15, 2025

Summary

  • preserve macOS ScrollView position during live resize in root and surface hosting views
  • align Fabric ScrollView contentInset and contentOffset behavior with the existing macOS scrollview implementation
  • prevent resize-driven scroll drift in the macOS Paper ScrollView path

Test Plan

  • Build RNTester-macOS
  • Verify RNTester runs normally
  • Manually resize windows containing ScrollView content and confirm content offset remains stable

@Saadnajmi Saadnajmi requested a review from a team as a code owner October 15, 2025 22:11
@Saadnajmi Saadnajmi changed the title chore: pick a change around viewDidEndLiveResize chore: pick a bunch of scrollview changes Oct 15, 2025
@changeset-bot
Copy link

changeset-bot bot commented Mar 26, 2026

⚠️ No Changeset found

Latest commit: f11265d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Saadnajmi Saadnajmi changed the title chore: pick a bunch of scrollview changes Fix macOS ScrollView resize and content inset behavior Mar 26, 2026
@Saadnajmi Saadnajmi changed the title Fix macOS ScrollView resize and content inset behavior fix: macOS ScrollView resize and content inset behavior Mar 26, 2026
@Saadnajmi Saadnajmi force-pushed the live-resize branch 2 times, most recently from 04db248 to e7202bc Compare March 26, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants