Investigate path direction + fill-rule about lottie-windows HOT 21 CLOSED

@FrayxRulez thanks for the bug report. Do you have the AEP file as well? Or any other lottie JSON files that also demonstrate the same incorrect behavior?

Reducing these animations to a minimal repro is a struggle across all Lottie implementations at present.

from lottie-windows.

I have a minimalized version of the same animation, with just head shape and eyes:
minimal.json

from lottie-windows.

I see that we're painting in the wrong order, but I'm having trouble figuring out what the correct order should be. In your minimal.json, layer indexes are:
21 - shoulders
20 - head
25 - eye
26 - eye
Currently we draw in decreasing order of index, but doing that for minimal.json draws the eyes before the head. I'm wondering if maybe this isn't a problem with order, but with a mask or a matte or a blend or something that should allow the eyes to show through from behind. There's definitely something special about your file, I'm just not sure what it is yet!

Can you give me any more hints. Breaking it down further to just the head and the eyes would help although I can do that myself with some effort on the JSON. Even better would be an AEP of the minimalized JSON.

from lottie-windows.

I am facing a similar problem
The following screenshots though might appear different are the same one from Web, other from Windows app.

Web

Windows

It works properly in Web, IOS and android.

from lottie-windows.

@NikhStash thanks for the repro; will take a look.

from lottie-windows.

@NikhStash can you re-attach your json file? I'm not sure if it disappeared or if I missed it in the first place.

from lottie-windows.

In Web: (Expected Beahavior)

In Windows:

Here is the json file for this
pwl_pushnotification.zip

from lottie-windows.

Are any steps being taken on this, if so any info on when this can be cleared!

from lottie-windows.

@NikhStash: I pushed the latest changes in master to the Lottie Viewer app a couple of days ago. Can you try it out? Your issue does not repro (even though your file results in warnings) / has been fixed.

The changes in master will be published in version 6.0, which should be released in about a month. If you want the fix earlier, you can grab the nuget from Azure DevOps here.

from lottie-windows.

I've finally managed to reduce @FrayxRulez's animation to a minimal repro with a single shape group that is stacked as follows:

1. Path for left eye socket
2. Path for face
3. Yellow Fill

The expected outcome is:

where the eye socket is actually transparent (white background).

layertest.zip

from lottie-windows.

Thanks, @sohchatt. It was not working in the previous build, but it is working perfectly in the latest build that you have specified.

from lottie-windows.

@NikhStash: I pushed the latest changes in master to the Lottie Viewer app a couple of days ago. Can you try it out? Your issue does not repro (even though your file results in warnings) / has been fixed.

The changes in master will be published in version 6.0, which should be released in about a month. (https://dev.azure.com/dotnet/WindowsCommunityToolkit/_build/results?buildId=20753&view=artifacts).

I have installed 6.0 version but it doesn't work. In this version, the screen is completely blank. It doesn't even show the faulted UI. Could you please share the working build again. And when can I expect a proper build.

from lottie-windows.

I actually had to move away from this library and I made my own porting of rlottie using Win2D.

from lottie-windows.

Actually @FrayxRulez I was asking @sohchatt . Sorry for the confusion.
@sohchatt If you don't remember what I'm talking please refer: #124 (comment)

I have installed 6.0 version but it doesn't work. In this version, the screen is completely blank. It doesn't even show the faulted UI. Could you please share the working build again. And when can I expect a proper build.

from lottie-windows.

@ajbennet / @scott-moore-ms / @simeoncran should be able to help you.

from lottie-windows.

Thanks @sohchatt for looping us in. @NikhStash, we are looking into this. Will keep you posted as soon as we know. Thanks.

from lottie-windows.

Thank you @ajbennet. Please let me posted.

from lottie-windows.

@ajbennet any update on the issue.

from lottie-windows.

@NikhStash this is working for me on the 6.1 preview code. Can you try it?

from lottie-windows.

Closing this for now because I think this is working now. The original FrayxRulez issue has also been solved. This is still one remaining issue for the FrayxRulez file due to multiple animated, which we will work on separately. Please reopen if you are still seeing issues.

from lottie-windows.

And the one remaining issue is fixed with: #274.

from lottie-windows.