unoplatform / uno

Open-source platform for building cross-platform native Mobile, Web, Desktop and Embedded apps quickly. Create rich, C#/XAML, single-codebase apps from any IDE. Hot Reload included! 90m+ NuGet Downloads!!
https://platform.uno
Apache License 2.0
9.01k stars 733 forks source link

Window Community Toolkit implicit animations do not work on android #17834

Open DavidM29 opened 3 months ago

DavidM29 commented 3 months ago

Current behavior

If page contains Windows Community Toolkit implicit animations, it does not get rendered at all on android. This bug might be on other platforms as well, however I have only tested it on windows and android.

Expected behavior

The page must get rendered and work as expected, if animations do not work for performance reasons it should not break the whole page.

How to reproduce it (as minimally and precisely as possible)

Run the app on windows, the page and animations work as expected. Run on android, button and rectangle does not get rendered at all. UnoApp1.zip

Workaround

Implicit animations can be created from code-behind using conditional compilation to make it not affect the page on android.

Works on UWP/WinUI

Yes

Environment

Other

NuGet package version(s)

"Uno.Sdk": "5.3.90" CommunityToolkit.WinUI.Animations: 8.0.240109

Affected platforms

Android

IDE

Visual Studio 2022

IDE version

17.10.5

Relevant plugins

No response

Anything else we need to know?

No response

MartinZikmund commented 3 months ago

Implicit animations are indeed not supported in Uno Platform yet. To work around this, you can keep them as WinUI specific using https://platform.uno/docs/articles/platform-specific-csharp.html and https://platform.uno/docs/articles/platform-specific-xaml.html

DavidM29 commented 3 months ago

Kind of related to this, can I check somewhere exactly which WCT controls work in uno, because I did have some similar experience with other WCT controls, for example Settings Expander, but I know it is tracked, however I could not find anything about wrap panel, so if it's not tracked I will open an issue for it.

d2dyno1 commented 1 month ago

Perhaps it's going to change with this PR? https://github.com/unoplatform/uno/pull/18267

MartinZikmund commented 1 month ago

@d2dyno1 Yes, but only for desktop targets for now