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

Sounds: Added multiple UI sounds #1460

Merged
merged 14 commits into from
Mar 7, 2024
Merged

Sounds: Added multiple UI sounds #1460

merged 14 commits into from
Mar 7, 2024

Conversation

TimGoll
Copy link
Member

@TimGoll TimGoll commented Mar 6, 2024

After testing #1437 I notices that it feels really weird to not have any kind of feedback when interacting with entities. When picking up entities, there already is a sound, but not when opening UIs. Therefore I added sound.ConditionalPlay that plays a sound if the sound type is enabled in the settings.

I also extended this to body search and UI buttons.

See this demo video: https://www.youtube.com/watch?v=c6ixkuFSYfw

Note: I also pulled in the changes from #1351 and added them to the same system. They are disabled by default.

Menu to disable sounds by type:
image

@TimGoll TimGoll added the type/enhancement Enhancement or simple change to existing functionality label Mar 6, 2024
@Histalek Histalek added this to the v0.14.0b milestone Mar 7, 2024
CHANGELOG.md Outdated Show resolved Hide resolved
lua/terrortown/menus/gamemode/gameplay/voiceandvolume.lua Outdated Show resolved Hide resolved
lua/ttt2/extensions/sound.lua Outdated Show resolved Hide resolved
@TimGoll
Copy link
Member Author

TimGoll commented Mar 7, 2024

all things should be done @Histalek

lua/terrortown/lang/en.lua Outdated Show resolved Hide resolved
@TimGoll TimGoll merged commit ad5cafe into master Mar 7, 2024
4 checks passed
@TimGoll TimGoll deleted the client-sound-rework branch March 7, 2024 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhancement or simple change to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants