Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Animation not picking up the current set/animated value #14262

Closed
eriklimakc opened this issue Nov 3, 2023 · 14 comments
Closed

Animation not picking up the current set/animated value #14262

eriklimakc opened this issue Nov 3, 2023 · 14 comments
Assignees
Labels
area/skia ✏️ Categorizes an issue or PR as relevant to Skia difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI kind/bug Something isn't working platform/all Categorizes an issue or PR as relevant to the all platforms project/animations 🎡 Categorizes an issue or PR as relevant to animations triage/potentially-fixed Categorizes an issue as potentially fixed by some unlinked PR, fix needs to be verified

Comments

@eriklimakc
Copy link
Contributor

eriklimakc commented Nov 3, 2023

Current behavior

This happens in all platforms except Windows.

Skia_vs_Win_issue

Apparently when the animation code is executed, the Text, Header, and Placeholder are quickly set to their default state and after that they are animated.

Expected behavior

No response

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

Checkout this branch on Uno.Themes and launch on non-windows platform.

For the FilledTextBoxStyle the animation is defined in: TextBox.xaml#L420.

Workaround

In order to test I replaced the animation for Setters like below and we don't have that behavior.

<Setter Target="HeaderElement_CompositeTransform.TranslateY"
	Value="-11" />
<Setter Target="ContentElement_CompositeTransform.TranslateY"
	Value="{Binding RelativeSource={RelativeSource TemplatedParent}, Converter={StaticResource EmptyToFocusedContentCompositeTransformTranslateY}, TargetNullValue=0, FallbackValue=0}" />
<Setter Target="PlaceholderElement_CompositeTransform.TranslateY"
	Value="{Binding RelativeSource={RelativeSource TemplatedParent}, Converter={StaticResource EmptyToFocusedContentCompositeTransformTranslateY}, TargetNullValue=0, FallbackValue=0}" />
<Setter Target="HeaderElement_CompositeTransform.ScaleX"
	Value="0.7" />
<Setter Target="HeaderElement_CompositeTransform.ScaleY"
	Value="0.7" />

Environment

Works on UWP/WinUI: YES
NuGet package version(s): [email protected]
Affected platforms: This happens in all platforms except Windows.

Anything else we need to know?

Comment left by @Xiaoy312:

"At a glance, this seems like an issue with the animation not picking up the current set/animated value as the starting point for the (start...end) animation. This is not probably not something we will fix here, but in uno."

@eriklimakc eriklimakc added kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Nov 3, 2023
@agneszitte
Copy link
Contributor

cc @MartinZikmund / @dr1rrb / @Xiaoy312

@Youssef1313 Youssef1313 added project/animations 🎡 Categorizes an issue or PR as relevant to animations platform/all Categorizes an issue or PR as relevant to the all platforms labels Nov 3, 2023
@MartinZikmund MartinZikmund added difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI area/skia ✏️ Categorizes an issue or PR as relevant to Skia and removed triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Nov 14, 2023
@agneszitte
Copy link
Contributor

agneszitte commented Nov 20, 2023

@jeromelaban / @MartinZikmund this issue is kind of blocking @eriklimakc's PR here unoplatform/Uno.Themes#1014
(cc @kazo0, @Xiaoy312)

@kazo0
Copy link
Contributor

kazo0 commented Dec 13, 2023

Since this will cause a regression in the animation for all Uno platforms, I'd say it's definitely blocking us from merging unoplatform/Uno.Themes#1014

@kazo0
Copy link
Contributor

kazo0 commented Dec 19, 2023

@dr1rrb does this happen to look familiar at all? Currently blocking this unoplatform/Uno.Themes#974

@dr1rrb
Copy link
Member

dr1rrb commented Jan 10, 2024

No nothing familiar here, sorry :(

@Xiaoy312 Xiaoy312 self-assigned this Jan 10, 2024
@kazo0
Copy link
Contributor

kazo0 commented Feb 19, 2024

@Youssef1313 is it possible this will be fixed/related to any of the animation stuff you are doing recently?

@Youssef1313
Copy link
Member

I don't think what I have will fix it.

@Xiaoy312
Copy link
Contributor

looks like a regression from #11859 ?

@agneszitte
Copy link
Contributor

@eriklimakc I imagine this is still happening with latest versions?

@eriklimakc
Copy link
Contributor Author

Apparently fixed on latest Uno packages. Quickly tested all platforms and didn't see the issue anymore. You can also see on the stage site: https://salmon-rock-0cfebe70f-1014.eastus2.azurestaticapps.net/ from unoplatform/Uno.Themes#1014.

Will do more deep tests and update here on the issue.

cc @Xiaoy312 @agneszitte @kazo0

@Xiaoy312
Copy link
Contributor

unassigned me and close this, if your deep tests are positive.

@Youssef1313 Youssef1313 added the triage/potentially-fixed Categorizes an issue as potentially fixed by some unlinked PR, fix needs to be verified label May 15, 2024
@eriklimakc
Copy link
Contributor Author

eriklimakc commented May 15, 2024

Closing as not reproducible anymore.

@eriklimakc eriklimakc closed this as not planned Won't fix, can't repro, duplicate, stale May 15, 2024
@eriklimakc
Copy link
Contributor Author

@kazo0 It was fixed on 5.3.0-dev.906.

@Youssef1313
Copy link
Member

For reference, that version belongs to #16474

FYI @ramezgerges :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/skia ✏️ Categorizes an issue or PR as relevant to Skia difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI kind/bug Something isn't working platform/all Categorizes an issue or PR as relevant to the all platforms project/animations 🎡 Categorizes an issue or PR as relevant to animations triage/potentially-fixed Categorizes an issue as potentially fixed by some unlinked PR, fix needs to be verified
Projects
None yet
Development

No branches or pull requests

7 participants