Skip to content

Conversation

@wjt
Copy link
Member

@wjt wjt commented Sep 30, 2025

The dialogue in the intro is set to play at half speed. I added this way back in March when I added this dialogue #46 but there's no explanation there for why I made it half speed.
Play it at the normal speed.

Also, simplify how we wait for the fade-in to finish before starting the intro dialogue.

wjt added 2 commits September 30, 2025 15:02
We want the intro dialogue to not start until the fade-in from the previous
scene has finished.

Previously this was done by playing an animation that does nothing for 1 second.
We could instead use Godot Dialogue Manager's built-in syntax for waiting:

    do wait(1.0)

but we can do even better. Wait for the Transitions.finished signal, but ony if
a transition is running. This is the same pattern used in a couple of other
places.

(Arguably Cinematic should do this automatically but that's a bigger change.)
The dialogue in the intro is set to play at half speed. I added this way
back in March when I added this dialogue:

#46

but there's no explanation there for why I made it half speed.
Play it at the normal speed.
@wjt wjt requested a review from a team as a code owner September 30, 2025 14:03
@github-actions
Copy link

Play this branch at https://endlessm.github.io/threadbare/branches/endlessm/intro-dialogue-normal-speed.

(This launches the game from the start, not directly at the change(s) in this pull request.)

Copy link
Collaborator

@manuq manuq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to also use a signal for the transition waiting.

@manuq manuq merged commit dd87be5 into main Sep 30, 2025
8 checks passed
@manuq manuq deleted the intro-dialogue-normal-speed branch September 30, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants