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

Arc StrokeLineCap lost after transition #8659

Closed
kikipoulet opened this issue Aug 3, 2022 · 7 comments
Closed

Arc StrokeLineCap lost after transition #8659

kikipoulet opened this issue Aug 3, 2022 · 7 comments
Labels
bug help-wanted A contribution from the community would be most welcome.

Comments

@kikipoulet
Copy link

Using Transitions with an Arc loose the StrokeLineCap="Round" property using AvaloniaUI 0.10.17

CircleProgressWithoutTransition

When adding a transition :
<Arc.Transitions> <Transitions> <DoubleTransition Duration="0:0:0.8" Easing="CircularEaseOut" Property="SweepAngle" /> </Transitions> </Arc.Transitions>

CircleProgressWithTransition

The StrokeLineCap property is lost after the Transition is finished

@kikipoulet kikipoulet added the bug label Aug 3, 2022
@kekekeks
Copy link
Member

kekekeks commented Aug 3, 2022

Looks like a clipping-related bug

@maxkatz6 maxkatz6 added the help-wanted A contribution from the community would be most welcome. label Aug 3, 2022
@timunie
Copy link
Contributor

timunie commented Aug 4, 2022

@kikipoulet can you provide the sample app you have shown in your issue?

@kikipoulet
Copy link
Author

This is the UserControl from my library SukiUI : https://github.com/kikipoulet/SukiUI/blob/main/SukiUI/Controls/CircleProgressBar.axaml

And this is the code I use for the demo : https://pastebin.com/dMiZTYVj

@timunie
Copy link
Contributor

timunie commented Aug 5, 2022

@kikipoulet I think it's the same issue I had back then. I had a PR for this, but I'm not sure if it was backported. See #7778

See dirty rects (you can enable them in dev tool / [F12] ) for reference:

image

@timunie
Copy link
Contributor

timunie commented Aug 5, 2022

@kekekeks do you think we can backport the linked PR to 0.10.18?

@kikipoulet
Copy link
Author

Thanks for the explanation ! Can't wait for 10.18 then :)

@timunie timunie mentioned this issue Aug 8, 2022
3 tasks
@timunie
Copy link
Contributor

timunie commented Aug 8, 2022

Created a backport PR. Let's see if it will be merged, can't promise anything ^^

@maxkatz6 maxkatz6 closed this as completed Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help-wanted A contribution from the community would be most welcome.
Projects
None yet
Development

No branches or pull requests

4 participants