Skip to content

[iOS26] Fix CarouselView scrolling to wrong item when navigating to last item#34013

Merged
kubaflo merged 11 commits into
dotnet:inflight/currentfrom
Vignesh-SF3580:fix-33770
Apr 7, 2026
Merged

[iOS26] Fix CarouselView scrolling to wrong item when navigating to last item#34013
kubaflo merged 11 commits into
dotnet:inflight/currentfrom
Vignesh-SF3580:fix-33770

Conversation

@Vignesh-SF3580
Copy link
Copy Markdown
Contributor

@Vignesh-SF3580 Vignesh-SF3580 commented Feb 12, 2026

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Issue Detail

CarouselView fails to scroll to the last item on iOS 26 when using programmatic navigation. Clicking "Go to LastItem" button navigates to the next item instead of the last item. The issue is iOS 26-specific and does not occur on iOS 18. The issue does not reproduce when breakpoints are used.

Root Cause

iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire intermediate scroll position callbacks during the animation, whereas iOS 18 only fired callbacks at the start and end. These intermediate callbacks trigger MapPosition in the handler, which calls UpdateFromPosition() again with an incorrect intermediate position value, interrupting the original scroll to the target position.

Description of Change

Added iOS 26-specific fix in UpdateFromPosition() method that introduces a 100ms delay using Task.Delay().ContinueWith() and MainThread.BeginInvokeOnMainThread() pattern. This debounces rapid position callbacks, allowing the scroll animation to complete before processing the position update. The fix only applies to iOS 26+ and does not affect other platforms or iOS versions.

Why Tests were not added

Regarding test case, existing test cases (CarouselViewShouldScrollToRightPosition and CarouselViewiOSCrashPreventionTest) already cover this scenario, so no new tests were added in this PR.

Tested the behavior in the following platforms

  • Android
  • Windows
  • iOS
  • Mac

Reference:
https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

Issues Fixed

Fixes #33770

Screenshots

Before Issue Fix After Issue Fix
Test17283Before.mov
Test17283After.mov
Before Issue Fix After Issue Fix
Test16020Before.mov
16020TestAfter.mov

@dotnet-policy-service dotnet-policy-service Bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Feb 12, 2026
@Vignesh-SF3580 Vignesh-SF3580 added the community ✨ Community Contribution label Feb 12, 2026
@sheiksyedm sheiksyedm marked this pull request as ready for review February 18, 2026 10:54
Copilot AI review requested due to automatic review settings February 18, 2026 10:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes a bug in CarouselView on iOS 26 where programmatic navigation to the last item would incorrectly scroll to the next item instead. The issue stems from iOS 26's new behavior of firing intermediate scroll position callbacks during UICollectionView animation, which was not present in iOS 18.

Changes:

  • Added iOS 26-specific workaround in UpdateFromPosition() using a 200ms delay to debounce rapid position callbacks
  • Added iOS 26 snapshot test baseline image for CarouselViewShouldScrollToRightPosition test

Reviewed changes

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

File Description
src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs Modified UpdateFromPosition() to async void, added iOS 26 version check with 200ms Task.Delay to debounce scroll position updates
src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png New snapshot baseline for iOS 26 test verification

Comment thread src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs Outdated
Comment thread src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs Outdated
Comment thread src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs Outdated
@rmarinho rmarinho added s/agent-changes-requested AI agent recommends changes - found a better alternative or issues s/agent-fix-lose Author adopted the agent's fix and it turned out to be bad s/agent-reviewed PR was reviewed by AI agent workflow (full 4-phase review) labels Feb 18, 2026
@kubaflo kubaflo added s/agent-fix-pr-picked AI could not beat the PR fix - PR is the best among all candidates and removed s/agent-fix-lose Author adopted the agent's fix and it turned out to be bad labels Feb 20, 2026
@Vignesh-SF3580
Copy link
Copy Markdown
Contributor Author

🤖 AI Summary

📊 Expand Full Review
🔍 Pre-Flight — Context & Validation
📝 Review Sessionadded ios26 image. · 9fe943f
Issue: #33770 - [iOS 26] CarouselView does not scroll to the correct last item Platforms Affected: iOS 26 only (confirmed working on iOS 18) Files Changed: 1 implementation file, 1 snapshot image

Issue Summary

CarouselView fails to scroll to the last item on iOS 26 when using programmatic navigation (Position or CurrentItem). Clicking "Go to LastItem" navigates to the next item instead of the last item. Confirmed iOS 26- does not reproduce on iOS 18. Validated by Syncfusion (partner) contributor.specific

Root Cause (from PR description)

iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire intermediate scroll position callbacks during animation, whereas iOS 18 only fired callbacks at the start and end. These intermediate callbacks trigger MapPosition in the handler, which calls UpdateFromPosition() again with incorrect intermediate position values, interrupting the original scroll to the target position.

Files Changed

  • Fix: src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs (+15/-2)

    • Changed UpdateFromPosition() to async void
    • Added OperatingSystem.IsIOSVersionAtLeast(26) guard
    • Adds 200ms Task.Delay().ContinueWith() + MainThread.BeginInvokeOnMainThread() debounce pattern
  • Snapshot: src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png (updated baseline)

Reviewer Feedback (Copilot automated review)

File:Line Issue Status
CarouselViewController2.cs:567 Stale state risk: closure captures carouselPosition/currentItemPosition before delay; no re-validation after INVESTIGATE 200ms
CarouselViewController2.cs:562 200ms delay is arbitrary, no documentation/comment explaining INVESTIGATE choice
CarouselViewController2.cs:560 Missing MacCatalyst 26 check (pattern in codebase: CarouselViewController2.cs:540 async voidlacks exception handling - unhandled exceptions crash the INVESTIGATE appCarouselViewController2.cs:568` Race condition: multiple rapid calls create multiple delayed tasks executing INVESTIGATE concurrently

Existing Tests

  • CarouselViewShouldScrollToRightPosition (Issue17283.cs) - screenshot test for scrolling to right position, iOS/Android/Mac
  • CarouselViewiOSCrashPreventionTest (Issue16020.cs) - navigates to last item, all platforms

Fix Candidates

Source Approach Test Result Files Changed Notes

PR PR #34013 200ms debounce delay on iOS 26+ via async PENDING (Gate) CarouselViewController2.cs (+15/-2) Original PR void
🚦 Gate — Test Verification
📝 Review Sessionadded ios26 image. · 9fe943f
** FAILEDResult:** Platform: ios Mode: Full Verification

  • Tests FAIL without (tests does not detect the bug)PASSED fix
  • Tests PASS with fix

Analysis

The Gate failure is explained by the snapshot test structure:

  1. The PR updates both CarouselViewController2.cs (code fix) AND ios-26/CarouselViewShouldScrollToRightPosition.png (updated snapshot baseline).
  2. When the verification script reverts the fix files, it also reverts the snapshot baseline to the old (incorrect) image.
    PASS.

Root Cause of Gate Failure: The existing tests use screenshot comparison (VerifyScreenshot()). The PR updated the snapshot as part of the fix, which means reverting the fix also reverts the "expected" snapshot, making the test pass regardless of whether the fix is present.

What's Needed: The tests need to include a non-visual assertion (e.g., checking the current item index or label text) that would fail when the wrong item is shown, independent of snapshot comparison.

🔧 Fix — Analysis & Comparison
📝 Review Sessionadded ios26 image. · 9fe943f

Fix Candidates

Source Approach Test Result Files Changed Notes

PR PR #34013 200ms debounce delay on iOS 26+ via async void PASS (Gate - with fix) CarouselViewController2.cs (+15/-2) Original PR
Exhausted: N/A ( Gate failed, Fix phase not executed)skipped Selected Fix: PR's Gate confirmed tests pass with the fix, but tests don't catch the bug without itfix Skipped Reason: Gate failed (tests don't detect the bug). Fix phase skipped per workflow rules.

📋 Report — Final Recommendation
📝 Review Sessionadded ios26 image. · 9fe943f
Final Recommendation: REQUEST CHANGES##

Summary

PR #34013 fixes a real iOS 26-specific bug where CarouselView scrolls to the wrong item when navigating programmatically. The root cause analysis and fix approach are sound. However, the Gate phase failed because the existing tests don't actually detect the the PR's snapshot update is bundled with the code fix, making the test pass regardless of whether the fix is present.bug

Additionally, the fix has several code quality concerns raised by Copilot automated review that should be addressed before merge.

Root Cause

UpdateFromPosition() with stale intermediate position values, interrupting the scroll to the intended position.

Fix Quality

The fix works (tests pass with fix), but has the following concerns:

UpdateFromPosition() captures carouselPosition and currentItemPosition before the 200ms delay. If called multiple times rapidly (which is exactly what iOS 26 does), multiple delayed tasks execute concurrently with potentially stale values. The last one to fire wins unpredictably. A CancellationTokenSource debounce pattern should be used instead.**1.

The 200ms delay can span view disposal, navigation away, or ItemsSource changes. There's no re-validation inside the MainThread.BeginInvokeOnMainThread callback. Compare with UpdateInitialPosition() (line 602) which validates IsViewLoaded, ItemsSource, and ItemCount inside its callback. UpdateFromPosition should do the same.**2.

Other iOS 26 workarounds in the codebase consistently use:**3.

OperatingSystem.IsIOSVersionAtLeast(26) || OperatingSystem.IsMacCatalystVersionAtLeast(26)

This fix only checks iOS 26. If the same UICollectionView behavior change applies to MacCatalyst 26, it will remain broken there.

** async void Lacks Exception Handling (Medium)** 4. async void methods swallow exceptions into the synchronization context, potentially crashing the app. The method should either wrap the body in try/catch or be restructured.

** Tests Don't Catch the Bug (High)** 5. The existing CarouselViewShouldScrollToRightPosition test is a snapshot test. Since the PR also updates the snapshot file, reverting the fix also reverts the "expected" making the test pass against broken behavior. A non-visual assertion (e.g., checking GoToLastItem results in the correct item index or label text) is needed.image

Suggested Changes

  1. Fix race condition: Use a CancellationTokenSource field to cancel any pending delayed scroll on each new call
  2. Add state re-validation: Inside the MainThread.BeginInvokeOnMainThread callback, re-read the current state (re-check InitialPositionSet, ItemsView, ItemsSource, and re-compute position)
  3. Add MacCatalyst 26 check: Update the condition to include || OperatingSystem.IsMacCatalystVersionAtLeast(26) (if applicable)
  4. Add exception handling: Wrap the async body in try/catch
  5. Strengthen tests: Add an assertion that verifies the correct item/position after navigation (e.g., verify the visible item label is "5" or the position equals the expected index)

References

📋 Expand PR Finalization Review

  • Fixed the race condition by introducing a CancellationTokenSource to cancel any pending delayed scroll.
  • Re-validated the current state inside the MainThread.BeginInvokeOnMainThread callback
  • Wrapped the async logic in a try/catch block for safety.
  • The Mac Catalyst–specific check is not required, as the current logic works correctly on both iOS and Mac Catalyst.
  • Existing tests already cover the expected behavior and pass with this fix.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 10, 2026

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 34013

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 34013"

@Vignesh-SF3580
Copy link
Copy Markdown
Contributor Author

🤖 AI Summary

📊 Expand Full Review
🔍 Pre-Flight — Context & Validation
📝 Review SessionUpdate CarouselViewController2.cs · 01be7b8
Issue: #33770 - [iOS 26] CarouselView does not scroll to the correct last item Platforms Affected: iOS 26 only (confirmed working on iOS 18) Files Changed: 1 implementation file, 1 snapshot image (tests NOT updated)

Issue Summary

CarouselView fails to scroll to the last item on iOS 26 when using programmatic navigation (Position or CurrentItem). Clicking "Go to LastItem" button navigates to the next item instead of the last item. iOS 26-specific; does not occur on iOS 18.

Root Cause (from PR description)

iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire intermediate scroll position callbacks during animation. Whereas iOS 18 only fires at start/end, iOS 26 fires intermediate callbacks that trigger MapPosition in the handler, which calls UpdateFromPosition() again with incorrect intermediate position values, interrupting the original scroll.

Prior Agent Review (Session Feb 18, 2026)

  • Pre-Flight COMPLETE:
  • ** Tests don't detect the bug (snapshot test reverts with fix)FAILED Gate:**
  • Fix: Skipped (Gate failed)
  • Report: REQUEST Code quality concerns + tests don't catch bugCHANGES

Author Responses to Prior Review

Concern Status
Stale state capture before delay Re-validates state inside callback FIXED
200ms delay undocumented Added comment; reduced to 100ms FIXED
Race condition: multiple rapid calls Added CancellationTokenSource debounce FIXED
lacks exception handling Added catch (Exception) (silent)
MacCatalyst 26 check Author claims not needed REJECTED missing
Tests skip iOS 26 with Assert. NOT ADDRESSED Ignore

Files Changed

  • Fix: src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs (+52/-2)

    • Changed UpdateFromPosition() to async void
    • Added CancellationTokenSource _scrollDebounce field
    • Added OperatingSystem.IsIOSVersionAtLeast(26) guard
    • 100ms debounce with CancellationToken to cancel previous calls
    • Re-validates state inside callback before scrolling
    • try/catch wraps the async body
  • Snapshot: src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png (new baseline)

CRITICAL TEST ISSUE

Both existing tests explicitly ignore/skip iOS 26:

// Issue17283.cs
if (App is AppiumIOSApp iosApp && HelperExtensions.IsIOS26OrHigher(iosApp))
{
    Assert.Ignore("Ignored due to CarouselView scroll item issue in iOS 26.");
}
// Issue16020. same guardcs 

The snapshot was added for ios-26/CarouselViewShouldScrollToRightPosition.png but the test that generates it is still being ignored on iOS 26. The Assert.Ignore guards must be removed now that the fix is present.

Fix Candidates

Source Approach Test Result Files Changed Notes

PR PR #34013 CancellationTokenSource debounce (100ms) + state re-validation on iOS PENDING (Gate) CarouselViewController2.cs (+52/-2) Original PR - updated based on review 26+
🚦 Gate — Test Verification
📝 Review SessionUpdate CarouselViewController2.cs · 01be7b8
Result PASSED: Platform: ios Mode: Full Verification Test: CarouselViewShouldScrollToRightPosition (Issue17283)

  • Tests FAIL without fix
  • Tests PASS with fix

Details

  • Build SDK: Microsoft.iOS.Sdk.net10.0_26.0 (iOS 26.0)
  • Fix files reverted: CarouselViewController2.cs (and infrastructure files)
  • Test duration: ~4 minutes 18 seconds
  • Merge-base: c5fadf8

🔧 Fix — Analysis & Comparison
📝 Review SessionUpdate CarouselViewController2.cs · 01be7b8

Fix Candidates

Source Approach Test Result Files Changed Notes

1 try-fix bool flag, cleared in UpdateIsScrolling(false) PASS 1 file Simplest: ~10 lines, synchronous
2 try-fix , SetPosition rejects non-matching, cleared in ScrollAnimationEnded PASS 1 file Precise target tracking
3 try-fix Suppress MapPosition when carouselView.IsScrolling== FAIL 1 file Too broad: blocks user scroll position updates true
4 try-fix _nativePositionUpdateCount counter around FAIL 1 file Build env issue (VisualRegressionTester compile error) SetValueFromRenderer
5 try-fix counter + CollectionView.Layer.AnimationKeys stale check PASS 1 file Complex but stale-detection variant
6 try-fix UIView.Animate wrapper with ScrollToItem(animated:false) on iOS 26 PASS 1 file Prevents callbacks at source; hardcoded 0.3s duration +
7 try-fix Geometric expectedFinalOffsetX spatial filter via WillEndDragging PASS 2 files Spatial approach; modifies LayoutFactory2.cs
8 try-fix bool + CollectionView.Decelerating check + CommitCenteredPosition PASS 1 file Gesture-aware; distinguishes user vs programmatic scroll
9 try-fix MapPosition transaction gate in CarouselViewHandler2.iOS.cs, commit in ScrollAnimationEnded PASS 3 files Handler-level; modifies 3 files
10 try-fix in SetPosition rejects non-matching, commits in UpdateIsScrolling PASS 1 file Variant of Attempt 2
11 try-fix CATransaction CompletionBlock clears flag PASS 1 file Precise CA-based timing; ~16 lines
12 try-fix Temporarily nullify CollectionView.Delegate during scroll, restore in CATransaction PASS 1 file Most aggressive suppression; complex
13 try-fix _programmaticScrollTarget=-1 defer in UpdateFromPosition, commit in FAIL 1 file Build env issue (VisualRegressionTester compile error) UpdateIsScrolling
14 try-fix SetContentOffset(targetOffset, animated:true) to bypass ScrollToItem on iOS 18 PASS 1 file Bypasses ScrollToItem; iOS 18+ scope too broad; uses FlowLayout assumption +
PR PR #34013 async void + CancellationTokenSource 100ms debounce + state re-validation on iOS 26 PASS (Gate) 1 file, 1 snapshot Original PR +
Exhausted: Yes (3 cross-pollination rounds; all 5 models confirmed NO NEW IDEAS)

Selected Fix: PR's fix

Reason: The PR's fix (after the author's revisions addressing the prior agent review) is the most defensively robust:

  • CancellationTokenSource correctly handles rapid successive calls (cancels the previous pending delay when a new call comes in)
  • 100ms delay matches the existing UpdateInitialPosition pattern in the same class
  • State re-validation inside the callback (re-checks InitialPositionSet, ItemsView, ItemsSource) prevents stale-state operations
  • try/catch prevents app crashes from async void exceptions

Simpler alternatives (Attempts 1, 11) are viable but trade defensive robustness for brevity. For a high-traffic callback path on iOS 26+ where rapid calls are the exact problem being fixed, the CancellationTokenSource approach is more appropriate.

Remaining issue: The tests still have Assert.Ignore guards for iOS 26, which means the test suite won't verify the fix in CI on iOS 26. The PR must remove these guards.

📋 Report — Final Recommendation
📝 Review SessionUpdate CarouselViewController2.cs · 01be7b8
Final Recommendation: REQUEST CHANGES##

Summary

PR #34013 correctly identifies and addresses a real iOS 26-specific bug in CarouselView where UICollectionView.ScrollToItem() fires intermediate scroll position callbacks that interrupt programmatic navigation to the last item. The code fix quality has been significantly improved from the initial submission (author addressed all prior review feedback with CancellationTokenSource, state re-validation, documentation, and try/catch). Gate ** tests correctly detect the bug on iOS 26.PASSED**

However, two issues require changes before merge:

  1. Tests still skip iOS Both existing tests have Assert.Ignore guards for iOS 26 that were never removed. The snapshot added by this PR will never be used in CI.26
  2. CancellationTokenSource is never Memory/resource leak risk.disposed

Root Cause

a new competing ScrollToItem() call, resulting in the wrong item being displayed.

Fix Quality

The PR's CancellationTokenSource debounce approach is defensively correct for this use case:

  • Each new UpdateFromPosition() call cancels the previous 100ms delay, ensuring only the final position update fires
  • State re-validation inside the callback prevents operating on stale views
  • Matches the existing UpdateInitialPosition() pattern (also uses 100ms delay)
  • try/catch prevents async void exceptions from crashing the app

11 alternative fixes were explored across 3 cross-pollination rounds. All 9 simpler alternatives (bool flags, counters, UIView.Animate, geometric filter, gesture state, CATransaction, SetContentOffset) passed tests. The simplest alternative (Attempt 1: _isScrollingProgrammatically bool flag, ~10 lines) is viable but trades defensive robustness for brevity. The PR's approach is more robust for rapid successive callbacks, which is the exact scenario causing this bug.

Issues Found

** Both tests referenced in the PR have Assert.Ignore guards that skip execution on iOS 26:

// Issue17283.cs line 22- MUST REMOVE:25 
if (App is AppiumIOSApp iosApp && HelperExtensions.IsIOS26OrHigher(iosApp))
{
    Assert.Ignore("Ignored due to CarouselView scroll item issue in iOS 26.");
}

// Issue16020. same  MUST REMOVEguard cs 

These guards were added before this fix to prevent CI failures from the known bug. Now that the fix is present, they must be removed so the tests actually run on iOS 26 and validate the fix going forward. The ios-26/CarouselViewShouldScrollToRightPosition.png snapshot added by this PR will never be compared in CI until these guards are removed.

Required changes:

  • src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue17283. remove the IsIOS26OrHigher guard blockcs
  • src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue16020. remove the IsIOS26OrHigher guard blockcs

** The _scrollDebounce field accumulates a new CancellationTokenSource on every UpdateFromPosition() call. The previous one is cancelled but not disposed. This should be:

_scrollDebounce?.Cancel();
_scrollDebounce?.Dispose();  // ADD THIS
_scrollDebounce = new CancellationTokenSource();

And in TearDown/disconnect:

_scrollDebounce?.Cancel();
_scrollDebounce?.Dispose();
_scrollDebounce = null;

** [iOS26] should be [iOS 26] with a space (consistent with issue labels like version/iOS-26).

**

  • CancellationTokenSource prevents race conditions from rapid calls
  • State re-validation inside callback (re-checks InitialPositionSet, ItemsView, ItemsSource)
  • try/catch wraps async void body
  • Comment explains the 100ms delay choice
  • Delay reduced from 200ms to 100ms (matching UpdateInitialPosition)

Alternative Fix Considered

The simplest validated alternative (Attempt 1, _isScrollingProgrammatically bool flag):

  • ~10 lines vs ~50 lines in the PR
  • Synchronous, no timing dependency
  • However: doesn't handle rapid successive calls as robustly as CancellationTokenSource

Selected Fix: PR's more robust for the specific problem (rapid intermediate callbacks). The PR's approach is appropriate; the required changes are removing the Assert.Ignore guards and disposing CancellationTokenSource.fix

What NOT to Do (for future agents)

Don't suppress MapPosition when carouselView.IsScrolling == IsScrolling is true for ALL scrolling (user + programmatic), blocking legitimate user-initiated position updatestrue - Don't use SetContentOffset with UICollectionViewFlowLayout CarouselView2 uses compositional layout, not FlowLayoutassumptions - Don't assume the snapshot validates the always check that the test body actually RUNS on the target OS version (Assert.Ignore guards can silently bypass test execution)fix -

References

📋 Expand PR Finalization Review

Fixed the memory leak by overriding Dispose in CarouselViewController2 to properly cancel and dispose of _scrollDebounce when the view controller is disposed. No other major concerns.

@kubaflo kubaflo added s/agent-review-incomplete and removed s/agent-changes-requested AI agent recommends changes - found a better alternative or issues labels Mar 16, 2026
@sheiksyedm
Copy link
Copy Markdown
Contributor

/azp run maui-pr-uitests , maui-pr-devicetests

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@dotnet dotnet deleted a comment from rmarinho Mar 18, 2026
@kubaflo
Copy link
Copy Markdown
Contributor

kubaflo commented Mar 18, 2026

🤖 AI Summary

📊 Expand Full Reviewcc4e585 · Addressed concerns.
🔍 Pre-Flight — Context & Validation

Issue: #33770 - [iOS 26] CarouselView does not scroll to the correct last item
PR: #34013 - [iOS26] Fix CarouselView scrolling to wrong item when navigating to last item
Platforms Affected: iOS 26
Files Changed: 1 implementation, 2 API baseline, 1 test snapshot asset

Key Findings

  • The linked issue is explicitly iOS 26-specific and was validated by MAUI team members as reproducing on iOS 26 but not iOS 18.
  • The PR changes src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs, adds Dispose(bool) API entries to the iOS and MacCatalyst PublicAPI baselines, and updates an iOS 26 snapshot image; it does not add or modify executable tests.
  • The PR description attributes the bug to iOS 26 UICollectionView.ScrollToItem() emitting intermediate callbacks that retrigger UpdateFromPosition() with transient positions during animation.
  • Prior agent-review history on the PR shows an earlier round of concerns around stale captured state, arbitrary delay, exception handling, and concurrent delayed callbacks; the author responded by adding cancellation-token debouncing, callback-time state revalidation, a comment explaining the delay, and Dispose(bool) cleanup.
  • Review discussion still includes an unresolved product question about whether the iOS 26 workaround should also be gated for MacCatalyst 26; the author stated the extra check was unnecessary.
  • Because only a snapshot asset changed on the test side, Gate needs to confirm that the existing tests actually fail against a broken baseline rather than simply matching a reverted expected image.

Edge Cases / Discussion Notes

  • The issue is reported as iOS 26-only and explicitly non-reproducible on iOS 18.
  • Prior review feedback questioned MacCatalyst 26 behavior; current PR discussion says the fix works there without an explicit MacCatalyst version check.
  • The PR body says existing tests already cover the scenario, so Gate must validate that claim directly.

Fix Candidates

# Source Approach Test Result Files Changed Notes
PR PR #34013 Debounce iOS 26 UpdateFromPosition() with cancellable 100ms delay, then re-read state on main thread before ScrollToPosition; add Dispose(bool) PENDING (Gate) src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs, src/Controls/src/Core/PublicAPI/net-ios/PublicAPI.Unshipped.txt, src/Controls/src/Core/PublicAPI/net-maccatalyst/PublicAPI.Unshipped.txt, src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png Original PR; executable tests were not modified cleanup

🚦 Gate — Test Verification

Gate FAILEDResult:

Platform: ios
Mode: Full Verification
Verification Agent: verify-tests-fail-without-fix via task agent
Test Filter Used by verifier: CarouselView

  • Tests FAIL without fix:
  • Tests PASS with fix:

Notes


🔧 Fix — Analysis & Comparison

Fix Candidates

# Source Approach Test Result Files Changed Notes
1 try-fix (claude-opus-4.6) Programmatic-scroll guard flag that ignores re-entrant UpdateFromPosition during animated programmatic scroll and clears on ScrollAnimationEnded PASS 2 files Deterministic lifecycle guard; no async, timer, or version gate
2 try-fix (claude-sonnet-4.6) In-flight target-position guard that suppresses duplicate re-entry while already animating to the same target PASS 1 file More selective than attempt 1; still synchronous and delay-free
3 try-fix (gpt-5.3-codex) End-of-animation position commit that suppresses intermediate callback-driven position updates and commits once scrolling ends PASS 1 file Passed after one transient Appium retry; artifacts landed in attempt-4
4 try-fix (gemini-3-pro-preview) Suppress intermediate Scrolled propagation during animated programmatic scroll while keeping offsets in sync FAIL 2 files Failed validation due repeated suite instability and unrelated CarouselView failure
5 try-fix (round 2 new idea) Use non-animated scroll for iOS 26 multi-item or last-item programmatic jumps, then explicitly SetPosition PASS 1 file Smallest passing alternative; avoids intermediate callbacks entirely
6 try-fix (round 2 new idea) Use geometry-based ScrollRectToVisible navigation via layout attributes and re-enable the two iOS 26 issue tests PASS 3 files Only candidate that also makes the skipped issue-specific tests run on iOS 26
7 try-fix (round 3 new idea) Wrap ScrollToItem in PerformBatchUpdates to force collection-view synchronization before scrolling FAIL 1 file Timed out on the relevant scenario and introduced broader instability
PR PR #34013 Cancellable 100ms debounce in UpdateFromPosition on iOS 26, with callback-time state revalidation and Dispose cleanup PASS (with fix), but gate coverage invalid 4 files Existing issue-specific tests are still skipped on iOS 26

Cross-Pollination

Model Round New Ideas? Details
claude-opus-4.6 2 No NO NEW IDEAS
claude-sonnet-4.6 2 Yes Use non-animated programmatic scroll for multi-item jumps or last-item jumps, then explicitly set final index
gpt-5.3-codex 2 No NO NEW IDEAS
gemini-3-pro-preview 2 Yes Use ScrollRectToVisible with the target item's frame from layout attributes instead of ScrollToItem
claude-opus-4.6 3 No NO NEW IDEAS
claude-sonnet-4.6 3 No NO NEW IDEAS
gpt-5.3-codex 3 No NO NEW IDEAS
gemini-3-pro-preview 3 Yes Wrap scroll in PerformBatchUpdates to synchronize layout and data-source state before scrolling

Exhausted: Yes (3 cross-pollination rounds completed)
Selected Fix: Candidate #6 - it is the only empirically passing alternative that also closes the current gate coverage gap by re-enabling Issue16020 and Issue17283 on iOS 26, making it the strongest merge-ready option among the explored candidates.


📋 Report — Final Recommendation

Final Recommendation: REQUEST CHANGES

Phase Status

Phase Status Notes
Pre-Flight COMPLETE Issue #33770 is iOS 26-specific; PR updates one iOS handler, two PublicAPI baselines, and one iOS-26 snapshot asset.
Gate FAILED Current iOS verification does not prove the bug is caught because the two issue-specific tests still Assert.Ignore on iOS 26.
Try-Fix COMPLETE 7 empirically tested alternatives (5 pass, 2 fail) across 3 cross-pollination rounds.
Report COMPLETE Final recommendation written.

Summary

PR #34013 addresses a plausible iOS 26 CarouselView regression and the current implementation is more defensively coded than the initial submission. However, the PR is not ready to approve because the current test story is still insufficient on the affected platform: Issue17283.CarouselViewShouldScrollToRightPosition and Issue16020.CarouselViewiOSCrashPreventionTest remain skipped on iOS 26, so Gate could not verify fail-without-fix and pass-with-fix on the scenario the PR claims to solve.

Try-fix exploration found multiple passing alternatives. The strongest explored option was Candidate #6, which replaced the ScrollToItem path with geometry-based ScrollRectToVisible navigation and re-enabled the two iOS 26 issue tests; that candidate passed validation and is the only explored option that also fixes the current gate-coverage gap.

Root Cause

The issue is consistent with iOS 26 UICollectionView behavior emitting intermediate callbacks during animated programmatic scrolling. Those transient positions feed back into MAUI's position synchronization path (MapPosition and UpdateFromPosition) and can redirect the carousel before the original scroll reaches the intended final item.

Fix Quality

The PR's current debounce-based fix is serviceable, but it is not the best merge-ready outcome from this review because it leaves the relevant iOS 26 tests disabled. A reviewable fix for this issue needs to do both of the following:

  • resolve the iOS 26 scrolling behavior, and
  • allow the affected iOS 26 issue tests to execute so the bug is actually covered in CI.

Requested Changes

  • Re-enable Issue17283.CarouselViewShouldScrollToRightPosition and Issue16020.CarouselViewiOSCrashPreventionTest on iOS 26 so Gate can validate the bug directly.
  • Prefer a fix approach that removes the need for timing-based debounce if possible. Candidate [Spec] Transitions #6 from try-fix is the strongest explored alternative because it passed while also enabling the skipped issue tests.
  • If the debounce-based PR fix is retained, it still needs accompanying test changes that make the iOS 26 scenario run rather than skip.

📋 Expand PR Finalization Review

PR #34013 Finalization Review

Title

Current: [iOS26] Fix CarouselView scrolling to wrong item when navigating to last item

Recommended: [iOS] CarouselView: Fix scrolling to the wrong item on iOS 26 when navigating to the last item

The current title is close, but [iOS26] is not a normal MAUI platform prefix. Using [iOS] keeps the title searchable and consistent with the repo's existing title pattern, while preserving the iOS 26 context in the description.

Description

Assessment: Good foundation - keep most of it.

What is already good:

  • The required NOTE block is present.
  • The issue, root cause, and screenshots are all documented.
  • The platform-specific scope is clear.

What should be updated so the description matches the final implementation:

  • The final fix is not just a plain delayed ScrollToPosition() call. It now adds a cancellable iOS 26 debounce in UpdateFromPosition(), re-reads CurrentItem/Position after the delay, and skips work if state changed.
  • The PR also adds cleanup logic for the debounce token source via a new Dispose(bool) override.
  • The PR refreshes the iOS 26 screenshot baseline at src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png.
  • If the PublicAPI.Unshipped.txt entries stay, the description should explicitly call out that this PR now introduces a protected API override on a public type. If that API change is unintentional, remove it instead of documenting it.

Code Review Findings

Significant Issues

Unexpected public API change

  • Files: src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs, src/Controls/src/Core/PublicAPI/net-ios/PublicAPI.Unshipped.txt, src/Controls/src/Core/PublicAPI/net-maccatalyst/PublicAPI.Unshipped.txt
  • Problem: Adding protected override void Dispose(bool disposing) on public CarouselViewController2 surfaces a new public/protected API, which is why PublicAPI.Unshipped.txt changed. For a bug-fix PR on main, this looks accidental rather than intentional.
  • Recommendation: Avoid the API surface change if possible by cleaning up _scrollDebounce through an existing disposal/teardown path, or explicitly treat this as an intentional API change and document it as such.

Canceled debounce tokens are never disposed when replaced

  • File: src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs
  • Problem: Each iOS 26 callback cancels the previous _scrollDebounce token source, but then immediately overwrites the field with a new CancellationTokenSource without disposing the old one. Under rapid scrolling/programmatic updates, this can accumulate undisposed token sources until controller disposal.
  • Recommendation: Dispose the previous token source when replacing it, for example by exchanging the field, canceling the old instance, and then disposing it before continuing.

Suggestions

Broad silent catch hides real failures

  • File: src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs
  • Problem: catch (Exception) swallows every failure from the async debounce path. That makes real bugs indistinguishable from expected cancellation and conflicts with the repo guidance to avoid broad silent fallbacks.
  • Recommendation: Catch OperationCanceledException specifically, or separate cancellation from unexpected exceptions and surface/log the latter.

Looks Good

  • The PR is scoped to the active iOS/MacCatalyst Items2 handler implementation, which is the correct area for this issue.
  • Re-reading CurrentItem and Position after the delay is safer than using the stale pre-delay values.
  • The description already captures the root cause well; it mostly needs an implementation refresh rather than a full rewrite.

@kubaflo kubaflo added s/agent-changes-requested AI agent recommends changes - found a better alternative or issues s/agent-fix-win AI found a better alternative fix than the PR and removed s/agent-review-incomplete s/agent-fix-pr-picked AI could not beat the PR fix - PR is the best among all candidates labels Mar 18, 2026
@Vignesh-SF3580
Copy link
Copy Markdown
Contributor Author

🤖 AI Summary

📊 Expand Full Reviewcc4e585 · Addressed concerns.
🔍 Pre-Flight — Context & Validation
Issue: #33770 - [iOS 26] CarouselView does not scroll to the correct last item PR: #34013 - [iOS26] Fix CarouselView scrolling to wrong item when navigating to last item Platforms Affected: iOS 26 Files Changed: 1 implementation, 2 API baseline, 1 test snapshot asset

Key Findings

  • The linked issue is explicitly iOS 26-specific and was validated by MAUI team members as reproducing on iOS 26 but not iOS 18.
  • The PR changes src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs, adds Dispose(bool) API entries to the iOS and MacCatalyst PublicAPI baselines, and updates an iOS 26 snapshot image; it does not add or modify executable tests.
  • The PR description attributes the bug to iOS 26 UICollectionView.ScrollToItem() emitting intermediate callbacks that retrigger UpdateFromPosition() with transient positions during animation.
  • Prior agent-review history on the PR shows an earlier round of concerns around stale captured state, arbitrary delay, exception handling, and concurrent delayed callbacks; the author responded by adding cancellation-token debouncing, callback-time state revalidation, a comment explaining the delay, and Dispose(bool) cleanup.
  • Review discussion still includes an unresolved product question about whether the iOS 26 workaround should also be gated for MacCatalyst 26; the author stated the extra check was unnecessary.
  • Because only a snapshot asset changed on the test side, Gate needs to confirm that the existing tests actually fail against a broken baseline rather than simply matching a reverted expected image.

Edge Cases / Discussion Notes

  • The issue is reported as iOS 26-only and explicitly non-reproducible on iOS 18.
  • Prior review feedback questioned MacCatalyst 26 behavior; current PR discussion says the fix works there without an explicit MacCatalyst version check.
  • The PR body says existing tests already cover the scenario, so Gate must validate that claim directly.

Fix Candidates

Source Approach Test Result Files Changed Notes

PR PR #34013 Debounce iOS 26 UpdateFromPosition() with cancellable 100ms delay, then re-read state on main thread before ScrollToPosition; add Dispose(bool) PENDING (Gate) src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs, src/Controls/src/Core/PublicAPI/net-ios/PublicAPI.Unshipped.txt, src/Controls/src/Core/PublicAPI/net-maccatalyst/PublicAPI.Unshipped.txt, src/Controls/tests/TestCases.iOS.Tests/snapshots/ios-26/CarouselViewShouldScrollToRightPosition.png Original PR; executable tests were not modified cleanup
🚦 Gate — Test Verification

Gate FAILEDResult:

Platform: ios Mode: Full Verification Verification Agent: verify-tests-fail-without-fix via task agent Test Filter Used by verifier: CarouselView

  • Tests FAIL without fix:
  • Tests PASS with fix:

Notes

🔧 Fix — Analysis & Comparison

Fix Candidates

Source Approach Test Result Files Changed Notes

1 try-fix (claude-opus-4.6) Programmatic-scroll guard flag that ignores re-entrant UpdateFromPosition during animated programmatic scroll and clears on ScrollAnimationEnded PASS 2 files Deterministic lifecycle guard; no async, timer, or version gate
2 try-fix (claude-sonnet-4.6) In-flight target-position guard that suppresses duplicate re-entry while already animating to the same target PASS 1 file More selective than attempt 1; still synchronous and delay-free
3 try-fix (gpt-5.3-codex) End-of-animation position commit that suppresses intermediate callback-driven position updates and commits once scrolling ends PASS 1 file Passed after one transient Appium retry; artifacts landed in attempt-4
4 try-fix (gemini-3-pro-preview) Suppress intermediate Scrolled propagation during animated programmatic scroll while keeping offsets in sync FAIL 2 files Failed validation due repeated suite instability and unrelated CarouselView failure
5 try-fix (round 2 new idea) Use non-animated scroll for iOS 26 multi-item or last-item programmatic jumps, then explicitly SetPosition PASS 1 file Smallest passing alternative; avoids intermediate callbacks entirely
6 try-fix (round 2 new idea) Use geometry-based ScrollRectToVisible navigation via layout attributes and re-enable the two iOS 26 issue tests PASS 3 files Only candidate that also makes the skipped issue-specific tests run on iOS 26
7 try-fix (round 3 new idea) Wrap ScrollToItem in PerformBatchUpdates to force collection-view synchronization before scrolling FAIL 1 file Timed out on the relevant scenario and introduced broader instability
PR PR #34013 Cancellable 100ms debounce in UpdateFromPosition on iOS 26, with callback-time state revalidation and Dispose cleanup PASS (with fix), but gate coverage invalid 4 files Existing issue-specific tests are still skipped on iOS 26

Cross-Pollination

Model Round New Ideas? Details
claude-opus-4.6 2 No NO NEW IDEAS
claude-sonnet-4.6 2 Yes Use non-animated programmatic scroll for multi-item jumps or last-item jumps, then explicitly set final index
gpt-5.3-codex 2 No NO NEW IDEAS
gemini-3-pro-preview 2 Yes Use ScrollRectToVisible with the target item's frame from layout attributes instead of ScrollToItem
claude-opus-4.6 3 No NO NEW IDEAS
claude-sonnet-4.6 3 No NO NEW IDEAS
gpt-5.3-codex 3 No NO NEW IDEAS
gemini-3-pro-preview 3 Yes Wrap scroll in PerformBatchUpdates to synchronize layout and data-source state before scrolling
Exhausted: Yes (3 cross-pollination rounds completed) Selected Fix: Candidate #6 - it is the only empirically passing alternative that also closes the current gate coverage gap by re-enabling Issue16020 and Issue17283 on iOS 26, making it the strongest merge-ready option among the explored candidates.

📋 Report — Final Recommendation

Final Recommendation: REQUEST CHANGES

Phase Status

Phase Status Notes
Pre-Flight COMPLETE Issue #33770 is iOS 26-specific; PR updates one iOS handler, two PublicAPI baselines, and one iOS-26 snapshot asset.
Gate FAILED Current iOS verification does not prove the bug is caught because the two issue-specific tests still Assert.Ignore on iOS 26.
Try-Fix COMPLETE 7 empirically tested alternatives (5 pass, 2 fail) across 3 cross-pollination rounds.
Report COMPLETE Final recommendation written.

Summary

PR #34013 addresses a plausible iOS 26 CarouselView regression and the current implementation is more defensively coded than the initial submission. However, the PR is not ready to approve because the current test story is still insufficient on the affected platform: Issue17283.CarouselViewShouldScrollToRightPosition and Issue16020.CarouselViewiOSCrashPreventionTest remain skipped on iOS 26, so Gate could not verify fail-without-fix and pass-with-fix on the scenario the PR claims to solve.

Try-fix exploration found multiple passing alternatives. The strongest explored option was Candidate #6, which replaced the ScrollToItem path with geometry-based ScrollRectToVisible navigation and re-enabled the two iOS 26 issue tests; that candidate passed validation and is the only explored option that also fixes the current gate-coverage gap.

Root Cause

The issue is consistent with iOS 26 UICollectionView behavior emitting intermediate callbacks during animated programmatic scrolling. Those transient positions feed back into MAUI's position synchronization path (MapPosition and UpdateFromPosition) and can redirect the carousel before the original scroll reaches the intended final item.

Fix Quality

The PR's current debounce-based fix is serviceable, but it is not the best merge-ready outcome from this review because it leaves the relevant iOS 26 tests disabled. A reviewable fix for this issue needs to do both of the following:

  • resolve the iOS 26 scrolling behavior, and
  • allow the affected iOS 26 issue tests to execute so the bug is actually covered in CI.

Requested Changes

  • Re-enable Issue17283.CarouselViewShouldScrollToRightPosition and Issue16020.CarouselViewiOSCrashPreventionTest on iOS 26 so Gate can validate the bug directly.
  • Prefer a fix approach that removes the need for timing-based debounce if possible. Candidate [Spec] Transitions #6 from try-fix is the strongest explored alternative because it passed while also enabling the skipped issue tests.
  • If the debounce-based PR fix is retained, it still needs accompanying test changes that make the iOS 26 scenario run rather than skip.

📋 Expand PR Finalization Review

The suggested fix does not resolve the CarouselViewShouldScrollToRightPosition test, so I have only enabled both tests with the current PR changes.

PureWeen and others added 2 commits March 25, 2026 09:44
…otnet#34548)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

## Description

Adds a [gh-aw (GitHub Agentic
Workflows)](https://github.github.com/gh-aw/introduction/overview/)
workflow that automatically evaluates test quality on PRs using the
`evaluate-pr-tests` skill.

### What it does

When a PR adds or modifies test files, this workflow:
1. **Checks out the PR branch** (including fork PRs) in a pre-agent step
2. **Runs the `evaluate-pr-tests` skill** via Copilot CLI in a sandboxed
container
3. **Posts the evaluation report** as a PR comment using gh-aw
safe-outputs

### Triggers

| Trigger | When | Fork PR support |
|---------|------|-----------------|
| `pull_request` | Automatic on test file changes (`src/**/tests/**`) |
❌ Blocked by `pre_activation` gate |
| `workflow_dispatch` | Manual — enter PR number | ✅ Works for all PRs |
| `issue_comment` (`/evaluate-tests`) | Comment on PR | ⚠️ Same-repo
only (see Known Limitations) |

### Security model

| Layer | Implementation |
|-------|---------------|
| **gh-aw sandbox** | Agent runs in container with scrubbed credentials,
network firewall |
| **Safe outputs** | Max 1 PR comment per run, content-limited |
| **Checkout without execution** | `steps:` checks out PR code but never
executes workspace scripts |
| **Base branch restoration** | `.github/skills/`,
`.github/instructions/`, `.github/copilot-instructions.md` restored from
base branch after checkout |
| **Fork PR activation gate** | `pull_request` events blocked for forks
via `head.repo.id == repository_id` |
| **Pinned actions** | SHA-pinned `actions/checkout`,
`actions/github-script`, etc. |
| **Minimal permissions** | Each job declares only what it needs |
| **Concurrency** | One evaluation per PR, cancels in-progress |
| **Threat detection** | gh-aw built-in threat detection analyzes agent
output |

### Files added/modified

- `.github/workflows/copilot-evaluate-tests.md` — gh-aw workflow source
- `.github/workflows/copilot-evaluate-tests.lock.yml` — Compiled
workflow (auto-generated by `gh aw compile`)
- `.github/skills/evaluate-pr-tests/scripts/Gather-TestContext.ps1` —
Test context gathering script (binary-safe file download, path traversal
protection)
- `.github/instructions/gh-aw-workflows.instructions.md` — Copilot
instructions for gh-aw development

### Known Limitations

**Fork PR evaluation via `/evaluate-tests` comment is not supported in
v1.** The gh-aw platform inserts a `checkout_pr_branch.cjs` step after
all user steps, which may overwrite base-branch skill files restored for
fork PRs. This is a known gh-aw platform limitation — user steps always
run before platform-generated steps, with no way to insert steps after.

**Workaround:** Use `workflow_dispatch` (Actions UI → "Run workflow" →
enter PR number) to evaluate fork PRs. This trigger bypasses the
platform checkout step entirely and works correctly.

**Related upstream issues:**
- [github/gh-aw#18481](github/gh-aw#18481) —
"Using gh-aw in forks of repositories"
- [github/gh-aw#18518](github/gh-aw#18518) —
Fork detection and warning in `gh aw init`
- [github/gh-aw#18520](github/gh-aw#18520) —
Fork context hint in failure messages
- [github/gh-aw#18521](github/gh-aw#18521) —
Fork support documentation

### Fixes

- Fixes dotnet#34602

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Jakub Florkowski <kubaflo123@gmail.com>
## Summary

Enables the copilot-evaluate-tests gh-aw workflow to run on fork PRs by
adding `forks: ["*"]` to the `pull_request` trigger and removing the
fork guard from `Checkout-GhAwPr.ps1`.

## Changes

1. **copilot-evaluate-tests.md**: Added `forks: ["*"]` to opt out of
gh-aw auto-injected fork activation guard. Scoped `Checkout-GhAwPr.ps1`
step to `workflow_dispatch` only (redundant for other triggers since
platform handles checkout).

2. **copilot-evaluate-tests.lock.yml**: Recompiled via `gh aw compile` —
fork guard removed from activation `if:` conditions.

3. **Checkout-GhAwPr.ps1**: Removed the `isCrossRepository` fork guard.
Updated header docs and restore comments to accurately describe behavior
for all trigger×fork combinations (including corrected step ordering).

4. **gh-aw-workflows.instructions.md**: Updated all stale references to
the removed fork guard. Documented `forks: ["*"]` opt-in, clarified
residual risk model for fork PRs, and updated troubleshooting table.

## Security Model

Fork PRs are safe because:
- Agent runs in **sandboxed container** with all credentials scrubbed
- Output limited to **1 comment** via `safe-outputs: add-comment: max:
1`
- Agent **prompt comes from base branch** (`runtime-import`) — forks
cannot alter instructions
- Pre-flight check catches missing `SKILL.md` if fork isn't rebased on
`main`
- No workspace code is executed with `GITHUB_TOKEN` (checkout without
execution)

## Testing

- ✅ `workflow_dispatch` tested against fork PR dotnet#34621
- ✅ Lock.yml statically verified — fork guard removed from `if:`
conditions
- ⏳ `pull_request` trigger on fork PRs can only be verified post-merge
(GitHub Actions reads lock.yml from default branch)

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@MauiBot
Copy link
Copy Markdown
Collaborator

MauiBot commented Mar 28, 2026

🚦 Gate - Test Before and After Fix

📊 Expand Full Gate8d22058 · Update CarouselViewController2.cs

Gate Result: ❌ FAILED

Platform: IOS · Base: main · Merge base: 794a9fa6

Test Without Fix (expect FAIL) With Fix (expect PASS)
🖥️ Issue16020 Issue16020 ❌ PASS — 204s ✅ PASS — 83s
🖥️ Issue17283 Issue17283 ❌ PASS — 87s ✅ PASS — 80s
🔴 Without fix — 🖥️ Issue16020: PASS ❌ · 204s
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 458 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 914 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 5.03 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Foldable/src/Controls.Foldable.csproj (in 6.48 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj (in 6.48 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Xaml/Controls.Xaml.csproj (in 6.48 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/BlazorWebView/src/Maui/Microsoft.AspNetCore.Components.WebView.Maui.csproj (in 6.49 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 6.49 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 6.5 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Maps/src/Controls.Maps.csproj (in 6.5 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/maps/src/Maps.csproj (in 6.53 sec).
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
  Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
  Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Detected signing identity:
    Code Signing Key: "" (-)
    Provisioning Profile: "" () - no entitlements
    Bundle Id: com.microsoft.maui.uitests
    App Id: com.microsoft.maui.uitests
  Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
  Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
  Optimizing assemblies for size. This process might take a while.

Build succeeded.

/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
    1 Warning(s)
    0 Error(s)

Time Elapsed 00:01:40.87
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Core/UITest.Core.csproj (in 711 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 711 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 711 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils/VisualTestUtils.csproj (in 711 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/CustomAttributes/Controls.CustomAttributes.csproj (in 647 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 767 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 784 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 10 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.NUnit/UITest.NUnit.csproj (in 4.34 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Appium/UITest.Appium.csproj (in 4.71 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Analyzers/UITest.Analyzers.csproj (in 5.33 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils.MagickNet/VisualTestUtils.MagickNet.csproj (in 3.66 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.iOS.Tests/Controls.TestCases.iOS.Tests.csproj (in 5.68 sec).
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.04]   Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.13]   Discovered:  Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/7/2026 3:08:58 AM FixtureSetup for Issue16020(iOS)
>>>>> 4/7/2026 3:09:02 AM CarouselViewiOSCrashPreventionTest Start
>>>>> 4/7/2026 3:09:05 AM CarouselViewiOSCrashPreventionTest Stop
  Passed CarouselViewiOSCrashPreventionTest [3 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 59.7039 Seconds

🟢 With fix — 🖥️ Issue16020: PASS ✅ · 83s
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 296 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 313 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 313 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 339 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 353 ms).
  6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
  Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
  Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
  Detected signing identity:
    Code Signing Key: "" (-)
    Provisioning Profile: "" () - no entitlements
    Bundle Id: com.microsoft.maui.uitests
    App Id: com.microsoft.maui.uitests
  Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
  Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
  Optimizing assemblies for size. This process might take a while.

Build succeeded.

/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
    1 Warning(s)
    0 Error(s)

Time Elapsed 00:00:40.94
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 475 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 478 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 479 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 496 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 517 ms).
  8 of 13 projects are up-to-date for restore.
  Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.04]   Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.12]   Discovered:  Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/7/2026 3:11:49 AM FixtureSetup for Issue16020(iOS)
>>>>> 4/7/2026 3:11:52 AM CarouselViewiOSCrashPreventionTest Start
>>>>> 4/7/2026 3:11:57 AM CarouselViewiOSCrashPreventionTest Stop
  Passed CarouselViewiOSCrashPreventionTest [4 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 19.4313 Seconds

🔴 Without fix — 🖥️ Issue17283: PASS ❌ · 87s
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 339 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 357 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 358 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 396 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 410 ms).
  6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
  Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
  Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Detected signing identity:
    Code Signing Key: "" (-)
    Provisioning Profile: "" () - no entitlements
    Bundle Id: com.microsoft.maui.uitests
    App Id: com.microsoft.maui.uitests
  Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
  Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
  Optimizing assemblies for size. This process might take a while.

Build succeeded.

/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
    1 Warning(s)
    0 Error(s)

Time Elapsed 00:00:42.09
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 351 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 364 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 370 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 386 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 364 ms).
  8 of 13 projects are up-to-date for restore.
  Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.04]   Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.12]   Discovered:  Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/7/2026 3:10:26 AM FixtureSetup for Issue17283(iOS)
>>>>> 4/7/2026 3:10:30 AM CarouselViewShouldScrollToRightPosition Start
>>>>> 4/7/2026 3:10:33 AM CarouselViewShouldScrollToRightPosition Stop
  Passed CarouselViewShouldScrollToRightPosition [2 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 19.3875 Seconds

🟢 With fix — 🖥️ Issue17283: PASS ✅ · 80s
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 405 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 420 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 424 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 455 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 467 ms).
  6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
  Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
  Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
  Detected signing identity:
    Code Signing Key: "" (-)
    Provisioning Profile: "" () - no entitlements
    Bundle Id: com.microsoft.maui.uitests
    App Id: com.microsoft.maui.uitests
  Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
  Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
  Optimizing assemblies for size. This process might take a while.

Build succeeded.

/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
    1 Warning(s)
    0 Error(s)

Time Elapsed 00:00:38.24
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 306 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 305 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 315 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 330 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 348 ms).
  8 of 13 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13763453
  Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.03]   Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.12]   Discovered:  Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/7/2026 3:13:10 AM FixtureSetup for Issue17283(iOS)
>>>>> 4/7/2026 3:13:14 AM CarouselViewShouldScrollToRightPosition Start
>>>>> 4/7/2026 3:13:16 AM CarouselViewShouldScrollToRightPosition Stop
  Passed CarouselViewShouldScrollToRightPosition [2 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 18.2556 Seconds

⚠️ Issues found
  • Issue16020 PASSED without fix (should fail) — tests don't catch the bug
  • Issue17283 PASSED without fix (should fail) — tests don't catch the bug
📁 Fix files reverted (4 files)
  • eng/pipelines/ci-copilot.yml
  • src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs
  • src/Controls/src/Core/PublicAPI/net-ios/PublicAPI.Unshipped.txt
  • src/Controls/src/Core/PublicAPI/net-maccatalyst/PublicAPI.Unshipped.txt

@MauiBot MauiBot added s/agent-changes-requested AI agent recommends changes - found a better alternative or issues and removed s/agent-review-incomplete labels Mar 30, 2026
@dotnet dotnet deleted a comment from MauiBot Mar 30, 2026
@dotnet dotnet deleted a comment from MauiBot Mar 30, 2026
@Vignesh-SF3580
Copy link
Copy Markdown
Contributor Author

🤖 AI Summary

📊 Expand Full Review9fac735 · enable ios26 test.

I validated the alternative fix suggested in the AI summary, but it did not resolve the reported issue. However, the summary also highlighted two additional concerns, which have been addressed. No other major concerns in the AI summary.

  • _scrollDebounce was canceled but not disposed before being overwritten, causing a CancellationTokenSource leak during rapid position changes. This is fixed by capturing the old reference, assigning the new one, then canceling and disposing the old instance.

  • catch (Exception) was swallowing all exceptions, including real issues. This is fixed by narrowing it to catch (OperationCanceledException), which is the only expected exception in this scenario.

Copy link
Copy Markdown
Contributor

@kubaflo kubaflo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both tests pass without a fix - could you please verify?

@Vignesh-SF3580
Copy link
Copy Markdown
Contributor Author

Both tests pass without a fix - could you please verify?

@kubaflo This issue reproduces only on iOS 26 and does not occur on iOS 18. I verified this locally using the latest main branch—the test fails without the fix and passes with the fix on iOS 26, while it passes even without the fix on iOS 18.

iOS 26 Test Results:

Without Fix With Fix

@kubaflo
Copy link
Copy Markdown
Contributor

kubaflo commented Apr 7, 2026

Code Review — PR #34013

Independent Assessment

What this changes: Adds an iOS 26-specific debounce in CarouselViewController2.UpdateFromPosition() using CancellationTokenSource + Task.Delay(100) + MainThread.BeginInvokeOnMainThread. On iOS <26, behavior is unchanged. Removes iOS 26 skip guards from two existing tests and updates a snapshot.

Inferred motivation: iOS 26 changed UICollectionView.ScrollToItem() to fire intermediate scroll position callbacks during animation. These callbacks re-trigger UpdateFromPosition() with stale positions, interrupting the target scroll.

Reconciliation with PR Narrative

Agreement: ✅ Code matches description. The debounce pattern is appropriate — iOS 26 fires intermediate callbacks that iOS 18 didn't, and the 100ms delay lets the scroll animation settle before processing the final position. Author tested on all 4 platforms.


Findings

✅ Good — CancellationTokenSource debounce is correct

  • old?.Cancel(); old?.Dispose() before creating new CTS — proper lifecycle
  • Token passed to both Task.Delay and ContinueWith — cancellation works at both stages
  • OperationCanceledException caught — expected when superseded by newer call
  • State re-validation inside MainThread.BeginInvokeOnMainThread callback (checking InitialPositionSet, ItemsView, ItemsSource) — avoids stale/disposed state after delay ✅
  • Follows the exact same pattern as existing UpdateInitialPosition() method

✅ Good — Cleanup in Dispose

  • _scrollDebounce?.Cancel(); _scrollDebounce?.Dispose(); _scrollDebounce = null; in Dispose(bool) — prevents dangling delayed callbacks after disposal
  • PublicAPI.Unshipped.txt updated for both net-ios and net-maccatalyst — correct since this file compiles for both platforms

✅ Good — Test changes

  • Removed iOS 26 skip guards from Issue16020 and Issue17283 — these tests should now pass on iOS 26 since the underlying fix addresses the regression
  • Updated iOS 26 snapshot for CarouselViewShouldScrollToRightPosition

⚠️ Issue — Stale comment about delay value

The comment says:

"A slightly longer delay than UpdateInitialPosition's 100ms was empirically chosen"

But the actual delay is Task.Delay(100, token) — the same 100ms, not longer. This comment was accurate when the delay was 200ms (earlier revision) but is now stale after the value was reduced.

Suggestion: Update comment to:

// On iOS 26, UICollectionView can emit intermediate scroll callbacks before settling.
// A 100ms delay (matching UpdateInitialPosition) was empirically chosen to ensure
// the scroll operation runs after those intermediate callbacks complete.

💡 Suggestion — Match codebase pattern for iOS 26 version check

The PR uses OperatingSystem.IsIOSVersionAtLeast(26) alone. While this technically works on MacCatalyst (returns true for MacCatalyst 26+), the established codebase pattern for iOS 26 checks consistently uses the dual form:

if (OperatingSystem.IsIOSVersionAtLeast(26) || OperatingSystem.IsMacCatalystVersionAtLeast(26))

Examples: NavigationRenderer.cs:958, ShellPageRendererTracker.cs:296,347,471,581,779, ShellSectionRenderer.cs:836, ShellItemRenderer.cs:133, ShellItemTransition.cs:23 — all use the dual pattern.

For consistency and future-proofing, consider matching this pattern. It's a no-op change functionally but keeps the codebase convention uniform.

ℹ️ Note — async void is an existing pattern

UpdateFromPosition() changed from void to async void. While async void is generally risky (unhandled exceptions crash the process), this class already uses async void in ViewDidLayoutSubviews() and AttachingToWindow(). The OperationCanceledException is caught, and the MainThread.BeginInvokeOnMainThread callback follows the same fire-and-forget pattern as UpdateInitialPosition. This is consistent.

ℹ️ Note — CI Status

Failing checks: Helix Unit Tests (Windows Debug) and Integration Tests (Build macOS). These appear to be infrastructure-related flaky failures, not related to the CarouselView iOS changes.


Devil's Advocate

  • "Is 100ms enough?" — The existing UpdateInitialPosition uses the same value and works. The debounce CancellationToken ensures that rapid re-calls are properly superseded, so even if 100ms isn't quite long enough in some edge case, the final call wins.
  • "Could this introduce scroll lag?" — The 100ms delay only applies to iOS 26+ UpdateFromPosition (programmatic position changes), not to user-initiated scrolling. The lag is imperceptible to users.
  • "What about MacCatalyst?" — The file compiles for both iOS and MacCatalyst. IsIOSVersionAtLeast(26) returns true on MacCatalyst 26+. The debounce activates on both platforms, which is correct since MacCatalyst shares the same UICollectionView behavior.

Verdict: LGTM with minor nits

Confidence: high
Summary: Correct fix for a real iOS 26 regression. The CancellationTokenSource debounce pattern is well-implemented, follows existing codebase patterns, and properly cleans up in Dispose. Two minor suggestions: (1) fix the stale delay comment, (2) consider matching the dual version check pattern. Neither blocks merge.

Review performed by Copilot CLI using the code-review skill

@kubaflo kubaflo added s/agent-fix-implemented PR author implemented the agent suggested fix s/agent-suggestions-implemented Maintainer applies when PR author adopts agent's recommendation and removed s/agent-fix-implemented PR author implemented the agent suggested fix labels Apr 7, 2026
@kubaflo kubaflo changed the base branch from main to inflight/current April 7, 2026 11:43
@kubaflo kubaflo merged commit 3bb3a8c into dotnet:inflight/current Apr 7, 2026
31 of 36 checks passed
PureWeen pushed a commit that referenced this pull request Apr 8, 2026
…ast item (#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes #33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
PureWeen pushed a commit that referenced this pull request Apr 14, 2026
…ast item (#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes #33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
@PureWeen PureWeen mentioned this pull request Apr 14, 2026
devanathan-vaithiyanathan pushed a commit to Tamilarasan-Paranthaman/maui that referenced this pull request Apr 21, 2026
…ast item (dotnet#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes dotnet#33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
Ahamed-Ali pushed a commit that referenced this pull request Apr 22, 2026
…ast item (#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes #33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
PureWeen pushed a commit that referenced this pull request Apr 22, 2026
…ast item (#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes #33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
PureWeen pushed a commit that referenced this pull request Apr 28, 2026
…ast item (#34013)

<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Issue Detail
CarouselView fails to scroll to the last item on iOS 26 when using
programmatic navigation. Clicking "Go to LastItem" button navigates to
the next item instead of the last item. The issue is iOS 26-specific and
does not occur on iOS 18. The issue does not reproduce when breakpoints
are used.

### Root Cause
iOS 26 changed the behavior of UICollectionView.ScrollToItem() to fire
intermediate scroll position callbacks during the animation, whereas iOS
18 only fired callbacks at the start and end. These intermediate
callbacks trigger MapPosition in the handler, which calls
`UpdateFromPosition()` again with an incorrect intermediate position
value, interrupting the original scroll to the target position.

### Description of Change
Added iOS 26-specific fix in UpdateFromPosition() method that introduces
a 100ms delay using `Task.Delay().ContinueWith()` and
`MainThread.BeginInvokeOnMainThread()` pattern. This debounces rapid
position callbacks, allowing the scroll animation to complete before
processing the position update. The fix only applies to iOS 26+ and does
not affect other platforms or iOS versions.

### Why Tests were not added

**Regarding test case**, existing test cases
(CarouselViewShouldScrollToRightPosition and
CarouselViewiOSCrashPreventionTest) already cover this scenario, so no
new tests were added in this PR.

### Tested the behavior in the following platforms
 
- [x] Android
- [x] Windows
- [x] iOS
- [x] Mac

**Reference:**

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Handlers/Items2/iOS/CarouselViewController2.cs#L589

### Issues Fixed

Fixes #33770

### Screenshots

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/00071186-60a8-4001-922e-bf62828e3cb9">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/506683bc-868a-42b2-9f79-862c4aa8fa37">)
|

| Before Issue Fix | After Issue Fix |
|----------|----------|
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/04743403-755f-44a9-8623-05efc3a1dcb4">
| <video width="300" height="600"
src="https://github.com/user-attachments/assets/233e2387-a602-4b46-80bc-350965e5fa94">)
|

---------
PureWeen added a commit that referenced this pull request Apr 29, 2026
## Blazor
- Fix: Filter precompressed RCL assets from MAUI Blazor Hybrid APKs by
@mattleibow in #33917
  <details>
  <summary>🔧 Fixes</summary>

- [.NET MAUI Blazor Hybrid App should not precompress
assets](#33773)
  </details>

- [Windows] Fix for Runtime error when closing external window with WPF
Webview Control by @BagavathiPerumal in
#34006
  <details>
  <summary>🔧 Fixes</summary>

- [Runtime error when closing external window with WPF Webview
Control](#32944)
  </details>

## Button
- [Android] ImageButton CornerRadius not being applied - fix by @kubaflo
in #30074
  <details>
  <summary>🔧 Fixes</summary>

- [ImageButton CornerRadius not being applied on
Android](#23854)
  </details>

- Fix Disabled visual state ignored when Button has locally-set
BackgroundColor/TextColor by @Dhivya-SF4094 in
#34444
  <details>
  <summary>🔧 Fixes</summary>

- [[regression/9.0] VisualState "Disabled" is not properly applied for
Button with custom
appearance](#34363)
  </details>

## CollectionView
- Fix CollectionView grid spacing updates for first row and column by
@KarthikRajaKalaimani in #34527
  <details>
  <summary>🔧 Fixes</summary>

- [[MAUI] I2_Vertical grid for horizontal Item Spacing and Vertical Item
Spacing - horizontally updating the spacing only applies to the second
column](#34257)
  </details>

- Fix CollectionView record struct selection on Windows by
@jeremy-visionaid in #33488

- [Android] Ensure disconnected ItemsViewHandler doesn't hold onto the
items source by @filipnavara in
#24610
  <details>
  <summary>🔧 Fixes</summary>

- [Crash on NullReferenceException with measurement cells in
CollectionView](#24304)
  </details>

- [Windows] Fixed VisualState Setters not working properly for
CollectionView by @Dhivya-SF4094 in
#27230
  <details>
  <summary>🔧 Fixes</summary>

- [VisualState Setters not working properly on Windows for a
CollectionView](#27086)
- [[regression/8.0.3] [Windows][CollectionView]Label Disappear when set
Style in
ContentPage.Resources](#19209)
- [[Windows] Label style defined as ContentPage Resource doesn't
propagate to
CollectionView](#18701)
  </details>

- [Windows] Fixed Margin doesn't work inside CollectionView EmptyView by
@Dhivya-SF4094 in #29897
  <details>
  <summary>🔧 Fixes</summary>

- [Margin doesn't work inside CollectionView
EmptyView](#8494)
  </details>

- [Android, Windows] Fix CarouselView PreviousPosition/PreviousItem
incorrect during animated ScrollTo() by @praveenkumarkarunanithi in
#34570
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] CurrentItemChangedEventArgs.PreviousItem and
PositionChangedEventArgs.PreviousPosition Not Updating Correctly When
Using ScrollTo or Setting
Position](#29544)
  </details>

- [iOS] CarouselView2: Update internal scroll indicators for
compositional layout by @SubhikshaSf4851 in
#33639
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS] Horizontal Scroll Bar Not Visible on CarouselView
(CV2)](#29390)
  </details>

- [CarouselViewHandler2] Fir fox CurrentItem does not work when
ItemSpacing is set by @SyedAbdulAzeemSF4852 in
#32135
  <details>
  <summary>🔧 Fixes</summary>

- [[CarouselViewHandler2] CurrentItem does not work when ItemSpacing is
set](#32048)
  </details>

- [iOS] Fix for Incorrect Scroll in Loop Mode When CurrentItem Is Not
Found in ItemsSource by @SyedAbdulAzeemSF4852 in
#32141
  <details>
  <summary>🔧 Fixes</summary>

- [[Android & iOS] Setting an invalid CurrentItem causes scroll to last
item in looped
CarouselView](#32139)
  </details>

- [Android] IndicatorView: Add TalkBack accessibility descriptions for
indicators by @praveenkumarkarunanithi in
#31775
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] IndicatorView does not convey correct accessibility
information](#31446)
  </details>

- [iOS, macOS] Fixed CollectionView KeepLastItemInView Not Updating
Correctly When Items Are Added Dynamically by @NanthiniMahalingam in
#32191
  <details>
  <summary>🔧 Fixes</summary>

- [[.NET10] I9 - Scroll_Position - "KeepLastItemInView" does not keep
the last item at the end of the displayed list when adding new
items.](#31825)
  </details>

- [Windows, Android] Resolved issue with dynamic Header/Footer
reassignment in CollectionView. by @prakashKannanSf3972 in
#28403
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows, Android] Toggling Header/Footer in CollectionView
Dynamically is not working](#27959)
- [CollectionView HeaderTemplate and FooterTemplate are not displayed
when ItemsSource is initially set to
null](#28337)
- [[Android] Header and Footer Not Visible in CollectionView When
EmptyView is Selected
First](#28351)
  </details>

- [Android] Fix CollectionView inside disabled RefreshView blocks scroll
by @Vignesh-SF3580 in #34702
  <details>
  <summary>🔧 Fixes</summary>

- [C6-The C6 page cannot scroll on Windows and Android
platforms.](#34666)
  </details>

- [Android] CollectionView: Fix SelectedItem visual state not applying
when re-selecting same item by @KarthikRajaKalaimani in
#31591
  <details>
  <summary>🔧 Fixes</summary>

- [CollectionView - SelectedItem visual state manager not
working](#20062)
  </details>

- [Windows] Fixed CollectionView.EmptyView can not be removed by setting
it to Null by @Dhivya-SF4094 in
#29487
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows] CollectionView.EmptyView can not be removed by setting it
to Null](#18657)
- [[Windows] EmptyViewTemplate Not Working in
CarouselView](#29463)
- [EmptyViewTemplate does not do
anything](#18551)
- [[MAUI] I5_EmptyView - The data template selector cannot display the
correct string.](#23330)
  </details>

- [iOS] Support for IsSwipeEnabled on CarouselView2 by @kubaflo in
#29996
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS] IsSwipeEnabled Not Working on CarouselView
(CV2)](#29391)
  </details>

- [iOS, MacOS] Fixed FlowDirection not working on Header/Footer in
CollectionView by @Dhivya-SF4094 in
#32775
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS, MacOS] FlowDirection not working on Header/Footer in
CollectionView](#32771)
  </details>

- [iOS] CollectionView: Fix drag-and-drop reordering into empty groups
by @SuthiYuvaraj in #34151
  <details>
  <summary>🔧 Fixes</summary>

- [CollectionView Drag and Drop Reordering Can't Drop in Empty
Group](#12008)
  </details>

- [Android] CollectionView: Fix drag-and-drop reordering into empty
groups by @SuthiYuvaraj in #31867
  <details>
  <summary>🔧 Fixes</summary>

- [CollectionView Drag and Drop Reordering Can't Drop in Empty
Group](#12008)
  </details>

- [iOS] Fix vertical CarouselView MandatorySingle snapping on iOS by
@Vignesh-SF3580 in #34700
  <details>
  <summary>🔧 Fixes</summary>

- [CarouselView vertical snap points ignored on iOS with
Microsoft.Maui.Controls v10.0.20 (regression from
v9.0.120)](#33308)
  </details>

- [iOS26] Fix CarouselView scrolling to wrong item when navigating to
last item by @Vignesh-SF3580 in
#34013
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS 26] CarouselView does not scroll to the correct last
item](#33770)
  </details>

- Fixed the OnPlatform does not work for header property in Collection
view by @NanthiniMahalingam in #28935
  <details>
  <summary>🔧 Fixes</summary>

- [OnPlatform does not work in Header of
CollectionView](#25124)
  </details>

- [Android] [Candidate branch] Fix
VerifySelectedItemClearsOnNullAssignment,
CollectionViewSelectionShouldClear, SelectedItemVisualIsCleared UI test
failure on Android by @KarthikRajaKalaimani in
#34928

## DateTimePicker
- [iOS] Fix for DatePicker FlowDirection Not Working on iOS by
@SyedAbdulAzeemSF4852 in #30193
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS] DatePicker FlowDirection Not Working on
iOS](#30065)
  </details>

## Drawing
- [Shapes] Line: Fix asymmetric Stretch.None path translation when
right/bottom edge overflows by @NirmalKumarYuvaraj in
#34385
  <details>
  <summary>🔧 Fixes</summary>

- [Line coordinates not computed
correctly](#11404)
- [Lines not drawing
correctly](#26961)
  </details>

- [Android] Fixed GraphicsView drawable is visible outside the canvas by
@NirmalKumarYuvaraj in #28353
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] GraphicsView, The drawn image can also be visible outside
the canvas](#20834)
  </details>

- Fixed Custom Drawable does not support binding by @NirmalKumarYuvaraj
in #29442
  <details>
  <summary>🔧 Fixes</summary>

- [Custom IDrawable control does not databind to a model property when
used inside a CollectionView
ItemTemplate](#20991)
  </details>

- Added a support for GradientBrushes on Shape.Stroke by @kubaflo in
#22208
  <details>
  <summary>🔧 Fixes</summary>

- [GradientBrushes are not supported on
Shape.Stroke](#21983)
  </details>

## Editor
- Fixed Editor HorizontalTextAlignment does not update at run time by
@NirmalKumarYuvaraj in #25129
  <details>
  <summary>🔧 Fixes</summary>

- [Editor HorizontalTextAlignment Does not
Works.](#10987)
- [[iOS/MacOs] Right-To-Left (RTL) alignment is not applied to Editor
placeholder](#30052)
  </details>

- [Windows] Fixed Entry Editor placeholder Text CharacterSpacing by
@SubhikshaSf4851 in #30324
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows] CharacterSpacing not applied to Placeholder text in Entry
and Editor controls](#30071)
  </details>

## Entry
- [Windows] Fix fo setting an Entry's Keyboard to Date causes it to be
interpreted as a password input by @SyedAbdulAzeemSF4852 in
#29344
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows] Entry Keyboad-Type "Date" results in
Password-Entry](#28975)
  </details>

- [Android] Exception thrown when give more than 5000 characters to the
Text property of Entry. by @KarthikRajaKalaimani in
#30242
  <details>
  <summary>🔧 Fixes</summary>

- [Android crash when Entry has >5000
characters](#30144)
  </details>

## Essentials
- Bump MonoApiToolsMSBuildTasksPackageVersion to 0.5.0 and ship
Essentials.AI public APIs by @mattleibow via @Copilot in
#34574

- [Mac] DeviceDisplay.KeepScreenOn not being respected on Mac OS by
@HarishwaranVijayakumar in #32708
  <details>
  <summary>🔧 Fixes</summary>

- [[Mac Catalyst] DeviceDisplay.KeepScreenOn not being respected on Mac
OS](#26059)
  </details>

## Flyoutpage
- [Windows] FlyoutPage: update CollapseStyle at runtime by
@devanathan-vaithiyanathan in #29927
  <details>
  <summary>🔧 Fixes</summary>

- [Flyout Page SetCollapseStyle doesn't have any
change](#18200)
  </details>

## Gestures
- [Android] Fix for TapGestureRecognizer doesn't fire by
@HarishwaranVijayakumar in #34497
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] TapGestureRecognizer doesn't
fire](#5825)
  </details>

## Image
- [Android] Fix Share.RequestAsync SecurityException on Android 10+
caused by missing ClipData by @HarishwaranVijayakumar in
#34417
  <details>
  <summary>🔧 Fixes</summary>

- [[Bug] Share.RequestAsync throws java.lang.SecurityException
(uid=1000) on Android 10+ due to missing
intent.ClipData](#34370)
  </details>

- [Windows]Fixed the MauiImage with logical name containing path issue
by @sheiksyedm in #32864
  <details>
  <summary>🔧 Fixes</summary>

- [MauiImage with LogicalName containing path - is not working on
Windows](#32356)
  </details>

- [Android, Windows & iOS] Fix Downsize/ScaleImage to maintain aspect
ratio and prevent upscaling by @SyedAbdulAzeemSF4852 in
#30808
  <details>
  <summary>🔧 Fixes</summary>

- [[Android & Windows] In GraphicsView, the aspect ratio is not
maintained when Downsize is called with both maxWidth and
maxHeight](#30803)
  </details>

## Label
- [iOS , macOS] Fixed Label text cropping when a width request is
specified on the label inside a VerticalStackLayout with specified width
request by @NanthiniMahalingam in
#29166
  <details>
  <summary>🔧 Fixes</summary>

- [Label text gets cropped when a width request is specified on the
label inside a
VerticalStackLayout](#28660)
- [[iOS] Label with a fixed WidthRequest has wrong
height](#26644)
  </details>

- [Android] Fix Label word wrapping clips text depending on alignment
and layout options by @Dhivya-SF4094 in
#34533
  <details>
  <summary>🔧 Fixes</summary>

- [Bug: Android Label word wrapping clips text depending on alignment
and layout options](#34459)
  </details>

- LineHeight and decorations for HTML Label - fix by @kubaflo in
#31202
  <details>
  <summary>🔧 Fixes</summary>

- [LineHeight with HTML Label not
working](#22193)
  - [lineheight is broken ](#22197)
  </details>

- [iOS] Fix Label with TailTruncation not rendering after
empty-to-non-empty text transition by @kubaflo in
#34812
  <details>
  <summary>🔧 Fixes</summary>

- [Label with LineBreakMode="TailTruncation" does not render text if
initial Text is null or empty on first render
(iOS)](#34591)
  </details>

## Layout
- [Android] Fix overflowing children clipped when parent Opacity < 1 by
@SyedAbdulAzeemSF4852 in #34565
  <details>
  <summary>🔧 Fixes</summary>

- [Maui Android parent view inappropriately creates clipping mask when
its opacity is less than 1, cropping out
children](#22038)
  </details>

- Fixed the FlexLayout reverse issue with the AlignContent by
@Ahamed-Ali in #32134
  <details>
  <summary>🔧 Fixes</summary>

- [FlexLayout alignment issue when Wrap is set to Reverse and
AlignContent is set to SpaceAround, SpaceBetween or
SpaceEvenly](#31565)
  </details>

- [iOS/Mac] Fixed BoxView in AbsoluteLayout did not return to its
default AutoSize for Height and Width after reset by @Dhivya-SF4094 in
#31648
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS, Catalyst] BoxView in AbsoluteLayout does not return to default
AutoSize for Height/Width after
reset](#31496)
  </details>

## Map
- [Windows] Implement WinUI 3 MapControl handler using Azure Maps by
@jfversluis in #34138

## Modal
- [Android] PopToRootAsync for modal pages - improvements by @kubaflo in
#26851
  <details>
  <summary>🔧 Fixes</summary>

- [Shell PopToRootAsync doesn't happen instantly - previous pages flash
quickly. Only happens in NET
9](#26846)
  </details>

- [Android] Fix HideSoftInputOnTapped doesn't work on Modal Pages by
@HarishwaranVijayakumar in #34770
  <details>
  <summary>🔧 Fixes</summary>

- [HideSoftInputOnTapped doesn't work on Modal
Pages](#34730)
  </details>

## Navigation
- [iOS] Alert popup may be displayed on wrong window when modal page
navigation is in progress - fix by @kubaflo in
#31016
  <details>
  <summary>🔧 Fixes</summary>

- [Alert popup may be displayed on wrong window when modal page
navigation is in progress on
iOS/MacOS](#30970)
  </details>

- [Android] Page: Fix OnNavigatedTo called twice when NavigationPage is
FlyoutPage Detail by @KarthikRajaKalaimani in
#31931
  <details>
  <summary>🔧 Fixes</summary>

- [NavigationPage and FlyoutPage both call OnNavigatedTo, so it is
called twice](#23902)
  </details>

## Picker
- Fixed the Picker didn't dismiss it when tapping outside on iOS and
MacCatalyst platform. by @KarthikRajaKalaimani in
#30067
  <details>
  <summary>🔧 Fixes</summary>

- [[regression/8.0.3] iOS Picker dismiss does not work when clicking
outside of the Picker](#19168)
  </details>

- [Windows] Fixed Picker items width wont resize back by
@SubhikshaSf4851 in #33042
  <details>
  <summary>🔧 Fixes</summary>

- [Picker items width won't resize back when its container window gets
resized down.](#32984)
  </details>

## RadioButton
- Fix TalkBack not correctly narrating RadioButtons with Content by
@SubhikshaSf4851 in #34521
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] TalkBack does not correctly narrate RadioButtons with
Content](#34322)
  </details>

## SafeArea
- [Android] Fix SafeAreaShouldWorkOnAllShellTabs test failure on API 36
by @praveenkumarkarunanithi in #34239

## ScrollView
- [iOS] Preserve ScrollView offsets when Orientation changes to Neither
by @Vignesh-SF3580 in #34672
  <details>
  <summary>🔧 Fixes</summary>

- [Incorrect implementation of
ScrollView.Orientation](#34583)
  </details>

## Searchbar
- [Android] Fix SearchBar text bleeding between instances after
navigation by @SyedAbdulAzeemSF4852 in
#34703
  <details>
  <summary>🔧 Fixes</summary>

- [MAUI Android: SearchBar copies content from one to the
other](#20348)
  </details>

- Fixed SearchBar CursorPosition and SelectionLength not updating when
typing by @Dhivya-SF4094 in #34347
  <details>
  <summary>🔧 Fixes</summary>

- [SearchBar - CursorPosition and SelectionLength are not updated when
the user types](#30779)
  </details>

## SearchBar
- [Windows] Fixed SearchHandler issues by @Tamilarasan-Paranthaman in
#29520
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows] SearchHandler APIs are not functioning
properly](#29493)
  </details>

## Shell
- [iOS, Mac] Fix for Background set to Transparent doesn't have the same
behavior as BackgroundColor Transparent by @HarishwaranVijayakumar in
#32245
  <details>
  <summary>🔧 Fixes</summary>

- [Background set to Transparent doesn't have the same behavior as
BackgroundColor =
Transparent](#22769)
  </details>

- [iOS] Fix App crash with NullReferenceException in
ShellSectionRenderer by @devanathan-vaithiyanathan in
#32109
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS] App crash with NullReferenceException in
ShellSectionRenderer](#31961)
  </details>

- [Android] Fixed back button icon selection logic in
ShellToolbarTracker by @kubaflo in
#32080
  <details>
  <summary>🔧 Fixes</summary>

- [IconOverride in Shell.BackButtonBehavior does not
work.](#32050)
  </details>

- Fix TabBarIsVisible Not Updating Dynamically When Set on ShellContent
by @Vignesh-SF3580 in #33090
  <details>
  <summary>🔧 Fixes</summary>

- [Shell.TabBarIsVisible is not updated dynamically at
runtime](#32994)
  </details>

- [iOS, macOS] Shell: Fix RTL flow direction for flyout, menu cells, tab
bar, and Locked flyout position by @NanthiniMahalingam in
#32701
  <details>
  <summary>🔧 Fixes</summary>

- [[iOS, Mac Catalyst] Shell Flyout and Content Do Not Fully Support
RightToLeft (RTL)](#32419)
  </details>

- [IOS] Inconsistent Resize Behavior for Header/Footer - fix by @kubaflo
in #28713
  <details>
  <summary>🔧 Fixes</summary>

- [[IOS, Mac] Inconsistent Resize Behavior for
Header/Footer](#26397)
- [Enable Shell Flyout Header/Footer resize tests on
iOS/Catalyst](#33501)
  </details>

- [Android] Fix for SearchHandler retaining previous page SearchView
data in pages within Shell sections by @BagavathiPerumal in
#29545
  <details>
  <summary>🔧 Fixes</summary>

- [[Shell][Android] The truth is out there...but not on top tab search
handlers](#8716)
  </details>

- [Android] Fix empty space above TabBar after navigating back when
TabBar visibility is toggled by @praveenkumarkarunanithi in
#34324
  <details>
  <summary>🔧 Fixes</summary>

- [Empty space appears above TabBar after navigating back when TabBar
visibility is toggled](#33703)
- [Grid with SafeAreaEdges=Container has incorrect size when tab bar
appears](#34256)
  </details>

## SwipeView
- [Android] SwipeView: Use MeasureSpecMode.Exactly for SwipeItem layout
to fix text visibility by @Ahamed-Ali in
#27399
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] Right SwipeView items are not visible in the
SwipeView.](#27367)
  </details>

- [Android] Prevent the tap that closes an open SwipeView from being
propagated to children by @sjordanGSS in
#24275
  <details>
  <summary>🔧 Fixes</summary>

- [Tapping to close a SwipeView will activate TapGestureRecognizers on
.Content](#23921)
  </details>

## Switch
- [iOS & Mac] Fix for SearchHandler retains previous page state when
switching top tabs by @BagavathiPerumal in
#34735
  <details>
  <summary>🔧 Fixes</summary>

- [[Shell] [iOS & Mac] SearchHandler retains previous page state when
switching top tabs](#34693)
  </details>

## TabbedPage
- [Android] Fixed NullReferenceException in app with TabBar after
returning from minimized state by @NirmalKumarYuvaraj in
#34779
  <details>
  <summary>🔧 Fixes</summary>

- [NullReferenceException in app with TabBar after returning from
minimized state](#34720)
  </details>

## Titlebar
- Fixed BindingContext of the Window TitleBar is not being passed on to
its child content. by @NirmalKumarYuvaraj in
#30080
  <details>
  <summary>🔧 Fixes</summary>

- [The BindingContext of the Window TitleBar is not being passed on to
its child content.](#24831)
  </details>

- [Windows/Mac] Fix RTL FlowDirection causes overlap with native window
control buttons in TitleBar by @devanathan-vaithiyanathan in
#30400
  <details>
  <summary>🔧 Fixes</summary>

- [[Windows, Mac] RTL FlowDirection causes overlap with native window
control buttons in
TitleBar](#30399)
  </details>

## WebView
- [Windows] Fix WebView background color not being applied by
@SubhikshaSf4851 in #34599
  <details>
  <summary>🔧 Fixes</summary>

- [WebView background color has changed after update, can't
override.](#34518)
  </details>

- [Android] Fix for WebView/HybridWebView briefly flashes full screen
before layout completes by @praveenkumarkarunanithi in
#33207
  <details>
  <summary>🔧 Fixes</summary>

- [[Android] HybridWebView briefly resizes to full screen when page is
opened before snapping back to correct
size](#31475)
  </details>

## Xaml
- Improved style inheritance by @kubaflo in
#31317
  <details>
  <summary>🔧 Fixes</summary>

- [Styles based on a style that is based on another style that uses
AppThemeBinding do not inherit properties
correctly.](#31280)
  </details>

- Fix for VisualStateManager Setter.TargetName failing when
ControlTemplate is applied by @BagavathiPerumal in
#33208
  <details>
  <summary>🔧 Fixes</summary>

- [Setter.TargetName + ControlTemplate
crash](#26977)
  </details>


<details>
<summary>🧪 Testing (4)</summary>

- [Testing] Additional Feature Matrix Event Test Cases for Slider and
ScrollView by @nivetha-nagalingam in
#34352
- [Testing] Fixed Build error on inflight/ candidate PR 34885 by
@NafeelaNazhir in #34891
- [Testing] Fixed UI test image failure in PR 34885 - [13/4/2026] by
@NafeelaNazhir in #34933
- Fixed test failure - CursorPositionUpdatesWhenSearchBarGainsFocus by
@Dhivya-SF4094 in #34938

</details>

<details>
<summary>📦 Other (3)</summary>

- Fix Loaded event not called for MAUI View added to native View by
@NirmalKumarYuvaraj in #34345
  <details>
  <summary>🔧 Fixes</summary>

- [Loaded event not called for MAUI View added to native
View](#34310)
  </details>
- Add public IAlertManager and IAlertManagerSubscription interfaces by
@Redth in #34228
  <details>
  <summary>🔧 Fixes</summary>

- [Alert/Dialog system (`DisplayAlert`, `DisplayActionSheet`,
`DisplayPromptAsync`) needs a public extensibility
point](#34104)
  </details>
- Fix crash when displaying alerts on unloaded pages by @kubaflo in
#33288

</details>

<details>
<summary>📝 Issue References</summary>

Fixes #5825, Fixes #8494, Fixes #8716, Fixes #10987, Fixes #11404, Fixes
#12008, Fixes #18200, Fixes #18551, Fixes #18657, Fixes #18701, Fixes
#19168, Fixes #19209, Fixes #20062, Fixes #20348, Fixes #20834, Fixes
#20991, Fixes #21983, Fixes #22038, Fixes #22193, Fixes #22197, Fixes
#22769, Fixes #23330, Fixes #23854, Fixes #23902, Fixes #23921, Fixes
#24304, Fixes #24831, Fixes #25124, Fixes #26059, Fixes #26397, Fixes
#26644, Fixes #26846, Fixes #26961, Fixes #26977, Fixes #27086, Fixes
#27367, Fixes #27959, Fixes #28337, Fixes #28351, Fixes #28660, Fixes
#28975, Fixes #29390, Fixes #29391, Fixes #29463, Fixes #29493, Fixes
#29544, Fixes #30052, Fixes #30065, Fixes #30071, Fixes #30144, Fixes
#30399, Fixes #30779, Fixes #30803, Fixes #30970, Fixes #31280, Fixes
#31446, Fixes #31475, Fixes #31496, Fixes #31565, Fixes #31825, Fixes
#31961, Fixes #32048, Fixes #32050, Fixes #32139, Fixes #32356, Fixes
#32419, Fixes #32771, Fixes #32944, Fixes #32984, Fixes #32994, Fixes
#33308, Fixes #33501, Fixes #33703, Fixes #33770, Fixes #33773, Fixes
#34104, Fixes #34256, Fixes #34257, Fixes #34310, Fixes #34322, Fixes
#34363, Fixes #34370, Fixes #34459, Fixes #34518, Fixes #34583, Fixes
#34591, Fixes #34666, Fixes #34693, Fixes #34720, Fixes #34730

</details>

**Full Changelog**:
main...inflight/candidate
@github-actions github-actions Bot locked and limited conversation to collaborators May 8, 2026
@kubaflo kubaflo added s/agent-gate-failed AI could not verify tests catch the bug s/agent-fix-implemented PR author implemented the agent suggested fix labels May 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration s/agent-changes-requested AI agent recommends changes - found a better alternative or issues s/agent-fix-implemented PR author implemented the agent suggested fix s/agent-fix-win AI found a better alternative fix than the PR s/agent-gate-failed AI could not verify tests catch the bug s/agent-reviewed PR was reviewed by AI agent workflow (full 4-phase review) s/agent-suggestions-implemented Maintainer applies when PR author adopts agent's recommendation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[iOS 26] CarouselView does not scroll to the correct last item

10 participants