Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[iOS] Fix CarouselViewLoopNoFreeze so that it's passing on both CV1 and CV2 sets of handlers #25775

Open
PureWeen opened this issue Nov 11, 2024 · 2 comments · May be fixed by #26053
Open

[iOS] Fix CarouselViewLoopNoFreeze so that it's passing on both CV1 and CV2 sets of handlers #25775

PureWeen opened this issue Nov 11, 2024 · 2 comments · May be fixed by #26053
Assignees
@SuthiYuvaraj
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView partner/macios Issues for the Mac / iOS SDK s/triaged Issue has been reviewed
Milestone
.NET 9 Servicing

Comments

@PureWeen
Copy link
Member

Description

This test was modified to only run on CV2 because it's not passing on CV1
https://github.com/dotnet/maui/pull/25719/files

We need to fix it so the test is passing on both sets of handlers
@PureWeen PureWeen added area-controls-collectionview CollectionView, CarouselView, IndicatorView partner/macios Issues for the Mac / iOS SDK labels Nov 11, 2024
@PureWeen PureWeen added this to the .NET 9 Servicing milestone Nov 11, 2024
@similar-issues-ai Similar Issues AI
Copy link

We've found some similar issues:

If any of the above are duplicates, please consider closing this issue out and adding additional context in the original issue.

Note: You can give me feedback by 👍 or 👎 this comment.

@dotnet-policy-service dotnet-policy-service bot added the s/triaged Issue has been reviewed label Nov 11, 2024
This was referenced Nov 11, 2024
Open
Open
Open
@NanthiniMahalingam
Copy link
Contributor

Issue Details

When removing the last item in the carosuel view, zeroth index item updated instead of previous items of the last index.

Steps to Reproduce

  • Go to this file CarouselViewLoopNoFreeze test case and Replace CarouselView2 with CarouselView1..
  • Run this test case
  • When click on Remove last button, last item removed. First item updated instead of updating the current last item.

Output

Expected Behavior

ViewHandler2.mp4

Actual Behavior

ViewHanlder1.mp4

@NanthiniMahalingam NanthiniMahalingam linked a pull request Nov 22, 2024 that will close this issue
Draft
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SuthiYuvaraj SuthiYuvaraj

Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView partner/macios Issues for the Mac / iOS SDK s/triaged Issue has been reviewed
Projects
None yet
Milestone
.NET 9 Servicing
3 participants
@PureWeen @SuthiYuvaraj @NanthiniMahalingam