[Android] Fixed the CarouselView Items overlap issue with PeekAreaInsets

[Ahamed-Ali]

Issue

• When CarouselView is placed inside a VerticalStackLayout with PeekAreaInsets, the items overlap each other. This issue occurs regardless of whether Loop is set to true or false.

Root Cause of the issue

When VerticalStackLayout is used, the OnMeasure method of MauiCarouselRecyclerView is called.

• In this method, heightMeasureSpec is 0, while widthMeasureSpec has a value.

• Since the measurement falls into the "Unspecified" condition, both widthMeasureSpec and heightMeasureSpec are recalculated.

• During this process, when viewHolder.ItemView.Measure is called, SizedItemContentView.OnMeasure is triggered.

• Inside this method, targetWidth is retrieved from the GetItemWidth method in CarouselViewHandler.Android, which subtracts PeekAreaInsets and ItemSpacing.

• Due to subsequent calls to OnMeasure in MauiRecyclerView, the MeasuredWidth keeps changing because of incorrect calculations in the "Unspecified" condition.

• As a result, the recalculated MeasuredWidth in GetItemWidth becomes too small, causing UI overlap.

Description of Change

• I fixed this by checking if WidthMeasureSpec and HeightMeasureSpec are 0. If they are, then calculated the new measurements using MeasuredWidth and MeasuredHeight. Otherwise, use the already calculated WidthMeasureSpec and HeightMeasureSpec, ensuring proper measurement calculations.

Regressed PR

#25924

Issues Fixed

Fixes #27418

Tested the behaviour in the following platforms

• Android
• Windows
• iOS
• Mac

Screenshot

Before Issue Fix	After Issue Fix
CarouselItemIssue.mov	CarouselViewItemsFix.mov

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

• src/Controls/tests/TestCases.HostApp/Issues/Issue27418.xaml: Language not supported

[PureWeen]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[jsuarezruiz]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[rmarinho]

/rebase

[rmarinho]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[PureWeen]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[Copilot]

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• src/Controls/tests/TestCases.HostApp/Issues/Issue27418.xaml: Language not supported

Comments suppressed due to low confidence (1)

src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue27418.cs:1

• Ensure that the test case CarouselItemsShouldRenderProperly is executed on all relevant platforms, and the issue with Catalyst is properly documented and tracked.

#if TEST_FAILS_ON_CATALYST

[PureWeen]

• failing tests are unrelated