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

Update Rotate3DTransition.cs to fix flickering #12254

Merged
merged 1 commit into from
Jul 19, 2023
Merged

Conversation

yankun
Copy link
Contributor

@yankun yankun commented Jul 19, 2023

Fix flickering after transition has finished on Rotate3DTransition. This was originally contained in my previous PR, but the commit was lost.

What does the pull request do?

Fixes the rotate 3d transition by not falling back to previous values, which would render the control being rotated away visible for a very short amount of time.

What is the current behavior?

After the current visual has been rotated away, it will become visible for a very short amount of time, so the control looks like its flickering.

What is the updated/expected behavior with this PR?

No more flickering after rotation should occur.

How was the solution implemented (if it's not obvious)?

FillMode of the animation has been set to forward, so that the visual being rotated away will not fall back to initial values, but instead stays away.

Fix flickering after transition has finished. This was originally contained in my previous PR, but was lost.
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0037911-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@kekekeks kekekeks added this pull request to the merge queue Jul 19, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 19, 2023
@grokys grokys added this pull request to the merge queue Jul 19, 2023
Merged via the queue into AvaloniaUI:master with commit 126a576 Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants