Skip to content

[QOL/Accessibility] Broken Item Sounds#8392

Open
kphoenix137 wants to merge 2 commits intodiasurgical:masterfrom
kphoenix137:broken-item-sounds
Open

[QOL/Accessibility] Broken Item Sounds#8392
kphoenix137 wants to merge 2 commits intodiasurgical:masterfrom
kphoenix137:broken-item-sounds

Conversation

@kphoenix137
Copy link
Copy Markdown
Collaborator

@kphoenix137 kphoenix137 commented Dec 30, 2025

Adds the unused sound effects armrfkd.wav (armor f**ked), Swrdfkd.wav (sword f**ked), hlmtfkd.wav (helmet f**ked), and shielfkd.wav (shield f**ked) to be played when items are destroyed at 0 durability. Provides auditory feedback to the player to assist with giving a clear indicator that an item broke, which also works as an accessibility feature for the visually impaired. Unobtrusive enough to not require a toggle (in my opinion).

@qndel
Copy link
Copy Markdown
Member

qndel commented Dec 30, 2025

Maybe you could come up with some nice refactor that calls some function in all these places which handles

			RemoveEquipment(player, INVLOC_HAND_LEFT, true);
			CalcPlrInv(player, true);
			PlaySfxLoc(SfxID::BrokeShield, player.position.tile);

?

also any idea why last 2 places don't call CalcPlrInv(player, true); ? Maybe a potential bug there? :D

@kphoenix137
Copy link
Copy Markdown
Collaborator Author

kphoenix137 commented Dec 30, 2025

Maybe you could come up with some nice refactor that calls some function in all these places which handles

			RemoveEquipment(player, INVLOC_HAND_LEFT, true);
			CalcPlrInv(player, true);
			PlaySfxLoc(SfxID::BrokeShield, player.position.tile);

?

also any idea why last 2 places don't call CalcPlrInv(player, true); ? Maybe a potential bug there? :D

CalcPlrInv isn't needed whenever just durability changes, and the item isn't destroyed because nothing changes (no graphic change, or character stat changes).

Also, I have refactored these functions already:
#8387
#8386
#8385

@qndel
Copy link
Copy Markdown
Member

qndel commented Dec 30, 2025

nvm I've expanded the code down and see CalcPlrInv(player, true); is there, just below :D

@StephenCWills
Copy link
Copy Markdown
Member

Does anyone see an issue with playing shielfkd.wav when Monk blocks and loses his staff?

@strich
Copy link
Copy Markdown
Collaborator

strich commented Jan 29, 2026

Does anyone see an issue with playing shielfkd.wav when Monk blocks and loses his staff?

Yeah I think that's a good shout!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants