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

Radio crackle QoL #3902

Merged

Conversation

MarkSuckerberg
Copy link
Member

About The Pull Request

Makes radio crackle a lot less spammy. Basically, first of all re-adds the toggle to disable it, second of all, simplifies the code into one place, and third of all, makes it so that messages (besides your own) only crackle when another message hasn't been sent in 5 seconds (subject to change).

As a side effect totally intended feature, people using radios near you will instead play the much less intrusive radio message receive sound instead of the noisy walkie-talkie sound.

Why It's Good For The Game

Making it less repetitive and focusing it down to when it's actually needed is good for everyone.

Changelog

🆑
add: A pref to totally disable radio crackle sounds.
tweak: Radio crackle will only play if it's been 5 seconds since the last radio message the mob has heard.
tweak: The radio crackle will only play once a message has been successfully transmitted.
/:cl:

@github-actions github-actions bot added the Code change Watch something violently break. label Dec 11, 2024
@FalloutFalcon FalloutFalcon added this pull request to the merge queue Jan 6, 2025
Merged via the queue into shiptest-ss13:master with commit 23bdb9c Jan 6, 2025
16 checks passed
MrCat15352 pushed a commit to MrCat15352/MrCat that referenced this pull request Jan 8, 2025
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
Makes radio crackle a lot less spammy. Basically, first of all re-adds
the toggle to disable it, second of all, simplifies the code into one
place, and third of all, makes it so that messages (besides your own)
only crackle when another message hasn't been sent in 5 seconds (subject
to change).

As a ~~side effect~~ totally intended feature, people using radios near
you will instead play the much less intrusive radio message receive
sound instead of the noisy walkie-talkie sound.

## Why It's Good For The Game
Making it less repetitive and focusing it down to when it's actually
needed is good for everyone.

## Changelog

:cl:
add: A pref to totally disable radio crackle sounds.
tweak: Radio crackle will only play if it's been 5 seconds since the
last radio message the mob has heard.
tweak: The radio crackle will only play once a message has been
successfully transmitted.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants