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

Disable pausing music during intro sequence #26411

Merged
merged 7 commits into from
Jan 8, 2024

Conversation

zacwm
Copy link
Contributor

@zacwm zacwm commented Jan 6, 2024

Resolves #25999

This disables the track playback from being controlled when the intro sequence begins, and allows control to the user again when the main menu appears.

Previously the intro sequence music was able to be paused, and after would suddenly resume to the one of the intro tracks or a beatmap track suddenly.

Before:

intro-paused-before.mp4

After:

intro-pausing-after.mp4

Just a heads up... The test does it's job, but I'm not 100% confident in the code quality of the test. I've tried my best and did digging around with other tests for understanding, but got a little lost when it seemed like I should use OsuGameTestScene but needed an input during it's intro sequence.
So I'm ready to learn if I wrote the test poorly. 👍

@bdach bdach self-requested a review January 8, 2024 11:44
@bdach bdach changed the title Disable pausing music during intro sequence. Disable pausing music during intro sequence Jan 8, 2024
@bdach bdach merged commit bd4b20c into ppy:master Jan 8, 2024
13 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants