dialogic-godot / dialogic

💬 Create Dialogs, Visual Novels, RPGs, and manage Characters with Godot to create your Game!
https://dialogic.pro
MIT License
3.42k stars 206 forks source link

Dialogue Voice Support #409

Closed RedXGames closed 2 years ago

RedXGames commented 2 years ago

I Have updated the voice support implementation from my previous pull after suggestions from @Jowan-Spooner. This feature was also suggested on Issue 401

This will allow users to play character voices during a text/question event event.

How to use -Enable voices from settings. -You will be able to load audio files depending on your text event

VoiceSupport

Jowan-Spooner commented 2 years ago

I'll try it out this afternoon. Looks great already!

One last thing I thought might be cool would be a default audio_bus in the settings, so if you want to have your voice be in a special AudioBus you would not have to select it everytime.

What do you think? Gonna try the other things later!

Jowan-Spooner commented 2 years ago

Okay so I'm trying it right now:

Functionality wise:

UI-wise:

Otherwise this looks great. Thanks so much for your contribution. If you need any help feel free to ask here or on emilios discord server!

RedXGames commented 2 years ago

That's weird. voices are stopping when new text events appear with not audio (at least from my end), this might be a bug. I'll look into it.

I will add the default bus and the labels this afternoon. (Did not notice the CharacterVoice being on the wrong folder!.. OOps) [Multiline mode] is active when you have "New lines will insert new messages on", I think I'll remove it because it may confuse people.

Jowan-Spooner commented 2 years ago

Oh I see. You can leave the MultiLine Mode in if you explain it better. I will retry maybe it was a mistake on my side that kept the sound playing! Good luck implementing!

RedXGames commented 2 years ago

I managed to find the problem with the voice not being interrupted. I'll commit the changes in a bit.

Jowan-Spooner commented 2 years ago

Hm I can still get the sound to keep playing: Some more info for debugging:

RedXGames commented 2 years ago

It should be fixed now. let me know if you're still having problems

Jowan-Spooner commented 2 years ago

Works perfectly!

RedXGames commented 2 years ago

I Have converted this pull into a draft as I'm going to add more refinements to it sometime this week.

Jowan-Spooner commented 2 years ago

Very good! I have added the Milstone 1.3 so we don't forget to add this to the new version. There is so much other stuff to do till then that I'm sure it will make it in!

Jowan-Spooner commented 2 years ago

Hey @RedXGames have you managed to work on this? If you don't have the time right now (or just lost interest for now) I would gladly do the final work as I think it's only small stuff missing. Would love to have this on main soon.

RedXGames commented 2 years ago

Hello @Jowan-Spooner, I really couldn't get any free time lately so I didn't work more on it. If you wanna take a crack at it feel free to, it's mostly just small things left such as adding the default audio bus and saving settings.

Jowan-Spooner commented 2 years ago

Cool! I'll give it a shot today and see what I can do.

Jowan-Spooner commented 2 years ago

Just so you guys know: I'm working on this. I have mangaged to get the default bus to work okay. I'm inverstigating small bugs now.

There is one thing though that I noticed and that is a pretty annoying workflow thing. Because the editor doesn't really know what audio file is for what text (only for which line) it's super annoying to add another line of text in the middle a multiline text event after setting all the audio files. Because it will just add a new audio event at the end. You will have to redo all the audio files after your new line... I'm thinking about adding a possibility to add a empty audio Line between exisitng audio lines, but I'm not sure how to UI/UX wise.

Jowan-Spooner commented 2 years ago

@RedXGames Once again huge thanks for this contribution. Feel free to test and critique the changes I made. I've merged it for now, so more people can easily try it out and find problems.

Keep up the good work!

RedXGames commented 2 years ago

@Jowan-Spooner Thank you for finishing the job! I'm working on a tight schedule these days. Will give it a try this afternoon And thanks again for your help.