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

Problem initializing the CurrentItem for the Carousel #18879

Open
mypp74 opened this issue Nov 19, 2023 · 5 comments · May be fixed by #22279
Open

Problem initializing the CurrentItem for the Carousel #18879

mypp74 opened this issue Nov 19, 2023 · 5 comments · May be fixed by #22279
Assignees
@rmarinho
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone
.NET 8 SR6

Comments

@mypp74
Copy link

mypp74 commented Nov 19, 2023

Description

`
public partial class SecondPage : ContentPage, IQueryAttributable
{
public SecondPage()
{
InitializeComponent();
BindingContext = new ViewModel();
}

public void ApplyQueryAttributes(IDictionary<string, object> query)
{
    if (BindingContext is ViewModel model)
    {
        model.Items = [
            "1",
            "2",
            "3"
        ];

        // Android : works as expected
        // iOS: "2" is ignored. 
        model.CurrentItem = "2";
    }
}

}
Carousel_CurrentItem.zip

`

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

8.0.3

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iPhone 15 iOS 17.0

Did you find any workaround?

No response

Relevant log output

No response
@mypp74 mypp74 added the t/bug Something isn't working label Nov 19, 2023
@jsuarezruiz jsuarezruiz added area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/iOS 🍎 labels Nov 20, 2023
@jsuarezruiz jsuarezruiz added this to the Backlog milestone Nov 20, 2023
@ghost
Copy link

ghost commented Nov 20, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@XamlTest
Copy link
Collaborator

XamlTest commented Dec 20, 2023 •
edited

Verified this on Visual Studio Enterprise 17.9.0 Preview 2(8.0.3). Repro on iOS 17.0 and MacCatalyst, not repro on Windows 11 and Android 14.0-API34 with below Project:
MauiApp2.zip

@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Dec 20, 2023
@RobTF
Copy link

RobTF commented Feb 6, 2024

+1 encountered this too - another Carousel bug. This control is clearly alpha quality.

@TheresasAccount
Copy link

+1, this makes the carousel near unusable on iOS as I was controlling the visible elements by setting CurrentItem

@TheresasAccount
Copy link

Potentially related #17283

@rmarinho rmarinho self-assigned this May 7, 2024
@rmarinho rmarinho linked a pull request May 7, 2024 that will close this issue
Open
@rmarinho rmarinho modified the milestones: Backlog, .NET 8 SR6 May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmarinho rmarinho

Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
MAUI SDK Ongoing
Status: In Progress
Milestone
.NET 8 SR6
Development

Successfully merging a pull request may close this issue.

[iOS] Fix setting the CurrentItem on CarouselView load dotnet/maui
6 participants
@rmarinho @RobTF @jsuarezruiz @TheresasAccount @XamlTest @mypp74