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

Added Ivy Cudgel Animation #5034

Merged
merged 4 commits into from
Jul 26, 2024
Merged

Conversation

hedara90
Copy link
Collaborator

Added an animation for Ivy Cudgel along with supporting changes.
Changed palette indices for Ogerpon normal forms to be able to reuse them for Icy Cudgel, fixed Ogerpon sprites to work with the new indices.
Added jumpifmovetypeequals command to animation scripts.

Description

Repurposed the Wood Hammer move animation for Ivy Cudgel with a new sprite for the cudgel.
Remapped the palettes for Ogerpon normal sprites so that shared colors are at the same indices so that only part of the Ivy Cudgel sprite changes colors when the palette is changed.
Added the jumpifmovetypeequals command to animation scripts so that move animations can change depending on which type they are are.

Images

cudgel.mp4

Discord contact info

hedara

@AlexOn1ine
Copy link
Collaborator

front
back
front
back
front
back
front
back

@hedara90 Thoughts on using these sprites? They are taken from Cyan's repo. I'm not sure who made those but they are originally from RR.
I think the quality is much better then the current ones

@hedara90
Copy link
Collaborator Author

I like the back sprites, but I'm in the camp that think the mask should cover the face of Ogerpon

@AlexOn1ine
Copy link
Collaborator

I like the back sprites, but I'm in the camp that think the mask should cover the face of Ogerpon

that's fair. I let you decide if it's worth it to do the change for the backsprite

@hedara90
Copy link
Collaborator Author

I'd say not then, but I guess a note needs to be left somewhere that the Ogerpon Palettes are used for the Ivy Cudgel anim in case the sprites are changed in the future.

@kittenchilly
Copy link

I was actually talking to someone else about the Ogerpon sprites and a good case they made for why the face should be shown at least a little bit is due to Ogerpon's shiny. If the mask is fully covering the face, you can't see the shiny difference at all. Although in SV you also can't see the shiny difference in battle or summary screen... I think a good alternative is the overworld walking sprite not having the mask.

@hedara90
Copy link
Collaborator Author

Changed the cudgel sprite a bit to match official graphics a bit more.

cudgel.mp4

@hedara90 hedara90 added the category: move animation Pertains to move animations label Jul 26, 2024
@AlexOn1ine AlexOn1ine merged commit b0b63f6 into rh-hideout:master Jul 26, 2024
1 check passed
@hedara90 hedara90 deleted the cudgels-anims branch July 26, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: move animation Pertains to move animations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants