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

[MIRROR] Placing things in a paperbin actually plays sounds. #2426

Merged
merged 1 commit into from
Mar 17, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1455
Original PR: tgstation/tgstation#81974

About The Pull Request

In contrast to my previous pr:
Really just changes the user.transferItemToLoc(...) used for transferring paper to the bin to have a silent = FALSE parameter.
Also did it for pens for good measure, though I believe most pens don't have such sounds yet.

Why It's Good For The Game

Placing paper on a table makes sounds, placing paper in a bin makes no sound. This feels awkward.
This changes that to actually make a sound.
Also adds silent = FALSE to placing pens on it for good measure, though most pens don't have such sounds yet.

Changelog

🆑 00-Steven
sound: Placing paper in a paperbin is no longer silent.
/:cl:

* Placing things in a paperbin actually plays sounds. (#81974)

## About The Pull Request

In contrast to my previous pr:
Really just changes the `user.transferItemToLoc(...)` used for
transferring paper to the bin to have a `silent = FALSE` parameter.
Also did it for pens for good measure, though I believe most pens don't
have such sounds yet.
## Why It's Good For The Game

Placing paper on a table makes sounds, placing paper in a bin makes no
sound. This feels awkward.
This changes that to actually make a sound.
Also adds `silent = FALSE` to placing pens on it for good measure,
though most pens don't have such sounds yet.
## Changelog
:cl:
sound: Placing paper in a paperbin is no longer silent.
/:cl:

* Placing things in a paperbin actually plays sounds.

---------

Co-authored-by: _0Steven <[email protected]>
@ReezeBL ReezeBL merged commit 1874c43 into master Mar 17, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1455 branch March 17, 2024 00:15
AnywayFarus added a commit that referenced this pull request Mar 17, 2024
Iajret pushed a commit that referenced this pull request May 10, 2024
* Changes

* Update modular_nova/modules/reagent_forging/code/forge.dm

* Missed one

* Missed two

---------

Co-authored-by: Bloop <[email protected]>
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.

3 participants