Skip to content

Fix Android CarouselView so you don't need to add a delay after switching ItemSources to Set Position #24499

Open

Description

Description

The following PR made the situation better, but I don't think it's still quite perfect. We merged it for now because it at least fixes the issue where setting the position completely didn't work.

#24431

Ideally this code should still work without the delay here

https://github.com/Tamilarasan-Paranthaman/maui/blob/8a1cbc3806a82ac1676da013761b397827fd66cd/src/Controls/tests/TestCases.HostApp/Issues/Issue17283.xaml.cs#L52-L57

I'm wondering if we should just skip the ScrollHelper code here if the _oldPosition is zero?

If the CurrentItem is null and the Position is zero it doesn't seem like the scroll code should run at all should it?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions