-
Notifications
You must be signed in to change notification settings - Fork 414
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
Fluffy tail for felinids #433
Fluffy tail for felinids #433
Conversation
After 9 years. I finished it. (Requires Phil's approval, will put it to ready for review as soon as he gets to see fully the fully finished PR)
RSI Diff Bot; head commit 8fb85d5 merging into d3028e7 Resources/Textures/DeltaV/Mobs/Customization/Felinid/felinid_tails.rsi
|
Make sure to fix your timing before marking it as ready |
Thing is, if im lookin at this right, then its the fact that half of it dont have the same amount of numbers and that was kinda intentional, the side sprites have less animations compared to the back sprites which is why I have given less numbers so it loops correctly thus why they are diferent, will it be an issue? I havent noticed any issues ingame when I was testing it. Unless its completely about somethin else that went over my head. |
It's a test fail so yes, it's an issue. Just have the speeds be consistent, use the same sprite several times for all I care |
Well hoppity damn, understandable. 👍 |
I'm not too familiar with the sprite system without looking at the code, but I suppose they wouldn't have that check without reason |
Updated meta.json to include all numbers to be equal. Updated PNG to allow updated numbers to have actual sprites and not just empty sprites.
in terms of sprite work, I'm having a hard time seeing what the sprites look like. I would like to see a screenshot of these in the dev environment and animated before I approve. It's good to make more customization options for felinids though |
After seeing a clip of the tail and markings animated on Discord, I would like the stripes animation to be changed. Particularly from the back view of it, the stripes look visually unclear while animated. The baseline fluffy tail sprite looks fine though. If you want, I can help you out with making the stripes look clearer on Discord. |
It would be my pleasure if you could help truly. |
* Fluffy tail for felinids After 9 years. I finished it. (Requires Phil's approval, will put it to ready for review as soon as he gets to see fully the fully finished PR) * An small update. Updated meta.json to include all numbers to be equal. Updated PNG to allow updated numbers to have actual sprites and not just empty sprites. * Improved the tail sprite. Proper animation and code. * Updates ftl for it to properly work. (cherry picked from commit 5bf246c)
(cherry picked from commit 3a72289)
# Description <sub>~~Change from Nyano~~</sub> Uses space a lot better. The old layout still exists if people want it for whatever reason. I also renamed "Default" to "Overlay", which should stay anyway if the default change is denied. --- <details><summary><h1>Media</h1></summary> <p> ## Separated  ## Overlay  </p> </details> --- # Changelog :cl: - tweak: UI layout now defaults to separated, the old one is still available in settings
After 9 years. I finished it.
(Requires Phil's approval, will put it to ready for review as soon as he gets to see fully the fully finished PR)
About the PR
Adds a fluffy tail for felinids
Why / Balance
Because Fluffy tails and lack of love for felinids
Technical details
adds 5 files = 2 pngs, meta.json, ftl and yml.
Media
Breaking changes
Changelog
Make sure to take this Changelog template out of the comment block in order for it to show up.
🆑