-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[iOS] Fix for CarouselView ItemSpacing property in ItemsLayout not working with CV2 #27056
base: main
Are you sure you want to change the base?
Conversation
@dotnet-policy-service agree company="Syncfusion, Inc." |
Hi @mattleibow, I have fixed the ItemSpacing issue in CarouselViewHandler2.iOS, which resolves the issues on both iOS and macOS. While testing to enable the test case in MacCatalyst, maximizing the window resulted in different behaviors in CV1 and CV2. As a result, I have not enabled the MacCatalyst condition yet. Could you please confirm the correct behavior on macOS—CV1 or CV2? Issue link : #26969 |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
2c8d2d7
to
b3c0c0d
Compare
/rebase |
b3c0c0d
to
f2dba16
Compare
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/rebase |
0ad7670
to
eb90334
Compare
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Root Cause of the issue
The itemSpacing variable is initialized to 0. As a result, itemSpacing remains 0, and no spacing is applied between the items.
Description of Change
To resolve this, apply the correct item spacing from the LinearItemsLayout.
Issues Fixed
Fixes #27025
Tested the behaviour in the following platforms
Screenshot