-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Telegram skin recolor #30
Comments
No, the C API isn't going to help you. The changes you linked to are only made to the C++ interface, the C interface is just a binding to that. |
Looks like bindgen generate binding to C++: https://rust-lang.github.io/rust-bindgen/cpp.html maybe I can make hpp wrapper and add |
Yeah I see basically 3 options:
Feel free to try these out and/or suggest other options that I might've missed. The last one might be favourable as it would work with either version of rlottie. |
I made first working version |
It looks like this should work with Samsung version of rlottie too (Samsung/rlottie@3be663e), but with a different api. |
Telegram's fitzModifier gets replacement colors from json, and recolors skin only when Samsung's colorFilter will be applied to all colors |
Color filter can be useful for conversion, but I don't want to use it for skin recolor |
I can make feature 'v2' and add color filter later, but 'v2' and 'tg' features will be exclusive |
Telegram can recolor some animations to different skin colors
but it's telegram's rlottie only feature
TelegramMessenger/rlottie@67f103b
If I understand correctly I need to implement capi in optional file and add feature to use it
The text was updated successfully, but these errors were encountered: