Skip to content

Elders: Hide balloon from dialogue#2365

Merged
manuq merged 1 commit into
mainfrom
wjt/elders-hide-balloon-from-dialogue
Jun 12, 2026
Merged

Elders: Hide balloon from dialogue#2365
manuq merged 1 commit into
mainfrom
wjt/elders-hide-balloon-from-dialogue

Conversation

@wjt

@wjt wjt commented Jun 12, 2026

Copy link
Copy Markdown
Member

Elders: Hide balloon from dialogue

I have just noticed that the balloon script adds itself to the context
that is available to dialogue. This means there is a cleaner way to hide
the dialogue balloon before showing the storybook than what I
implemented in commit dfa9e69: call
balloon.hide() directly from the dialogue, right before calling
show_storybook() on the Elder (which talk_behavior.gd passes as
context).

Implement this. Remove the plumbing that I added to make the balloon
node accessible from the elder script.

I have just noticed that the balloon script adds itself to the context
that is available to dialogue. This means there is a cleaner way to hide
the dialogue balloon before showing the storybook than what I
implemented in commit dfa9e69: call
`balloon.hide()` directly from the dialogue, right before calling
`show_storybook()` on the Elder (which talk_behavior.gd passes as
context).

Implement this. Remove the plumbing that I added to make the balloon
node accessible from the elder script.
@wjt wjt requested a review from a team as a code owner June 12, 2026 14:03
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown

Test build no longer available.

@manuq manuq left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oh much better!

@manuq manuq merged commit 0026059 into main Jun 12, 2026
7 checks passed
@manuq manuq deleted the wjt/elders-hide-balloon-from-dialogue branch June 12, 2026 17:23
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.

2 participants