-
Notifications
You must be signed in to change notification settings - Fork 580
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
Doc Actor: En_Brob (Jabu Jabu Wobble Flesh Block) #1581
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple comments. Overall great job, you are very in-tune to our naming conventions already :D
I did not really review the asset naming, if someone else wants to take a closer look at that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job :)
EnBrob_Stunned
is a bit weird because it's not the stunned state itself, it's only the "goes to stunned" animation and the actual stunned state (including when you can walk on the blob) is part of EnBrob_Idle
, but oh well
I only have one comment on the limb symbol names
Co-authored-by: Dragorn421 <Dragorn421@users.noreply.github.com>
* brob function naming * modified brob header file * renamed skel, collision, and anims * named limbs and dls * renamed shock function * ran formatter * doc modifications * Update assets/xml/objects/object_brob.xml Co-authored-by: Dragorn421 <Dragorn421@users.noreply.github.com> --------- Co-authored-by: Dragorn421 <Dragorn421@users.noreply.github.com>
First actor documentation attempt, feel free to let me know anything I've done wrong, should be changed, or that I missed. Suggestion welcome as well.
Don't be afraid to tear into it, no better way to learn than through experience.
I was very unsure about the DL/Limb names, there were limbs inside of limbs that did nothing, so I checked another actor and they seemed to follow this pattern.