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

Character Event and Animations with Anima #747

Merged

Conversation

Jowan-Spooner
Copy link
Collaborator

ONE CHARACTER EVENT TO RULE THEM ALL

  • Adds the new Character event with three modes (JOIN, LEAVE, UPDATE)

  • Adds new CharacterAction and CharacterActionSettings event parts

  • Tries to convert all Character Join and Character Leave events into this new one (has id 'dialogic_002')

  • adds simple animation support for character leave (but this shall be replaced with anima, as all animations should)

ANIMA
I feel a bit bad, because I don't understand completely how the code I ported works. But it works. for now. There is probably parts in there, that we don't even need. Maybe for 2.0 we can take another look, who knows.

Also added 'change' controls, so a Character Update event doesn't have to update everything, only the things you want.

Also adds a repeat option that repeats the animation the given time.

Also added a wait for animation to finish option.

Also added default animation setting. These can be (for join and leave) set in the settings menu.

Should also fix #744

- Adds the new Character event with three modes (JOIN, LEAVE, UPDATE)
- Adds new CharacterAction and CharacterActionSettings event parts
- Tries to convert all Character Join and Character Leave events into this new one (has id 'dialogic_002')

- adds simple animation support for character leave (but this shall be replaced with anima, as all animations should)
I feel a bit bad, because I don't understand completely how the code I ported works. But it works. for now. There is probably parts in there, that we don't even need. Maybe for 2.0 we can take another look, who knows.

Also added 'change' controls, so a Character Update event doesn't have to update everything, only the things you want.

Also adds a repeat option that repeats the animation the given time. 

Also added a wait for animation to finish option. 

Also added default animation setting. These can be (for join and leave) set in the settings menu.
Old join and leave events were not correctly updated.
Now it worked, at least for me.
Suggested by @TheBardsRc on discord
It will not keep portrait or animations that aren't actually possible anymore if you change character or type.
@coppolaemilio
Copy link
Collaborator

coppolaemilio commented Jan 31, 2022

Upon testing an existing project I get all this notifications. I assume it is because of the timeline event scene changes, but just wanted to flag just in case:
image

I'll keep testing since even with the errors everything works as expected


@Jowan-Spooner I can't find any license or reference to the Anima library (I think we have to add their https://github.com/ceceppa/anima/blob/master/LICENSE somewhere in our plugin) Did I miss it or do we still have to add it?


I get this error when updating the animation on a leave event:
image

image

Again, errors on the output but it works as expected.

@coppolaemilio
Copy link
Collaborator

That's all I think. Feel free to either fix these or just merge as is and we can fix/patch later on. I might change a few things (like the order of the effects and a few settings like that) but nothing that would block this one to get merged. Thank you so much, it seems like it was a lot of work!

@Jowan-Spooner
Copy link
Collaborator Author

@coppolaemilio You are right, we still need to add the license. I'm not too familiar with the MIT license. Where should we put it? Do need to say that it was modified?

Also I've pushed another fix that should remove all the warnings hopefully.

@Jowan-Spooner Jowan-Spooner merged commit 0dcdd37 into dialogic-godot:main Feb 1, 2022
@Jowan-Spooner Jowan-Spooner deleted the unify-character-event branch March 1, 2022 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move "mirror portrait" setting to a checkbox inside the body of the event
2 participants