Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@aviollaz
Copy link
Contributor

Fixes #35865.
This fixes a bug where the background music speed during the quick play queue is affected by the mods selected in map selection. The bug was caused by the screen failing to reset the global musicController.ApplyModTrackAdjustments state. The fix explicitly sets the value to false in OnEntering and OnResuming.

{
base.OnEntering(e);

musicController.ApplyModTrackAdjustments = false;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Rather than touching musicController directly this should be specified via

/// <summary>
/// Whether mod track adjustments should be applied on entering this screen.
/// A <see langword="null"/> value means that the parent screen's value of this setting will be used.
/// </summary>
bool? ApplyModTrackAdjustments { get; }

Compare e.g.

public override bool? ApplyModTrackAdjustments => false;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the correction, I'm just now learning about the osu! project's structure, so it's quite possible I miss trivial things like that!

@aviollaz aviollaz closed this Dec 16, 2025
@aviollaz aviollaz reopened this Dec 16, 2025
@aviollaz aviollaz requested a review from bdach December 16, 2025 20:24
Copy link
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

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

Looks logical

@peppy peppy merged commit 3b635f6 into ppy:master Dec 17, 2025
11 of 16 checks passed
@aviollaz aviollaz deleted the fix/quick-play-bgm-speed branch December 17, 2025 06:07
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.

Mods like double time and nightcore affect the background song in the quick play queue

3 participants