Adjust RankedPlayCard to always show card back (when loading content too)

This was mentioned in vivi's feedback. Basically now the card backside
is always loaded and there, rather than faffing with switching the
content around.

Keeping it simple for now. Can probably hide it when not in use in the
future.

Author: peppy

osu.Game.Tests/Visual/RankedPlay/TestScenePickScreen.cs

@@ -2,15 +2,15 @@
 // See the LICENCE file in the repository root for full licence text.
 
 using osu.Framework.Extensions;
+using osu.Game.Online.API;
 using osu.Game.Online.Multiplayer;
 using osu.Game.Online.Multiplayer.MatchTypes.RankedPlay;
 using osu.Game.Online.Rooms;
 using osu.Game.Screens.OnlinePlay.Matchmaking.RankedPlay;
-using osu.Game.Tests.Visual.Multiplayer;
 
 namespace osu.Game.Tests.Visual.RankedPlay
 {
-    public partial class TestScenePickScreen : MultiplayerTestScene
+    public partial class TestScenePickScreen : RankedPlayTestScene
     {
         private RankedPlayScreen screen = null!;
 
@@ -26,7 +26,26 @@ public override void SetUpSteps()
             AddStep("load screen", () => LoadScreen(screen = new RankedPlayScreen(MultiplayerClient.ClientRoom!)));
             AddUntilStep("screen loaded", () => screen.IsLoaded);
 
+            var requestHandler = new BeatmapRequestHandler();
+
+            AddStep("setup request handler", () => ((DummyAPIAccess)API).HandleRequest = requestHandler.HandleRequest);
+
             AddStep("set pick state", () => MultiplayerClient.RankedPlayChangeStage(RankedPlayStage.CardPlay, state => state.ActiveUserId = API.LocalUser.Value.OnlineID).WaitSafely());
+
+            AddWaitStep("wait some", 5);
+
+            AddStep("reveal cards", () =>
+            {
+                for (int i = 0; i < 5; i++)
+                {
+                    int i2 = i;
+                    MultiplayerClient.RankedPlayRevealCard(hand => hand[i2], new MultiplayerPlaylistItem
+                    {
+                        ID = i2,
+                        BeatmapID = requestHandler.Beatmaps[i2].OnlineID
+                    }).WaitSafely();
+                }
+            });
         }
     }
 }

osu.Game/Screens/OnlinePlay/Matchmaking/RankedPlay/Card/RankedPlayCard.cs

@@ -97,6 +97,7 @@ public RankedPlayCard(RankedPlayCardWithPlaylistItem item)
                         Origin = Anchor.Centre,
                         Children =
                         [
+                            new RankedPlayCardBackSide(),
                             cardContent = new Container
                             {
                                 RelativeSizeAxes = Axes.Both,
@@ -123,9 +124,7 @@ protected override void LoadComplete()
         {
             base.LoadComplete();
 
-            playlistItem.BindValueChanged(e => onPlaylistItemChanged(e.NewValue));
-            if (playlistItem.Value != null)
-                loadCardContent(playlistItem.Value, false);
+            playlistItem.BindValueChanged(e => onPlaylistItemChanged(e.NewValue), true);
         }
 
         protected override void UpdateAfterChildren()
@@ -147,14 +146,14 @@ private void onPlaylistItemChanged(MultiplayerPlaylistItem? playlistItem)
         {
             if (playlistItem == null)
             {
-                SetContent(new RankedPlayCardBackSide(), true);
+                SetContent(null);
                 return;
             }
 
-            loadCardContent(playlistItem, true);
+            loadCardContentAsync(playlistItem);
         }
 
-        private void loadCardContent(MultiplayerPlaylistItem playlistItem, bool flip) => Task.Run(async () =>
+        private void loadCardContentAsync(MultiplayerPlaylistItem playlistItem) => Task.Run(async () =>
         {
             var beatmap = await beatmapLookupCache.GetBeatmapAsync(playlistItem.BeatmapID).ConfigureAwait(false);
 
@@ -168,22 +167,16 @@ private void loadCardContent(MultiplayerPlaylistItem playlistItem, bool flip) =>
 
             Schedule(() =>
             {
-                SetContent(new RankedPlayCardContent(beatmap), flip);
+                SetContent(new RankedPlayCardContent(beatmap));
                 songPreviewContainer.LoadPreview(beatmap);
             });
         });
 
-        public void SetContent(Drawable newContent, bool flip)
+        public void SetContent(Drawable? newContent)
         {
-            if (!flip)
-            {
-                cardContent.Child = newContent;
-                return;
-            }
-
             content.ScaleTo(new Vector2(0, 1), 100, Easing.In)
                    .Then()
-                   .Schedule(() => cardContent.Child = newContent)
+                   .Schedule(() => cardContent.Child = newContent ?? Empty())
                    .ScaleTo(new Vector2(1), 300, Easing.OutElasticQuarter);
 
             SamplePlaybackHelper.PlayWithRandomPitch(cardFlipSample);