Navigation Menu

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

Unable to turn off Jetpack Mode-Switching Sound #5143

Closed
tzhouhc opened this issue Aug 21, 2018 · 0 comments
Closed

Unable to turn off Jetpack Mode-Switching Sound #5143

tzhouhc opened this issue Aug 21, 2018 · 0 comments
Labels
To be confirmed Looks like a bug, hasn't been confirmed by a dev

Comments

@tzhouhc
Copy link

tzhouhc commented Aug 21, 2018

Please use the search functionality before reporting an issue. Also take a look at the closed issues!

Issue description:

Unable to disable the HYDRAULIC sound played when switching jetpack/scube modes. Not sure if intentional.

Steps to reproduce:

  1. Equip jetpack.
  2. Press button to switch mode.
  3. Go into mod config, turn off both player sound, machine sound, and turn the global volume to 0.
  4. Press button to switch mode.

Version (make sure you are on the latest version before reporting):

Forge: 14.23.4.2749
Mekanism: 1.12.2-9.4.13.349
Other relevant version:

The code (src/main/java/mekanism/client/MekanismKeyHandler.java, L153) seems to indicate that is indeed intended behavior, and I've not read the code base enough to be sure if the SoundHandler is capable of controller volume globally. If that is indeed the case I can close the issue right away. Still, it would be really nice to have these sounds also under global configuration.

Also, as a personal pet peeve that is completely unrelated to the functionality of the mod (which is excellent), the HYDRAULIC sound is quite obnoxious, and even more so if I do turn all the other machine sounds down. CLICK seem to be a better choice to me -- the player is not carrying a hydraulics machine, the mode switch should just be a click on a personal device, and it's a lot quieter and shorter, making it a better fit for such notifications. But that, again, is your own artistic license and not for me to judge :)

If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)

[gist/pastebin/etc link here]

@thiakil thiakil added the To be confirmed Looks like a bug, hasn't been confirmed by a dev label Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
To be confirmed Looks like a bug, hasn't been confirmed by a dev
Projects
None yet
Development

No branches or pull requests

2 participants