Skip to content
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

Implement SFX for bombs #12

Merged
merged 8 commits into from Dec 1, 2022
Merged

Implement SFX for bombs #12

merged 8 commits into from Dec 1, 2022

Conversation

Piturnah
Copy link
Collaborator

@Piturnah Piturnah commented Nov 30, 2022

Current implementation:

  • Fuse ignition sound on bomb placed
  • Bomb explosion sound played on explosion
  • Explosion sound played on player death

TODO

  • experiment with volume
  • cleanup implementation

Open request for feedback

Will close #11

`String` implements `Deref` to `&str` so this isn't necessary.
Plays an explosion sound on a bomb going off as well as a random bassy
explosion sound when a wall is destroyed.
@Piturnah Piturnah added the A:Audio This issue relates to audio label Nov 30, 2022
@Piturnah Piturnah added this to the MVP milestone Nov 30, 2022
@Piturnah
Copy link
Collaborator Author

9d8db09 was supposed to be committed to main but that's fine :P

- use only CC0 sounds.
- now there is a "normal" bomb effect, and a player death boom.
This way we aren't having to generate a new RNG source every time.
@Piturnah Piturnah marked this pull request as ready for review December 1, 2022 13:08
@Piturnah Piturnah merged commit 771a2fb into main Dec 1, 2022
@Piturnah Piturnah deleted the feat/sfx branch December 1, 2022 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:Audio This issue relates to audio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bomb SFX
1 participant