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

Silence reagent dispensers on spawn. #2862

Merged

Conversation

whatston3
Copy link
Contributor

@whatston3 whatston3 commented Feb 5, 2025

About the PR

Calls ContainerSystem.Insert directly instead of ItemSlot.Insert when initializing a reagent dispenser.

Why / Balance

Currently, the reagent dispenser plays the clicky insert sound when inserting things into the dispenser. It does this for each item in the dispenser on spawn. If you warp to a chemistry ship, your ears currently bleed.

How to test

  1. Spawn a chemical dispenser.
  2. Enjoy the silence.
  3. View the contents of the chemical dispenser.
  4. They should match.
  5. Insert more jugs, you should be able to fit five. Each insertion is clicky.
  6. Remove and reinsert jugs. Removal/insertion is clicky.

Media

The contents of a chemical dispenser after spawning. Everything is accounted for.
image

Requirements

Breaking changes

Changelog

🆑

  • fix: Reagent dispensers no longer play a sound for inserting all jugs on initializing.

@whatston3 whatston3 added S: Needs Review This PR is awaiting reviews and removed S: Untriaged labels Feb 5, 2025
@dvir001 dvir001 self-requested a review February 5, 2025 19:11
@dvir001
Copy link
Contributor

dvir001 commented Feb 5, 2025

Not a fan of this one.

@dvir001 dvir001 merged commit 87be818 into new-frontiers-14:master Feb 5, 2025
15 checks passed
FrontierATC added a commit that referenced this pull request Feb 5, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Feb 24, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Feb 24, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Mar 1, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Mar 1, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Mar 3, 2025
percent-temp-percent pushed a commit to Corvax-Frontier/Frontier that referenced this pull request Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# S: Needs Review This PR is awaiting reviews size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants