From 4cb9e1d4438895b7176c72464fcd6c65e738ac71 Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 10:33:43 +0200
Subject: [PATCH 1/9] Initial commit

---
 .../TestSceneLegacyBeatmapSkin.cs                        | 2 +-
 osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs    | 9 ++++++---
 osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs    | 5 +++--
 osu.Game/Beatmaps/BeatmapManager_WorkingBeatmap.cs       | 2 +-
 osu.Game/Beatmaps/IWorkingBeatmap.cs                     | 2 +-
 osu.Game/Beatmaps/WorkingBeatmap.cs                      | 8 ++++----
 osu.Game/Skinning/BeatmapSkinProvidingContainer.cs       | 4 ++--
 osu.Game/Skinning/DefaultBeatmapSkin.cs                  | 9 +++++++++
 osu.Game/Skinning/IBeatmapSkin.cs                        | 9 +++++++++
 osu.Game/Skinning/LegacyBeatmapSkin.cs                   | 2 +-
 osu.Game/Tests/Beatmaps/HitObjectSampleTest.cs           | 2 +-
 11 files changed, 38 insertions(+), 16 deletions(-)
 create mode 100644 osu.Game/Skinning/DefaultBeatmapSkin.cs
 create mode 100644 osu.Game/Skinning/IBeatmapSkin.cs

diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
index 3ff37c4147..03d18cefef 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
@@ -110,7 +110,7 @@ namespace osu.Game.Rulesets.Osu.Tests
                 this.hasColours = hasColours;
             }
 
-            protected override ISkin GetSkin() => new TestBeatmapSkin(BeatmapInfo, hasColours);
+            protected override IBeatmapSkin GetSkin() => new TestBeatmapSkin(BeatmapInfo, hasColours);
         }
 
         private class TestBeatmapSkin : LegacyBeatmapSkin
diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
index 075bf314bc..c3c19de17c 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
@@ -80,15 +80,18 @@ namespace osu.Game.Rulesets.Osu.Tests
 
         public class CustomSkinWorkingBeatmap : ClockBackedTestWorkingBeatmap
         {
-            private readonly ISkinSource skin;
+            private readonly ISkinSource skinSource;
 
             public CustomSkinWorkingBeatmap(IBeatmap beatmap, Storyboard storyboard, IFrameBasedClock frameBasedClock, AudioManager audio, ISkinSource skin)
                 : base(beatmap, storyboard, frameBasedClock, audio)
             {
-                this.skin = skin;
+                if (!(skinSource is IBeatmapSkin))
+                    throw new ArgumentException("The provided skin source must be of type IBeatmapSkin.");
+
+                skinSource = skin;
             }
 
-            protected override ISkin GetSkin() => skin;
+            protected override IBeatmapSkin GetSkin() => (IBeatmapSkin)skinSource;
         }
 
         public class SkinProvidingPlayer : TestPlayer
diff --git a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
index b30870d057..996d495e17 100644
--- a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
+++ b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
@@ -116,7 +116,8 @@ namespace osu.Game.Tests.Gameplay
             AddAssert("sample playback rate matches mod rates", () => sample.Channel.AggregateFrequency.Value == expectedRate);
         }
 
-        private class TestSkin : LegacySkin
+        // TODO: adding IBeatmapSkin changes are as minimal as possible, but this shouldn't exist or should be reworked to work with LegacyBeatmapSkin
+        private class TestSkin : LegacySkin, IBeatmapSkin
         {
             public TestSkin(string resourceName, AudioManager audioManager)
                 : base(DefaultLegacySkin.Info, new TestResourceStore(resourceName), audioManager, "skin.ini")
@@ -156,7 +157,7 @@ namespace osu.Game.Tests.Gameplay
                 this.audio = audio;
             }
 
-            protected override ISkin GetSkin() => new TestSkin("test-sample", audio);
+            protected override IBeatmapSkin GetSkin() => new TestSkin("test-sample", audio);
         }
 
         private class TestDrawableStoryboardSample : DrawableStoryboardSample
diff --git a/osu.Game/Beatmaps/BeatmapManager_WorkingBeatmap.cs b/osu.Game/Beatmaps/BeatmapManager_WorkingBeatmap.cs
index 39c5ccab27..44728cc251 100644
--- a/osu.Game/Beatmaps/BeatmapManager_WorkingBeatmap.cs
+++ b/osu.Game/Beatmaps/BeatmapManager_WorkingBeatmap.cs
@@ -140,7 +140,7 @@ namespace osu.Game.Beatmaps
                 return storyboard;
             }
 
-            protected override ISkin GetSkin()
+            protected override IBeatmapSkin GetSkin()
             {
                 try
                 {
diff --git a/osu.Game/Beatmaps/IWorkingBeatmap.cs b/osu.Game/Beatmaps/IWorkingBeatmap.cs
index 31975157a0..dac9389822 100644
--- a/osu.Game/Beatmaps/IWorkingBeatmap.cs
+++ b/osu.Game/Beatmaps/IWorkingBeatmap.cs
@@ -44,7 +44,7 @@ namespace osu.Game.Beatmaps
         /// <summary>
         /// Retrieves the <see cref="Skin"/> which this <see cref="WorkingBeatmap"/> provides.
         /// </summary>
-        ISkin Skin { get; }
+        IBeatmapSkin Skin { get; }
 
         /// <summary>
         /// Constructs a playable <see cref="IBeatmap"/> from <see cref="Beatmap"/> using the applicable converters for a specific <see cref="RulesetInfo"/>.
diff --git a/osu.Game/Beatmaps/WorkingBeatmap.cs b/osu.Game/Beatmaps/WorkingBeatmap.cs
index b4bcf285b9..163b62a55c 100644
--- a/osu.Game/Beatmaps/WorkingBeatmap.cs
+++ b/osu.Game/Beatmaps/WorkingBeatmap.cs
@@ -44,7 +44,7 @@ namespace osu.Game.Beatmaps
             background = new RecyclableLazy<Texture>(GetBackground, BackgroundStillValid);
             waveform = new RecyclableLazy<Waveform>(GetWaveform);
             storyboard = new RecyclableLazy<Storyboard>(GetStoryboard);
-            skin = new RecyclableLazy<ISkin>(GetSkin);
+            skin = new RecyclableLazy<IBeatmapSkin>(GetSkin);
 
             total_count.Value++;
         }
@@ -275,10 +275,10 @@ namespace osu.Game.Beatmaps
         private readonly RecyclableLazy<Storyboard> storyboard;
 
         public bool SkinLoaded => skin.IsResultAvailable;
-        public ISkin Skin => skin.Value;
+        public IBeatmapSkin Skin => skin.Value;
 
-        protected virtual ISkin GetSkin() => new DefaultSkin();
-        private readonly RecyclableLazy<ISkin> skin;
+        protected virtual IBeatmapSkin GetSkin() => new DefaultBeatmapSkin();
+        private readonly RecyclableLazy<IBeatmapSkin> skin;
 
         /// <summary>
         /// Transfer pieces of a beatmap to a new one, where possible, to save on loading.
diff --git a/osu.Game/Skinning/BeatmapSkinProvidingContainer.cs b/osu.Game/Skinning/BeatmapSkinProvidingContainer.cs
index 40335db697..346bfe53b8 100644
--- a/osu.Game/Skinning/BeatmapSkinProvidingContainer.cs
+++ b/osu.Game/Skinning/BeatmapSkinProvidingContainer.cs
@@ -11,7 +11,7 @@ namespace osu.Game.Skinning
     /// <summary>
     /// A container which overrides existing skin options with beatmap-local values.
     /// </summary>
-    public class BeatmapSkinProvidingContainer : SkinProvidingContainer
+    public class BeatmapSkinProvidingContainer : SkinProvidingContainer, IBeatmapSkin
     {
         private readonly Bindable<bool> beatmapSkins = new Bindable<bool>();
         private readonly Bindable<bool> beatmapHitsounds = new Bindable<bool>();
@@ -21,7 +21,7 @@ namespace osu.Game.Skinning
         protected override bool AllowTextureLookup(string componentName) => beatmapSkins.Value;
         protected override bool AllowSampleLookup(ISampleInfo componentName) => beatmapHitsounds.Value;
 
-        public BeatmapSkinProvidingContainer(ISkin skin)
+        public BeatmapSkinProvidingContainer(IBeatmapSkin skin)
             : base(skin)
         {
         }
diff --git a/osu.Game/Skinning/DefaultBeatmapSkin.cs b/osu.Game/Skinning/DefaultBeatmapSkin.cs
new file mode 100644
index 0000000000..7b5ccd45c3
--- /dev/null
+++ b/osu.Game/Skinning/DefaultBeatmapSkin.cs
@@ -0,0 +1,9 @@
+// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
+// See the LICENCE file in the repository root for full licence text.
+
+namespace osu.Game.Skinning
+{
+    public class DefaultBeatmapSkin : DefaultSkin, IBeatmapSkin
+    {
+    }
+}
diff --git a/osu.Game/Skinning/IBeatmapSkin.cs b/osu.Game/Skinning/IBeatmapSkin.cs
new file mode 100644
index 0000000000..77c34b8ad7
--- /dev/null
+++ b/osu.Game/Skinning/IBeatmapSkin.cs
@@ -0,0 +1,9 @@
+// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
+// See the LICENCE file in the repository root for full licence text.
+
+namespace osu.Game.Skinning
+{
+    public interface IBeatmapSkin : ISkin
+    {
+    }
+}
diff --git a/osu.Game/Skinning/LegacyBeatmapSkin.cs b/osu.Game/Skinning/LegacyBeatmapSkin.cs
index d647bc4a2d..d53349dd11 100644
--- a/osu.Game/Skinning/LegacyBeatmapSkin.cs
+++ b/osu.Game/Skinning/LegacyBeatmapSkin.cs
@@ -11,7 +11,7 @@ using osu.Game.Rulesets.Objects.Legacy;
 
 namespace osu.Game.Skinning
 {
-    public class LegacyBeatmapSkin : LegacySkin
+    public class LegacyBeatmapSkin : LegacySkin, IBeatmapSkin
     {
         protected override bool AllowManiaSkin => false;
         protected override bool UseCustomSampleBanks => true;
diff --git a/osu.Game/Tests/Beatmaps/HitObjectSampleTest.cs b/osu.Game/Tests/Beatmaps/HitObjectSampleTest.cs
index ab4fb38657..db080d889f 100644
--- a/osu.Game/Tests/Beatmaps/HitObjectSampleTest.cs
+++ b/osu.Game/Tests/Beatmaps/HitObjectSampleTest.cs
@@ -188,7 +188,7 @@ namespace osu.Game.Tests.Beatmaps
                 this.resourceStore = resourceStore;
             }
 
-            protected override ISkin GetSkin() => new LegacyBeatmapSkin(skinBeatmapInfo, resourceStore, AudioManager);
+            protected override IBeatmapSkin GetSkin() => new LegacyBeatmapSkin(skinBeatmapInfo, resourceStore, AudioManager);
         }
     }
 }

From 1b81415a16e82131bb170f5473202112a82182ab Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 10:50:25 +0200
Subject: [PATCH 2/9] Correct comment

---
 osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
index 996d495e17..1e3755c186 100644
--- a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
+++ b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
@@ -116,7 +116,7 @@ namespace osu.Game.Tests.Gameplay
             AddAssert("sample playback rate matches mod rates", () => sample.Channel.AggregateFrequency.Value == expectedRate);
         }
 
-        // TODO: adding IBeatmapSkin changes are as minimal as possible, but this shouldn't exist or should be reworked to work with LegacyBeatmapSkin
+        // TODO: adding IBeatmapSkin to keep changes as minimal as possible, but this shouldn't exist or should be reworked to inherit LegacyBeatmapSkin
         private class TestSkin : LegacySkin, IBeatmapSkin
         {
             public TestSkin(string resourceName, AudioManager audioManager)

From 08329aa382d7afd64e9ce5afe30a94d55d0557ec Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 11:05:10 +0200
Subject: [PATCH 3/9] Remove comment again

---
 osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs | 1 -
 1 file changed, 1 deletion(-)

diff --git a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
index 1e3755c186..bc9528beb6 100644
--- a/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
+++ b/osu.Game.Tests/Gameplay/TestSceneStoryboardSamples.cs
@@ -116,7 +116,6 @@ namespace osu.Game.Tests.Gameplay
             AddAssert("sample playback rate matches mod rates", () => sample.Channel.AggregateFrequency.Value == expectedRate);
         }
 
-        // TODO: adding IBeatmapSkin to keep changes as minimal as possible, but this shouldn't exist or should be reworked to inherit LegacyBeatmapSkin
         private class TestSkin : LegacySkin, IBeatmapSkin
         {
             public TestSkin(string resourceName, AudioManager audioManager)

From 82acb3506cbc36a0546ff77fc76c20d8854bb2ed Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 11:07:28 +0200
Subject: [PATCH 4/9] Add and change xmldocs

---
 osu.Game/Beatmaps/IWorkingBeatmap.cs | 2 +-
 osu.Game/Skinning/IBeatmapSkin.cs    | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/osu.Game/Beatmaps/IWorkingBeatmap.cs b/osu.Game/Beatmaps/IWorkingBeatmap.cs
index dac9389822..aac41725a9 100644
--- a/osu.Game/Beatmaps/IWorkingBeatmap.cs
+++ b/osu.Game/Beatmaps/IWorkingBeatmap.cs
@@ -42,7 +42,7 @@ namespace osu.Game.Beatmaps
         Storyboard Storyboard { get; }
 
         /// <summary>
-        /// Retrieves the <see cref="Skin"/> which this <see cref="WorkingBeatmap"/> provides.
+        /// Retrieves the <see cref="IBeatmapSkin"/> which this <see cref="WorkingBeatmap"/> provides.
         /// </summary>
         IBeatmapSkin Skin { get; }
 
diff --git a/osu.Game/Skinning/IBeatmapSkin.cs b/osu.Game/Skinning/IBeatmapSkin.cs
index 77c34b8ad7..91caaed557 100644
--- a/osu.Game/Skinning/IBeatmapSkin.cs
+++ b/osu.Game/Skinning/IBeatmapSkin.cs
@@ -3,6 +3,9 @@
 
 namespace osu.Game.Skinning
 {
+    /// <summary>
+    /// Marker interface for skins that originate from beatmaps.
+    /// </summary>
     public interface IBeatmapSkin : ISkin
     {
     }

From 9ce9ba3a0d1906448611749a0dc391dcf3944e3d Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 13:50:29 +0200
Subject: [PATCH 5/9] Update TestSceneSkinFallbacks.cs

---
 .../TestSceneSkinFallbacks.cs                        | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
index c3c19de17c..0fe8949360 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
@@ -34,7 +34,7 @@ namespace osu.Game.Rulesets.Osu.Tests
         public TestSceneSkinFallbacks()
         {
             testUserSkin = new TestSource("user");
-            testBeatmapSkin = new TestSource("beatmap");
+            testBeatmapSkin = new BeatmapTestSource();
         }
 
         [Test]
@@ -85,7 +85,7 @@ namespace osu.Game.Rulesets.Osu.Tests
             public CustomSkinWorkingBeatmap(IBeatmap beatmap, Storyboard storyboard, IFrameBasedClock frameBasedClock, AudioManager audio, ISkinSource skin)
                 : base(beatmap, storyboard, frameBasedClock, audio)
             {
-                if (!(skinSource is IBeatmapSkin))
+                if (!(skin is IBeatmapSkin))
                     throw new ArgumentException("The provided skin source must be of type IBeatmapSkin.");
 
                 skinSource = skin;
@@ -115,6 +115,14 @@ namespace osu.Game.Rulesets.Osu.Tests
             }
         }
 
+        public class BeatmapTestSource : TestSource, IBeatmapSkin
+        {
+            public BeatmapTestSource()
+                : base("beatmap")
+            {
+            }
+        }
+
         public class TestSource : ISkinSource
         {
             private readonly string identifier;

From 43e91877a71c3451f4b9617212dbd0c38f7bdb2f Mon Sep 17 00:00:00 2001
From: Craftplacer <craftplacer@outlook.de>
Date: Sat, 29 Aug 2020 14:47:26 +0200
Subject: [PATCH 6/9] Scope and limit parameter to IBeatmapSkin

---
 .../TestSceneSkinFallbacks.cs                     | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
index 0fe8949360..64da80a88e 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneSkinFallbacks.cs
@@ -29,7 +29,7 @@ namespace osu.Game.Rulesets.Osu.Tests
     public class TestSceneSkinFallbacks : TestSceneOsuPlayer
     {
         private readonly TestSource testUserSkin;
-        private readonly TestSource testBeatmapSkin;
+        private readonly BeatmapTestSource testBeatmapSkin;
 
         public TestSceneSkinFallbacks()
         {
@@ -80,18 +80,15 @@ namespace osu.Game.Rulesets.Osu.Tests
 
         public class CustomSkinWorkingBeatmap : ClockBackedTestWorkingBeatmap
         {
-            private readonly ISkinSource skinSource;
+            private readonly IBeatmapSkin skin;
 
-            public CustomSkinWorkingBeatmap(IBeatmap beatmap, Storyboard storyboard, IFrameBasedClock frameBasedClock, AudioManager audio, ISkinSource skin)
+            public CustomSkinWorkingBeatmap(IBeatmap beatmap, Storyboard storyboard, IFrameBasedClock frameBasedClock, AudioManager audio, IBeatmapSkin skin)
                 : base(beatmap, storyboard, frameBasedClock, audio)
             {
-                if (!(skin is IBeatmapSkin))
-                    throw new ArgumentException("The provided skin source must be of type IBeatmapSkin.");
-
-                skinSource = skin;
+                this.skin = skin;
             }
 
-            protected override IBeatmapSkin GetSkin() => (IBeatmapSkin)skinSource;
+            protected override IBeatmapSkin GetSkin() => skin;
         }
 
         public class SkinProvidingPlayer : TestPlayer
@@ -115,7 +112,7 @@ namespace osu.Game.Rulesets.Osu.Tests
             }
         }
 
-        public class BeatmapTestSource : TestSource, IBeatmapSkin
+        private class BeatmapTestSource : TestSource, IBeatmapSkin
         {
             public BeatmapTestSource()
                 : base("beatmap")

From d22768a98cba3c9d312ba2408536905371840668 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= <dach.bartlomiej@gmail.com>
Date: Sat, 29 Aug 2020 23:20:59 +0200
Subject: [PATCH 7/9] Add scale specification to spinner scene for visibility

---
 osu.Game.Rulesets.Osu.Tests/TestSceneSpinner.cs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneSpinner.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneSpinner.cs
index 47b3926ceb..94d1cb8864 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneSpinner.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneSpinner.cs
@@ -9,6 +9,7 @@ using osu.Game.Beatmaps.ControlPoints;
 using osu.Game.Rulesets.Mods;
 using osu.Game.Rulesets.Osu.Objects;
 using osu.Game.Rulesets.Osu.Objects.Drawables;
+using osuTK;
 
 namespace osu.Game.Rulesets.Osu.Tests
 {
@@ -62,7 +63,8 @@ namespace osu.Game.Rulesets.Osu.Tests
             drawableSpinner = new TestDrawableSpinner(spinner, auto)
             {
                 Anchor = Anchor.Centre,
-                Depth = depthIndex++
+                Depth = depthIndex++,
+                Scale = new Vector2(0.75f)
             };
 
             foreach (var mod in SelectedMods.Value.OfType<IApplicableToDrawableHitObjects>())

From c9723e541a464c84b8e3e5c27e94af4f0c3fd32d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= <dach.bartlomiej@gmail.com>
Date: Sat, 29 Aug 2020 23:21:19 +0200
Subject: [PATCH 8/9] Add metrics skin resources for old style spinner

---
 .../metrics-skin/spinner-background@2x.png      | Bin 0 -> 14515 bytes
 .../metrics-skin/spinner-circle@2x.png          | Bin 0 -> 6561 bytes
 .../Resources/metrics-skin/spinner-metre@2x.png | Bin 0 -> 14516 bytes
 3 files changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-background@2x.png
 create mode 100644 osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-circle@2x.png
 create mode 100644 osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-metre@2x.png

diff --git a/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-background@2x.png b/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-background@2x.png
new file mode 100644
index 0000000000000000000000000000000000000000..4f50f638c5de06d711fc7d09a3a4ee13618cd59e
GIT binary patch
literal 14515
zcmeHNO)G>^6n=(b#_-liY7o6PHa3!?BrnZ0Grp3Pln{j-@{v*u+4u)Seu8Xl*pOm;
z6rvQ$EbPd_Mxi)&ym$V<Y%tHM>7L$?+uU=Wd(Lz2dzRbVf`+Hi1HcG}{2hQ>)Qk~#
zrVQ0L^+z&f4m1b-G6G-M4V|wtW{riqCxGlc^>bi#-6t1ali}8Y>(O1DpIcjAr!JWY
z`x`oEZlCY>4U~EhteVUD)7h#1p{D5F$;S4OL(M7D_0jU90B<tok600{A<d~(uc@^N
z^xJT%zZ@6SvK7FQrd79BYn&3aB|s2?y9W##5}+9`nJ6UXGeeRZkQxY0BsCy45E!tc
zAT<yeu!bZx07wl;4M+{d0)Lts#8zYhGq)=+2iKIg=29zCl1%k8H~kWG0!s_N#gFsn
zlHtn6hoFPGQ>@C|nG~>iHlwyCK|~OdfT^9pPhdQo`P&4{QZSASqk2k3_0NLiL%>(1
zdI7}$^nOZtpB`5+FPR$i_ml!#WwMmsqIiqWMJ&;OLvBKDLT;kAIlmi$EhYbp;`HoJ
zo9P1Wis-|8N#D2}cJ4(K)Z>z#XtK;n^k7z)_Dw!W^Jpj#6xtFXD6}aE3T+93g6YG?
s0<$YQg$)J9N5)6n0;Ja(3XG2_6dx^P)VSK|ID1mw4+q-(n~nX8-{y?j8UO$Q

literal 0
HcmV?d00001

diff --git a/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-circle@2x.png b/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-circle@2x.png
new file mode 100644
index 0000000000000000000000000000000000000000..daf28e09cbe6bbb7fc531b4773e4d381054e65cf
GIT binary patch
literal 6561
zcmai32UHVT+fF7SA(Vt}D24z5L;(S5(u{(VP&J|=H6lf-EK(xPWJO>BAp{XriekhK
zA|N(Ez*$+Ast8g90fUW>QKT#g{NuN-{@*$OIsZ8`XYPIPo#!p{&O7%$^V@zWTX7Ku
z5f}_6PNmq;U@$oQ+bN8KB+SRgozMm4V{dB%U16||-rX*rp(`ewa`Y4oCMx~ygu^Zu
z;2|M8lIpk@{Z4qZEJieI!*CV`lh>u%>~>)c&5k5w5d3OeK7I6TE_G{<4&@Hf(gF{8
z+1*iCl``y;lEI$PA>6YCiSSs}-9Sp(fjuHc0#zgO5JiP9(j2=#j?bJ>kuwR}Vl+f+
zPp`AF3%s8n^@tpGu)A=+B?B$tHdJ^i;6u^hiJY}ZU8hEtydHb)FkfPlsGR@jm^4D#
zU;T?Zff~hTphr;|9f@@;T|SX67ETn%(OnkjOJ~N<gNj~@tJ$zK=ylXrqgDCN#C5TH
zm<7%PEe2a7doRx0($NAmu%??SCcztG;&Be}QH&DG0i^*?fl1%h`~CWEx}rd=O7<mS
zA@mi?;DFdoM^LFQ4*MNYnLs8IiylSPn_p4Ad;+#CE(qph`1<?<fcvnR0hw!=(kuzC
zGUY*iFp^$=4o=ss!Z7eJVPDh0w-@)KQit$zo}dz4PEUH7eD-FV!OdK6*fcAJiN=Vz
z%E=Rh+~|>rQ!E@@xwBME_9Q~EgMX8pWiso>+|vI%AKi(t;yB7r)h~bu&<zOINfDFi
zGt$!lG-1k!O!PkY3vPo#!?>@)%POqUi#Dj#GO_c4U>isULqw%=qre8x5^3y?l_yTw
zoNz$P!RA@lbtq~Uz_3ucN>y7#E9^k_HGLsd*<~4lmuSxkMb<}nj1;2>8t4w+bKI~>
zVp(kReE9t?1Q^5YLq;EkkYXG7`RD>vyqqcct1v^d2I2bN`&1jOdsoUnlW+~ePWok`
zZm$s3Vc_-Wozn8eS#0JYcoZBT5gqzCf_Ml#lyD8$DJF4${Ky%VaO~uZF`Abp=!iV*
zE`qRmDt0~_+{D}qx`>uYeF02>4$RC0B~<DgT;^HUe6=!8+eK3h*1_~c7Ty0Q7R|I~
zi8AL|l5#e(cpwZ`p(eQZUXSdu!Os`-xZtQn^VCLMT#3*|*er-;V!&o_lTLEbOe}vI
z`QV$nK6DG7FU7ZK@a%rU0~M$WRYCo?+Z`F&sKyMF7TKF{z8x^&3Qa8(&D_DPd{g|k
zOg@%XDOvNBh9_>_Zu$Yp8$&RIN(pofi(r3@`FjmNAEvEh8Eq3(=g#b??+#;bIm7Tk
zdcZTP@j=<U`82>@=N!Cb4VQTuJOk{|Np=gd6v$>}n~?9F{6k-%Chg1cRx0m3sNVRp
z9oAjcbX+A|T(8@QQduqTI>UR6O7)UgLL&w^jhOvCqql+ern3s-66yumcmzH`pPSH+
z_Mq^iAw%xzo7OR>Dc)+^0vU&aO%nJj85G6+$y;e|i-&VHo)+fLn#T->YMFI3Etx43
z27nlO0_~UvJjU;_djt^Z8}5K^#B@Hu(*pb98&33`8+zo+QiOn<#%)8tId6R}QI!fl
zz4!$7o25k<^4Wa}TFY%>RjUD>sE=wf<ojEs*_8Ff@<V_tkh4<u3~iF-fnJ2fjVW;;
z;BREyr+N+Vc!mcy{*{0obX-O-I}>xziFjWuGaC#Ad2-fex6N5Qgl8T=CVM`4E6k6k
zCpWW$P9nn1SS=7BEs9Y<iLH6%ny#|<xMsKlYgM~ha2iMQW)z5i1qJMq9dgUN7tOU1
z+_z(QlD>?Qsh{ffS3Udn7?@zj*jN*bc6a&tH8?FcJOkD!s$GSry)B2Y=Ynj=$uw+M
z7=IP?NLMjBj0+cGB|@>3ktO8Z-6*Ud@S&W4DCtV3r$ZAzC%0sSXAlQ<l6w^zKuwH=
zzm^z#EV*Ts>b-9GhY#iU-Oc&V{5&W|gRceo;$Lxk&mdtRI=%ojgD_`;8A_tVh{&U2
zyag092lI6pN~l#S!V-W!Dn`9D&dd>$MEIT@(^stNYP0SkXk<$_6!<ag4>RhOmsLjR
z+4lkxK+LGC3E7K4umv{K&xEQ^E`*}3rizyph#L%wATj=#bx-58Q0B?irsP>H`wN4k
z{8st{XYutuevPqmT+b+sireRr9^g(@Xe7|R+;Q}f`BidbFoV(_7}js^#w&z#8oQd*
z2GwPc?e0G)#+FNJk&2hU!1XD7^_Pj_gJ}X~W&OCf^){Y%x^Ov^ADal8z^?E!_FD%Z
zOVo710-YF2@UKev>CuZW)rH_4B*BFI2A7o$o(090?sKt&uuhe#M{VBK;wIfEcUX|3
zUS^ap8M@s?kj5hTE=$#1k%G$$SN{r{Q!h_C2R<BW;%r?!i9D=G{FKyUNVn8$Z`j83
zISV=fSFS8JUyj90`!`wB95dQuX>Sd;lY7*J{ObbL`&v6S2r)h7_O6%!gh}lY^}gPn
zNgo{4o?_TZliV|LmQO6V@=BAL@<?N&u?|DtVB!k_TV{9Hntx8CgLG+DFl%tmkt&&A
zxKKXPd<L^^Od9l@qf)akIrGC1+&aMoU<}V0PUooJmCbje6Z^MItg4q!i{Yr*moF^n
zX|hF1vt9)Lam<gS@KFqSs7Ndn%{t4nW&JW|NzI-{WU2~6LX<7~?Og{H2pk0;kNy`0
zWHY@4MtcQUBW<pCv<4s@GZY~8x*Ifj#~#2_q+iN*pfhKs`VS?iaFZ|RFPLhwrAo6-
zgIOcTISO;8K(IL52k}nDqSTW%e_K;;TQW|e1&UufWRZ^kj_K1*M4_Y><kd~D^|tUV
zQ7j(vfjR5l-9H8FUW4n^qvJ@<@s)hhuYWd>csHyZuC`!&8M>!7C_#7JmcSJh<%vA#
zv6RJ?yK-gWiCfu62>3N{aAX(9ts)P6R6t&|w2)IIaFAecxuzLw#+s1<FGAG~n?Mp+
z_!ic9A$f65s>zz;b#2@9c_(7rB`Xbfqb4J1buY*3m~g(!%@on;{@=G7TaW^?)>eEp
z*#kvZ8VKiFTT);pR;{WT^Db96!ka>~Xs9MgQ=VMgpJkKF6?V`N+>@qE0ZV5|&3!v=
z6_cf~sKhLjF7-ffarj4~Amo)Mdmi$C=;LUD*CS1KQZ|T<58uyO9DwN+fpZU>h~9dT
zM%TJ(!q5=rPw5!-z~!x$yoYrd(y?n}D%4tsr?G(@0P@){v+7!-UT#kzw+F$q6^}@=
zK3G;%=q2*Ahz517xuT)JF{EpX6W5<rzR2a)kU&#l&9RvL%+P|w6CC4r8aoiPU@tsQ
zQn&W{1GXkH05v~=L7{1t`dL1>1_y4|{>ACdAiS!DQuR73kq;Yi<=A~veeOs<@<M}s
zE;)siJH;#?h_=y0yxUQ6n`CrzAo_#U;U_Xz2cmMhnfZ47wo(V;&AK8^um0xA+CmYw
zbM5!`d+#T9`B;mYSUZ~SW^4Mv8R=GF-~h+0;>+l(-(Z(Mn%U7>a!2Dj8j^DL#2J4^
zUEta&Inqy+OSiUF6q62>_7XgznFa4ws5C#j*+1TULTJJkZ=voDtV)&61I#<*fFoz?
z5x4ESJIGca>Zr8$cGJy%UQc4-_}L+7R<xw4UJ-_ru<arg&OSxY{wpFKbxB5DMmgM}
z)U9NzH?dS&=!Kq~^v7t<bMdcj3mmT7>20j_I%nc3PfcoB;gh&i^6<|p15{4XO~I(g
zK4$#$C=Pc>tKMW&9FBBN^yQ=(m8NrEU2p#Y#y4|Xje7ski7Z_+vVW}=$Ho2J;zS9;
zxAb4}3Zd2MYKIY|LxHIGAQ4I=Nhr8|Zb$lW1<MfyRs;JuL6ucrp1#b^a8D9%QDnMr
zFRE3)BAwLdb*!PT=E?+SZU697c`8jST56)A3(3(N6tKBlGM>gpI|9he2uG5Z!nrRq
zCBQq%fHkM&0HsTJD;XPO`9o}3kwV&gon7SF>3!^U(r=_@E?dp<vojI%Rvbr7vpdrN
zXd6}{|BV}k78$WLrwY6)tT;>Bs~PiASCx!CwV9-4RvQ;12{#ywVsj_n&5i6uwvq;R
zb4qd^#cYx!rqp0LGKIqp<{PJom^1KwwQ?Mp+}*&*iZFmW0liyVcGs&3UI&=qD^CP9
z?QjL5$b|ft`<Z{6qyB5^T5h$mj8T|;z(36zxGEJzQHS@0e(UG1%U*1$v8UBXEC}l1
z5$uIL?uqwPW2X_m89$ou$kW@Y-R*r%^Qcz$^2emT+m7J66p2M=)v2E#skj8yfV>sM
zHaYr_+At=C1E*1$_1!D>w0r--{fzp7K&e{i$lUg@j89s~Asvz={-%E6yT*4P`@K0O
zxwlfr#o3R)V?o}m2mQC*1}FZz|2g0XcB|-5gz?)y4DI>}hWcs#+RwRG-x2=xWS85I
z07kq1Xs*-$QOsD6xvcqB2>;%ULUTc3>TBCENcxp&BDK41<WagXJHP`?O>>$X^|WDz
zME!Wa)y+Sj@4=rLkT?#Zd`}GO_s5>H$7<gbD61EsJQS0_yQg4~e3O5(%!T=N2pTvc
z({ndngstfT<=8$I%SXS%PIvwk+j{;ZHkVoaMQTN5!!v<O<9*BT7f7J0Eh?Kkw5|Ne
zCV3+6kuJw2?%cwZD#G{dv?}%f5!Xsx19IZ~Itp!8w*Qv(HRTR9M-uOu$j&F+49JX4
z6<MUd!`Ce9CSXTDu25*9vP0L?vFsENWop^&J#mrUOXNCcVMR8nug4WYEEJG?a36~2
z?PzNuTMR7HaO~bLkDn#r;P(=sB(jl;83Btuwo%J+A4*MBE+ILI<c)$OoT=jp-D3xw
zm|Q|Q^bs?*n{4wFYIPg1=LB89P^4tHT7%gRyY6X1E$h_<(0>`SV8b8(Em)}5EDvbF
zBe0>S)818@72shGO;bTWXf|STl^KP>n$+B*M-DSCC=z*bp1Y`J4m``68CY%0LJqh6
z!qC|caW?4BmmJv7F`LUSN86f1;P1cJR&z;Wu!k*ms-`FI<lk_qm&lMv9d+JuWB&$|
z6(~q?6*oyy4z{sj6Rk(6`M{UDQckbrL5#&<28OhzO8BHpeIIvKdaSz^Monme%Ke8g
zL(lJ2UeHsRKx4xb)Oj-7fe~c1Q7}M#{CsmoB`Nnqrqemw00ilN`$8_)|I5qgpDObN
z!g^1mE^_yTYJ^8>Zu}uB8{d{&ahKHm+Wl5OxsKInw;oCjDn;ULuRb1>rmP5^jA~Hl
z={<}#vna!`dvz)vlA1>j-omg!v#TvedEDXH>Y^bIj5_kJ4k`u>_l6V)F^`2x>()7j
z+6~CjLw&ofO5SWga~*n9ep!58rpboByH`6Bh}2a-7~Yv4W3I{mlD_?vGNmWc{jeXy
zWfzNku=Nk{BX2$8GO5`|n1N;m-1;ifXEJ*aW>TX}3M?MkAI0cB4p48htywyA80wH_
z4#vWBA?5cLzw7rTMn$u^0}sowV+V>cq{2Pl77ix=Svd5*O>;~L^SCKgJKCT^jf+h1
zd~)4Onqqd&0rudHw=cD4mm;7c`O+&7=Tf<^Q0_y}(r=`O_P!{QtQAM%q>KTIp-}9U
z$326GVbulkg;p+Wy0SNf>U!2xsT&{L>(1F)YO?b*B%P;nxoOVB=YxUMZ3EDv$K|yb
zKFISyQZek*w<S_p3YapzGB&k7UTr@N^~?J$t485c6W2=en(b-LVQsPW^X`!&H*lL4
zf$PK095)|{e7A?=R&Oqods1IvJ*E;ly{dWrmKJaF$zGRj2!MZGCCO+w7P#xPd{w`2
z=Ys4~E>|wKbf8(9$hr?kEM4TH+=~|KC_@nrQu`3jFzJy1j)K2pxGBriiTM7{5ncM7
zRR<bECsWJfjVRmD<}l*ca4<)~_fY>U)!-I=J5xME20z}IBU5X|-vV^~wfM)|_<3))
z3DJ=ljspJ)zDW(*L0G?wrr(Hjru})?zd^glSm|z(E?ru$En$n6fjeswiSk(k)Jk;E
zxGm3*q;OOt6FbKw&oH?v5hhi=4O_HS?eC7to?Z=2OF~2IY<sHF!%VP1BR*S0s5O#q
ztya2~>S%=QBv$b--laRA8hf($&N)+!9{jE~)?NNhw?F+{oC{6-c5AH5S+_9hzvm>V
z3sO(bt;E<#fnaRBcm)^Qaqhr`gbi&9{vbXcdF@%mYr8Vm-OG5wQ}O?}e558u{m?`j
z-g$6j$@o<yW<#i8%Iae8u^6f4$ehP?MZlLNAnG-(Y4fa&VW)&=!anDPtXs!S0!7%F
zNN4*gW+6D~h|7vbQWR9P5I1Wp)uX_`qg{dZi%P4KHMnAzJeP{SM=Jx*MHec{y23r?
z2zf5QcHAYrD7iSa4`Tw1*S%Jk^%c67aKv;^aGv?tDhMn!a0~h+OmCd`u73(Q09-{o
zqc-dnAc}#&x@MgC_hO^Dny26(^ZYuNqL6$Z)Cv{Pibg}~*SL9iV%I#`_mpE<LPv3d
zI<H4Jy#(T=Ue0|?;wDsWUiR%bcYFFRZ2EMo=K=bp*}ckoyQbxD2QcepR39U)uwt%a
z)XHP9Wk&xtUuOMQn|`<^q6^xNZl6~QkLXwPWzOi^Bw`ERkyI!Ba$2ZIPqN|F&MFbA
zqte+!6*$vwfgX9$<l9|*?*MA({mrI-bR0&thFibO18qO;l;nlRva({EhGA`rs#)-s
zx6;!z_Yh>}l5><eyJU~AXt>g3tqJGyEwt&g6eJ|F<|^l-%kSW6YPVG^@poQBJ7mA4
z*8R~T^CHmpyj^6^Biueive_n5y5MOt?V!i#UwOM_Ux*mRW5xj!H(@Qe^r1xsHw4!U
zGO<boAFR(kF@tnc3opKbx7vQ}eHMR}d7LFKXPx#0#dmFLg7NHAEwG*NxqEop?Ga38
zTVjA8vsc3=T6)<q{2W9;Oo{--8da}oGJT6K_0X172i%gbawtr~G#w1GA45u6YgeND
zfoYv&7v2Ks00I7{V`D6xe-5|V?$FWQbUD`3izVg8H$STvIIh&EvUGA!rhfT56QW3@
zDbhYG{&du`7o6ui&6H10f%n{#7_^~dt9pGXH(c%LSeE0(l7pd+g?R!DA4TUhvm;)K
z6hM1WNN-Q_%SOd~Y&m4M?l#tBa*8y?{bu$H)=Ah~?c{r1D@;+$aF|vBc}QvOA;LDB
z(7I)E;Wm(IU{OH+(jMshzX>L5mJ=~{*3?^E39J1O-?L2{jWw|llbE%%ftK(%Q*73r
z?0!Uxq-hqY+i~Dw^o>~in$#~z-Z+1kD`G&NRD4)F#^$tYqLyCnmq>4(co@7?f#5{k
z8m(|i)C1LdA0Kq}2T@i}+=*BYGQLE)GcDA(k8k{qJUg|!8ozAt+)rQT+7IHbFRAq(
zj?UTszc{)CwgXHT)p$bR<@f-?TeE5e%8oK`wgXBRcy<?GmhH!Fxqb4E3@UX>nUn%-
zf{nkup>ybX>X-XM1CorP2s|-uI|ci!G<#m)|H$k(@rvn1Nk;!S@h-|fz#AqvR=iQB
z)Bv6bBY;ObDew~i<hDJ0rFi0j<_UP%=d0ETn^_Ug|4D2<l#N6L0L6&XqY|^t{yhV&
ziRt9W5Ut3~OKG#<8}jdha5)<=9o!6V{fEzO@M346<bYzhmCtUsNjTpe40_$Hg0N{4
z&(Q*T;7#U|8++d?;doq44w!S%8kL$RmNN+gbgS@tLV-!P%tuTkr13XqcFHE-0pWyl
z0{q?Czn3HCgyvML2ETE<xyKA7P2tP<a0|VmlHPV*ej?u%#9>12k_^Z$1LRBSPPGR3
zt{1<$_*hP!7u;W8xF9Ccte`Q%b#6E#ybRU9k_$fHiRrFD2u~VuLo^@J*<PB1<L}N|
z4Tdf6HTJVWT2Sa<H8iYf{a%=#Fc=vjrnea%d<^yy69N#Pm1cNLGKQh?>_w%mIjrkU
zHC7XA3HC8eaL)kt6}4<6Knn1J-?ty*z*S{8X}@J@f#$3xW*bQ2OVCyI%1VplC0?Q|
z5FtVjV9xNzh$2!|Q*$m{^^l_82>6Qi34F!;WzadsxIqb-0mmXoIYQ62@Er%c6j>(q
zGfQglb?jI1<>18~`D<V08E^)Hp?gJjh1EDSo&{q0xY6E=|IdF2=tl)lb{%)>)ZUv%
U!O2zVX9|qE*U6@05B=Bw0VYd*8vp<R

literal 0
HcmV?d00001

diff --git a/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-metre@2x.png b/osu.Game.Rulesets.Osu.Tests/Resources/metrics-skin/spinner-metre@2x.png
new file mode 100644
index 0000000000000000000000000000000000000000..6ef1068420cf8abb9b12781061801321e3ea298c
GIT binary patch
literal 14516
zcmeHNPbh<N7=AXNZEVXVg-l=6&Mv0qVAEt@B1y@~zZAC>n&Wa)oHgy>;=0`AK-hts
z0}&;PqoOHpc;DH#+pbEUxAuGAw(tA4-aXIrzP~?fBf|s!+Ll@Xek&0l1w0Z~&wE_r
zy03dr#kpn%2jb#^ugln2L3~XzF|i1E8}#MCR<=prbT3&${q84Eu+bY?Zgal?CaidD
zY~}X(_P8FVFTZ>8*|QmWwK|?MZ<!m1+*4?k0fE$zI<+r(<c~$ub_AzkhXK=$($M>h
zX6R2Yni(wxfWv8ro`4<kkzg(bZl(<gf(RgcK-r*PiX^a5fEAJzQkcLB3AP4o4TK47
z4cHp6HINLjHIN)|A%d*|*cu2ExN6YWlS(?_%SYFv@00s?kL?=|AJY})kXHt9v~X^q
zc~+JsAHsDisA_Dq$tHmT*~CU$V8A_#3i>JX`D4^e>ZKrodP%($7*H?&f?n3|1m}RJ
zE<Fn%e;RjLxRfF=;8F@Ff0dR}rt~;aHXM=>+s9Sv)L8Xd^#um3`mCYcj!;4TjT)N1
z&jru)%Y22s!_)CrId$nPpSSEZ^%J$DbkXThRX%yi3+<#ZJp@Gw3*?MV3MvJKw1frf
pBlVH`Sj+&aKq`<5q=IJP-%q8CgOLB?u=&}m*|7SD<GH@6)o)L?*O~wT

literal 0
HcmV?d00001


From 3ab32a5d35ab4768f4ddb71e862de6b9d9b7f182 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= <dach.bartlomiej@gmail.com>
Date: Sat, 29 Aug 2020 23:29:29 +0200
Subject: [PATCH 9/9] Change structure of old style spinner to be closer to
 stable

---
 .../Skinning/LegacyOldStyleSpinner.cs         | 85 +++++++++++--------
 1 file changed, 51 insertions(+), 34 deletions(-)

diff --git a/osu.Game.Rulesets.Osu/Skinning/LegacyOldStyleSpinner.cs b/osu.Game.Rulesets.Osu/Skinning/LegacyOldStyleSpinner.cs
index 81a0df5ea5..e157842fd1 100644
--- a/osu.Game.Rulesets.Osu/Skinning/LegacyOldStyleSpinner.cs
+++ b/osu.Game.Rulesets.Osu/Skinning/LegacyOldStyleSpinner.cs
@@ -22,11 +22,11 @@ namespace osu.Game.Rulesets.Osu.Skinning
     {
         private DrawableSpinner drawableSpinner;
         private Sprite disc;
+        private Sprite metreSprite;
         private Container metre;
 
-        private const float background_y_offset = 20;
-
         private const float sprite_scale = 1 / 1.6f;
+        private const float final_metre_height = 692 * sprite_scale;
 
         [BackgroundDependencyLoader]
         private void load(ISkinSource source, DrawableHitObject drawableObject)
@@ -35,50 +35,58 @@ namespace osu.Game.Rulesets.Osu.Skinning
 
             RelativeSizeAxes = Axes.Both;
 
-            InternalChildren = new Drawable[]
+            InternalChild = new Container
             {
-                new Sprite
+                // the old-style spinner relied heavily on absolute screen-space coordinate values.
+                // wrap everything in a container simulating absolute coords to preserve alignment
+                // as there are skins that depend on it.
+                Width = 640,
+                Height = 480,
+                Anchor = Anchor.Centre,
+                Origin = Anchor.Centre,
+                Children = new Drawable[]
                 {
-                    Anchor = Anchor.BottomCentre,
-                    Origin = Anchor.BottomCentre,
-                    Texture = source.GetTexture("spinner-background"),
-                    Y = background_y_offset,
-                    Scale = new Vector2(sprite_scale)
-                },
-                disc = new Sprite
-                {
-                    Anchor = Anchor.Centre,
-                    Origin = Anchor.Centre,
-                    Texture = source.GetTexture("spinner-circle"),
-                    Scale = new Vector2(sprite_scale)
-                },
-                metre = new Container
-                {
-                    Anchor = Anchor.BottomCentre,
-                    Origin = Anchor.BottomCentre,
-                    Y = background_y_offset,
-                    Masking = true,
-                    Child = new Sprite
+                    new Sprite
                     {
-                        Texture = source.GetTexture("spinner-metre"),
-                        Anchor = Anchor.BottomCentre,
-                        Origin = Anchor.BottomCentre,
+                        Anchor = Anchor.Centre,
+                        Origin = Anchor.Centre,
+                        Texture = source.GetTexture("spinner-background"),
+                        Scale = new Vector2(sprite_scale)
                     },
-                    Scale = new Vector2(0.625f)
+                    disc = new Sprite
+                    {
+                        Anchor = Anchor.Centre,
+                        Origin = Anchor.Centre,
+                        Texture = source.GetTexture("spinner-circle"),
+                        Scale = new Vector2(sprite_scale)
+                    },
+                    metre = new Container
+                    {
+                        AutoSizeAxes = Axes.Both,
+                        // this anchor makes no sense, but that's what stable uses.
+                        Anchor = Anchor.TopLeft,
+                        Origin = Anchor.TopLeft,
+                        // adjustment for stable (metre has additional offset)
+                        Margin = new MarginPadding { Top = 20 },
+                        Masking = true,
+                        Child = metreSprite = new Sprite
+                        {
+                            Texture = source.GetTexture("spinner-metre"),
+                            Anchor = Anchor.TopLeft,
+                            Origin = Anchor.TopLeft,
+                            Scale = new Vector2(0.625f)
+                        }
+                    }
                 }
             };
         }
 
-        private Vector2 metreFinalSize;
-
         protected override void LoadComplete()
         {
             base.LoadComplete();
 
             this.FadeOut();
             drawableSpinner.State.BindValueChanged(updateStateTransforms, true);
-
-            metreFinalSize = metre.Size = metre.Child.Size;
         }
 
         private void updateStateTransforms(ValueChangedEvent<ArmedState> state)
@@ -93,7 +101,16 @@ namespace osu.Game.Rulesets.Osu.Skinning
         {
             base.Update();
             disc.Rotation = drawableSpinner.RotationTracker.Rotation;
-            metre.Height = getMetreHeight(drawableSpinner.Progress);
+
+            // careful: need to call this exactly once for all calculations in a frame
+            // as the function has a random factor in it
+            var metreHeight = getMetreHeight(drawableSpinner.Progress);
+
+            // hack to make the metre blink up from below than down from above.
+            // move down the container to be able to apply masking for the metre,
+            // and then move the sprite back up the same amount to keep its position absolute.
+            metre.Y = final_metre_height - metreHeight;
+            metreSprite.Y = -metre.Y;
         }
 
         private const int total_bars = 10;
@@ -108,7 +125,7 @@ namespace osu.Game.Rulesets.Osu.Skinning
             if (RNG.NextBool(((int)progress % 10) / 10f))
                 barCount++;
 
-            return (float)barCount / total_bars * metreFinalSize.Y;
+            return (float)barCount / total_bars * final_metre_height;
         }
     }
 }