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

Make sound effects pause with the game #4058

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NotHyper-474
Copy link
Contributor

@NotHyper-474 NotHyper-474 commented Jan 29, 2025

Does this PR close any issues? If so, link them below.

Closes #2879 and #3600

Briefly describe the issue(s) fixed.

Scripts like spookyMansion.hxc, phillyTrain.hxc (and Erect versions) play sound effects that don't pause with the game.
Philly's train sound does get paused although that's done manually through the stage script.
This is based on how PlayState handles pausing of camera tweens.

Include any relevant screenshots or videos.

demo2.mp4

@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. pr: haxe PR modifies game code. labels Jan 29, 2025
@NotHyper-474 NotHyper-474 force-pushed the bugfix/sound-pause branch 2 times, most recently from 4621a84 to 4a5f97b Compare February 4, 2025 20:58
@NotHyper-474 NotHyper-474 changed the title Make sounds pause with the game Make sound effects pause with the game Feb 9, 2025
Copy link

@JackXson-Real JackXson-Real left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! I've tested every SFX and they all pause properly. This should be good to be merged.

- Change behaviour to not pause on Game Over.

This commit also does a required change which unintentionally fixes FunkinCrew#3600.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants