From a8989eb117ec3b2bc2db11716ae9e9a502cfdf93 Mon Sep 17 00:00:00 2001 From: Denis Titovets Date: Mon, 12 Jan 2026 18:03:20 +0300 Subject: [PATCH] Expand settings in `ReplayPlayer` by default (#36308) ~i recently saw an suggestion to do this, but don't remember where~ - addresses https://github.com/ppy/osu/discussions/36189 i think it's logical, since the settings have been displayed in a separate overlay for more than six months, and not on top of the gameplay itself, and for example, it can be difficult to expand section on phones | master | pr | |-|-| | osu_2026-01-12_09-40-23 | osu_2026-01-12_09-39-40 | --- osu.Game/Screens/Play/HUD/PlayerSettingsOverlay.cs | 8 +------- osu.Game/Screens/Play/ReplayPlayer.cs | 6 +----- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/osu.Game/Screens/Play/HUD/PlayerSettingsOverlay.cs b/osu.Game/Screens/Play/HUD/PlayerSettingsOverlay.cs index 635d140a4a..4fe207a6f9 100644 --- a/osu.Game/Screens/Play/HUD/PlayerSettingsOverlay.cs +++ b/osu.Game/Screens/Play/HUD/PlayerSettingsOverlay.cs @@ -21,8 +21,6 @@ namespace osu.Game.Screens.Play.HUD { public partial class PlayerSettingsOverlay : ExpandingContainer { - public VisualSettings VisualSettings { get; private set; } - private const float padding = 10; public const float EXPANDED_WIDTH = player_settings_width + padding * 2; @@ -66,11 +64,7 @@ namespace osu.Game.Screens.Play.HUD Direction = FillDirection.Vertical, Spacing = new Vector2(0, 20), Margin = new MarginPadding(padding), - Children = new PlayerSettingsGroup[] - { - VisualSettings = new VisualSettings { Expanded = { Value = false } }, - new AudioSettings { Expanded = { Value = false } } - } + Children = new PlayerSettingsGroup[] { new VisualSettings(), new AudioSettings() } }); // For future consideration, this icon should probably not exist. diff --git a/osu.Game/Screens/Play/ReplayPlayer.cs b/osu.Game/Screens/Play/ReplayPlayer.cs index 7fec8d6332..b9a6de54b4 100644 --- a/osu.Game/Screens/Play/ReplayPlayer.cs +++ b/osu.Game/Screens/Play/ReplayPlayer.cs @@ -77,11 +77,7 @@ namespace osu.Game.Screens.Play /// Add a settings group to the HUD overlay. Intended to be used by rulesets to add replay-specific settings. /// /// The settings group to be shown. - public void AddSettings(PlayerSettingsGroup settings) => Schedule(() => - { - settings.Expanded.Value = false; - HUDOverlay.PlayerSettingsOverlay.Add(settings); - }); + public void AddSettings(PlayerSettingsGroup settings) => Schedule(() => HUDOverlay.PlayerSettingsOverlay.Add(settings)); [BackgroundDependencyLoader] private void load(OsuConfigManager config)