mirror of
https://github.com/shadps4-emu/shadPS4.git
synced 2026-04-25 15:56:00 +03:00
[PR #3271] [MERGED] Add default trophy sound #3353
Labels
No labels
Bloodborne
bug
contributor wanted
documentation
enhancement
frontend
good first issue
help wanted
linux
pull-request
question
release
verification progress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/shadPS4#3353
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/shadps4-emu/shadPS4/pull/3271
Author: @rainmakerv3
Created: 7/19/2025
Status: ✅ Merged
Merged: 7/21/2025
Merged by: @georgemoralis
Base:
main← Head:TrophySound📝 Commits (4)
4ba4100Add default trophy sound4def7b3delete include to removed folderc840648remove redundant conditions925536cChange trophy sound - credit to Tlarok📊 Changes
4 files changed (+44 additions, -3 deletions)
View changed files
📝
CMakeLists.txt(+1 -1)📝
REUSE.toml(+1 -0)📝
src/core/libraries/np_trophy/trophy_ui.cpp(+42 -2)➕
src/images/trophy.wav(+0 -0)📄 Description
Adds a default sound upon unlocking a trophy if no custom sound file exists.
Got the sound from here: https://pixabay.com/sound-effects/search/achievement/ so should be ok to use.
At first I thought this would need some kind of volume normalization because some games are much louder than others, but it seems to play at reasonable levels for both the loud game and soft game I tried so normalization seems to be unneeded.
The wav file is embedded in the binary similar to the trophy icons. For its source, i just put it in src/images with the other trophy files which may be kinda weird but I can't see a better location for it and probably doesn't justify a new folder.
EDIT: instead using a sound that was made by @tlarok . Thanks very much!
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.