Skip to content

Comments

Replace throwing enemy action sound effect#1973

Merged
wjt merged 2 commits intomainfrom
ink-drinker-sfx
Feb 24, 2026
Merged

Replace throwing enemy action sound effect#1973
wjt merged 2 commits intomainfrom
ink-drinker-sfx

Conversation

@manuq
Copy link
Collaborator

@manuq manuq commented Feb 23, 2026

Rename it from "spit" to "spray". And move it to the first-party directory. Removing the previous license information.

Also adjust the AnimationPlayer node in the enemy scene to delay a bit playing this sound effect, so it matches the frame-by-frame animation.

The sound was recoded by @PlayMatters as InkDrinker_AirPop_01.wav. So attribution is Threadbare's Authors.

There are unrelated changes in ink combat rounds 4, 5, 6 because of the recent projectile changes.

Rename it from "spit" to "spray". And move it to the
first-party directory. Removing the previous license
information.

Also adjust the AnimationPlayer node in the enemy scene
to delay a bit playing this sound effect, so it matches
the frame-by-frame animation.

The sound was recoded by @PlayMatters as InkDrinker_AirPop_01.wav
and converted to OGG with ffmpeg. So attribution is Threadbare's
Authors.
@manuq manuq requested a review from a team as a code owner February 23, 2026 17:49
@github-actions
Copy link

Play this branch at https://play.threadbare.game/branches/endlessm/ink-drinker-sfx.

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

Copy link
Member

@wjt wjt left a comment

Choose a reason for hiding this comment

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

@manuq
Copy link
Collaborator Author

manuq commented Feb 24, 2026

@wjt thanks for the feedback. Ready for review again!

@wjt wjt merged commit 4c69a7e into main Feb 24, 2026
6 checks passed
@wjt wjt deleted the ink-drinker-sfx branch February 24, 2026 13:16
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