[iOS] ScrollView content offset shifts unexpectedly when FlowDirection is changed #30053
+3
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kubaflo
added
area-controls-scrollview
Contributor
|
Hey there @@kubaflo! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an issue where switching the layout direction on iOS could leave stale scroll offsets, causing misaligned content in a ScrollView. It explicitly resets ContentOffset to (0,0) before rearranging and then reapplies the correct horizontal offset for RTL layouts.
- Reset
ContentOffsetto zero before re-arranging content in RTL - Compute and apply the correct horizontal offset after layout
Comments suppressed due to low confidence (1)
src/Core/src/Platform/iOS/MauiScrollView.cs:64
- Consider adding a UI or unit test in
TestCases.Shared.TestsandTestCases.HostAppto verify thatContentOffsetis reset when switching the FlowDirection, ensuring this behavior is covered by automated tests.
ContentOffset = new CGPoint(0, 0);
Member
|
@KishoreJey would you be able to test the resulting NuGets from this PR and see if that resolves it for you? See the instructions all the way at the top |
Member
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
jfversluis
previously approved these changes
Aug 7, 2025
Member
|
/rebase |
github-actions
bot
force-pushed
the
ScrollView-content-offset-shifts-unexpectedly-when-FlowDirection-is-set-to-RightToLeft-iOS/MacCatalyst]
branch
from
e61e9e8 to
45c06ed
Compare
PureWeen
requested changes
Aug 7, 2025
ghost
mentioned this pull request
Member
|
/rebase |
github-actions
bot
force-pushed
the
ScrollView-content-offset-shifts-unexpectedly-when-FlowDirection-is-set-to-RightToLeft-iOS/MacCatalyst]
branch
from
45c06ed to
e373acb
Compare
Member
|
/azp run MAUI-public |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note
Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!
Description of Change
This PR addresses an issue where switching the layout direction from Right-to-Left (RTL) to Left-to-Right (LTR) could result in incorrect content positioning due to stale or inconsistent scroll offsets.
To ensure layout consistency, we now explicitly reset the ContentOffset to (0,0) before performing layout adjustments. This prevents any residual offset from affecting the new layout direction.
Issues Fixed
This PR fixes a problem addressed here
#29458 (comment)
Fixes #31322
Screen.Recording.2025-06-18.at.16.34.17.mov
Screen.Recording.2025-06-18.at.16.33.26.mov