From 7b3bd0b04245f16ba4695235b29134871a5a1c96 Mon Sep 17 00:00:00 2001 From: Dean Herbert Date: Tue, 25 Jul 2023 16:15:16 +0900 Subject: [PATCH] Reword comments to make more sense --- osu.Game.Tournament/Components/TournamentBeatmapPanel.cs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/osu.Game.Tournament/Components/TournamentBeatmapPanel.cs b/osu.Game.Tournament/Components/TournamentBeatmapPanel.cs index e364a3c27d..64148fa0cb 100644 --- a/osu.Game.Tournament/Components/TournamentBeatmapPanel.cs +++ b/osu.Game.Tournament/Components/TournamentBeatmapPanel.cs @@ -56,7 +56,7 @@ namespace osu.Game.Tournament.Components RelativeSizeAxes = Axes.Both, Colour = Color4.Black, }, - new TournamentUpdateableOnlineBeatmapSetCover + new NoUnloadBeatmapSetCover { RelativeSizeAxes = Axes.Both, Colour = OsuColour.Gray(0.5f), @@ -181,13 +181,12 @@ namespace osu.Game.Tournament.Components } } - private partial class TournamentUpdateableOnlineBeatmapSetCover : UpdateableOnlineBeatmapSetCover + private partial class NoUnloadBeatmapSetCover : UpdateableOnlineBeatmapSetCover { - // no need to wait for Load because beatmap cover information does not change. + // As covers are displayed on stream, we want them to load as soon as possible. protected override double LoadDelay => 0; - // Use DelayedLoadWrapper to avoid beatmap cover unload in map pool. - // see https://github.com/ppy/osu/discussions/24337 + // Use DelayedLoadWrapper to avoid content unloading when switching away to another screen. protected override DelayedLoadWrapper CreateDelayedLoadWrapper(Func createContentFunc, double timeBeforeLoad) => new DelayedLoadWrapper(createContentFunc, timeBeforeLoad); }