-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fixed the vertical orientation issue in the CarouselViewHandler2 on iOS #27273
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hey there @Ahamed-Ali! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- src/Controls/tests/TestCases.HostApp/Issues/Issue27241.xaml: Language not supported
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/rebase |
3a80af3
to
8e78f45
Compare
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/rebase |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
8e78f45
to
db1fd31
Compare
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
|
Root Cause of the issue
OrthogonalScrollingBehavior
of the NsCollectionLayoutSection was previously set toGroupPagingCentered
for both horizontal and vertical orientations. This behavior is designed for horizontal scrolling, which is why it only worked correctly for horizontal orientation.Description of Change
UICollectionLayoutSectionOrthogonalScrollingBehavior.None
. This disables orthogonal scrolling, which is correct for vertical scrolling because we don't want horizontal paging in this case.Issues Fixed
Fixes #27241
Tested the behaviour in the following platforms
Screenshot
CarouselViewIssue.mov
CarouselViewFixed.mov