Skip to content

feat: Add input gathering example project#459

Merged
elementbound merged 6 commits intofoxssake:mainfrom
jonandrewdavis:patch-1
May 28, 2025
Merged

feat: Add input gathering example project#459
elementbound merged 6 commits intofoxssake:mainfrom
jonandrewdavis:patch-1

Conversation

@jonandrewdavis
Copy link
Copy Markdown
Contributor

In the process of implementing this pattern using the example code, I could not get the event to fire. (Environment: Godot 4.5, Mac M4 chip)

During troubleshooting, I discovered that the buffer value should be set to false, not the actual value.

While I'm not exactly sure on the loop-related details, I might recommend this fix. It seems if you reset the real value, the gather never actually sets it to true.

See this discord thread about these docs for the code & issue: https://discord.com/channels/1253434107656933447/1366836798319231039/1374158824289599628

If this isn't the right recommendation, close this PR.

jonandrewdavis and others added 3 commits May 19, 2025 19:20
In the process of implementing this pattern using the example code, I could not get the event to fire. (Environment: Godot 4.5, Mac M4 chip)

During troubleshooting, I discovered that the buffer value should be set to false, not the actual value.

While I'm not exactly sure on the loop-related details, I might recommend this fix. It seems if you reset the real value, the gather never actually sets it to true.

See this discord thread about these docs for the code & issue:
https://discord.com/channels/1253434107656933447/1366836798319231039/1374158824289599628

If this isn't the right recommendation, close this PR.
@elementbound elementbound changed the title doc: update example to reset the buffer, not the value feat: Add input gathering example project May 28, 2025
@elementbound elementbound merged commit 65d11e4 into foxssake:main May 28, 2025
2 checks passed
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