-
Notifications
You must be signed in to change notification settings - Fork 12
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
Issue-75: Event Dialogue System #92
Issue-75: Event Dialogue System #92
Conversation
…able to show test script when clicking on an elligible room
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.
It seems to look nice
Did you write everything in balloon.gd or did you copy from an example ?
It's good to have a separate scene so we can implement scene switching on the map once #87 has been discussed
Tests are failing, for the convention it's normal since no line has been added for events.
For the map maybe we should have a boolean to not load dialogues ? i'm not sure, would need to look at the tests in mapUI again
…ny location in Art folder. Update comments
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.
I have a few questions which I left in comments.
My main concern is that I feel like we will need one script file for each new dialogue, but maybe I misunderstood
# Conflicts: # Global/DEBUG_VAR.gd # project.godot
@Tysterman74 should this close #75 ? |
Yeah. Were we gonna have another issue later to include in the book background? If so, we can close this. |
Yeah I think that was the idea |
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.
Seems good. customizing the event and adding functionality like adding inv items and interacting with the player seems easy enough so I have complain about
Description
Creation of Dialog Screen for showing events.
Related issue(s)
Relates to #75
List of changes
A more detailed list of the changes.
Tests
Not added yet.
Additional notes
fauna_dialogue_system.mp4