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

Adds a scaling direct hit sound to the XM88 #5117

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

VileBeggar
Copy link
Contributor

@VileBeggar VileBeggar commented Dec 3, 2023

About the pull request

Adds a scaling impact sound for every direct hit you obtain with the XM88, depending on the current penetration value.

Explain why it's good for the game

The bullseye mechanic makes this weapon very, very powerful. However, you don't really feel like you're doing any more damage with it. The XM88 doesn't really give the user any proper feedback if they're managing to nail several direct hits. A good sound effect can give them a dopamine rush, and in turn, make the weapon more fun to use.

Testing Photographs and Procedure

Screenshots & Videos
xm88.mp4

Changelog

🆑
soundadd: The XM88 now has a scaling hit sound for every direct hit you manage to land with it.
/:cl:

@github-actions github-actions bot added the Sound Blast 5 minutes of bass boosted music to our players label Dec 3, 2023
Copy link
Member

@fira fira left a comment

Choose a reason for hiding this comment

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

Code OK. I'd still suggest to leave the vary out so the sounds are more easily recognizable

phone edit
@harryob harryob added this pull request to the merge queue Dec 6, 2023
Merged via the queue into cmss13-devs:master with commit e448b00 Dec 6, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sound Blast 5 minutes of bass boosted music to our players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants