[Windows]Fixed the ViewExtensions RotateYTo and RotateXTo with length 0 crashes

[Ahamed-Ali]

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!

Root Cause of the issue

• When RotateYTo is dynamically updated with a duration of 0, it causes the value calculation in Tweener class to result in NaN. This NaN value is then passed to the UpdatedTransformation method on Windows, where the subtraction between the PlaneProjection's Rotation property and the MAUI rotation leads to an ArgumentException.

https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Tweener.cs#L124

Description of Change

• If the calculated value is NaN, a defensive check is added to prevent invalid transformations from being applied. This ensures that when any rotation value is NaN (Not a Number), it is skipped. On subsequent updates, a valid value will be used, allowing the transformation to proceed as expected.

Issues Fixed

Fixes #18420

Tested the behaviour in the following platforms

• Android
• Windows
• iOS
• Mac

Screenshot

Before Issue Fix	After Issue Fix
CrashOccured.mp4	NoCrashwithFix.mp4

[dotnet-policy-service]

Hey there @@Ahamed-Ali! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

[jsuarezruiz]

/azp run MAUI-UITests-public

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull Request Overview

This PR fixes a crash on Windows when ViewExtensions.RotateYTo and RotateXTo are called with a duration of 0, which resulted in NaN values being passed to the transformation system causing ArgumentExceptions.

• Adds defensive NaN checks in the Windows transformation logic to prevent invalid transformations
• Includes comprehensive UI tests to validate the fix across multiple button taps with zero-duration rotations
• Fixes a variable reference issue in the transformation condition check

Reviewed Changes

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

File	Description
src/Core/src/Platform/Windows/TransformationExtensions.cs	Adds NaN validation and fixes variable reference in rotation condition
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue18420.cs	NUnit test that verifies multiple zero-duration rotations don't crash
src/Controls/tests/TestCases.HostApp/Issues/Issue18420.cs	Host app UI test page with button that triggers zero-duration RotateYTo calls

[jfversluis]

/rebase