Theming Optimizations about dotlottie-web

[theashraf]

@khuynh92

You're right, setting the "sid" on lottie json properties manually is not easy. We're currently working on abstracting the theme creation process in Lottie Creator, but until that happens, there is a workaround that allows you to assign slot IDs to animated color properties to match the parent shape's existing class name or ID. You can use relottie to parse the Lottie JSON into a relottie AST and traverse the AST to locate the target shapes by their class name or ID, and then modify their animated color properties' slot ID to match their parent Shapes class name or ID.

It would be nice to be able to apply theme updates using the existing class name or id, that way we don't have to manually add sid to the JSON file, or if there was a script provided that added the sid automatically based off of existing class names or ids in that are in the JSON file.

I've noted down the other two issues and have been able to replicate them. I will update you as soon as I have a fix

[theashraf]

@khuynh92

This part of the issue is fixed in [email protected] release.

Memory access out of bounds error when reverting back to default theme
Issue: Using loadThemeData, I'm running into issues when trying to revert an asset's theme back to the original data that's within the JSON file. Changing the theme works properly, but when reverting the theme back using loadThemeData("") or loadTheme(""), the manipulated property changes to black and error messages are thrown.

Will update you once the other part of the issue is resolved, thanks

[theashraf]

Thanks @khuynh92 for reporting these issues. We will look into it and get back to you with an easier way to set the slot IDs

[khuynh92]

Will update you once the other part of the issue is resolved, thanks

Appreciate it! Thanks for the consistent status updates 🙂