Allow LN tail to have an origin of TopCentre

osu.Game.Rulesets.Mania.Tests/Skinning/TestSceneHoldNote.cs

@@ -10,6 +10,8 @@
 using osu.Game.Beatmaps.ControlPoints;
 using osu.Game.Rulesets.Mania.Objects;
 using osu.Game.Rulesets.Mania.Objects.Drawables;
+using osu.Game.Rulesets.UI.Scrolling;
+using osu.Game.Skinning;
 
 namespace osu.Game.Rulesets.Mania.Tests.Skinning
 {
@@ -37,6 +39,25 @@ public void TestFadeOnMiss()
             });
         }
 
+        [Test]
+        public void TestTailOrigin()
+        {
+            AddStep("set tail origin to regular", () =>
+            {
+                foreach (var holdNote in CreatedDrawables.SelectMany(d => d.ChildrenOfType<DrawableHoldNote>()))
+                    holdNote.Tail.TailOrigin = HoldNoteTailOrigin.Regular;
+            });
+            AddStep("change direction to down", () => ScrollingInfo.Direction.Value = ScrollingDirection.Down);
+            AddStep("change direction to up", () => ScrollingInfo.Direction.Value = ScrollingDirection.Up);
+            AddStep("set tail origin to inverted", () =>
+            {
+                foreach (var holdNote in CreatedDrawables.SelectMany(d => d.ChildrenOfType<DrawableHoldNote>()))
+                    holdNote.Tail.TailOrigin = HoldNoteTailOrigin.Inverted;
+            });
+            AddStep("change direction to down", () => ScrollingInfo.Direction.Value = ScrollingDirection.Down);
+            AddStep("change direction to up", () => ScrollingInfo.Direction.Value = ScrollingDirection.Up);
+        }
+
         private IEnumerable<DrawableHoldNote> holdNotes => CreatedDrawables.SelectMany(d => d.ChildrenOfType<DrawableHoldNote>());
 
         protected override DrawableManiaHitObject CreateHitObject()

osu.Game.Rulesets.Mania/Objects/Drawables/DrawableHoldNote.cs

@@ -239,7 +239,12 @@ protected override void Update()
             // Position and resize the body to lie half-way under the head and the tail notes.
             // The rationale for this is account for heads/tails with corner radius.
             bodyPiece.Y = (Direction.Value == ScrollingDirection.Up ? 1 : -1) * Head.Height / 2;
-            bodyPiece.Height = DrawHeight - Head.Height / 2 + Tail.Height / 2;
+            bodyPiece.Height = DrawHeight - Head.Height / 2;
+
+            if (Tail.TailOrigin == HoldNoteTailOrigin.Inverted)
+                bodyPiece.Height -= Tail.Height / 2;
+            else
+                bodyPiece.Height += Tail.Height / 2;
 
             if (Time.Current >= HitObject.StartTime)
             {

osu.Game.Rulesets.Mania/Objects/Drawables/DrawableHoldNoteTail.cs

@@ -3,9 +3,13 @@
 
 #nullable disable
 
+using osu.Framework.Bindables;
 using osu.Framework.Graphics;
 using osu.Framework.Input.Events;
+using osu.Game.Rulesets.Mania.Skinning;
 using osu.Game.Rulesets.Scoring;
+using osu.Game.Rulesets.UI.Scrolling;
+using osu.Game.Skinning;
 
 namespace osu.Game.Rulesets.Mania.Objects.Drawables
 {
@@ -18,6 +22,18 @@ public partial class DrawableHoldNoteTail : DrawableNote
 
         protected internal DrawableHoldNote HoldNote => (DrawableHoldNote)ParentHitObject;
 
+        private HoldNoteTailOrigin tailOrigin = HoldNoteTailOrigin.Regular;
+
+        public HoldNoteTailOrigin TailOrigin
+        {
+            get => tailOrigin;
+            set
+            {
+                tailOrigin = value;
+                updateTailOrigin();
+            }
+        }
+
         public DrawableHoldNoteTail()
             : this(null)
         {
@@ -52,5 +68,25 @@ protected override HitResult GetCappedResult(HitResult result)
         public override void OnReleased(KeyBindingReleaseEvent<ManiaAction> e)
         {
         }
+
+        protected override void ApplySkin(ISkinSource skin, bool allowFallback)
+        {
+            base.ApplySkin(skin, allowFallback);
+            tailOrigin = skin.GetConfig<ManiaSkinConfigurationLookup, HoldNoteTailOrigin>(new ManiaSkinConfigurationLookup(LegacyManiaSkinConfigurationLookups.HoldNoteTailOrigin))?.Value ?? HoldNoteTailOrigin.Regular;
+        }
+
+        protected override void OnDirectionChanged(ValueChangedEvent<ScrollingDirection> e)
+        {
+            base.OnDirectionChanged(e);
+            updateTailOrigin();
+        }
+
+        private void updateTailOrigin()
+        {
+            if (Direction.Value == ScrollingDirection.Up)
+                Origin = tailOrigin == HoldNoteTailOrigin.Inverted ? Anchor.BottomCentre : Anchor.TopCentre;
+            else
+                Origin = tailOrigin == HoldNoteTailOrigin.Inverted ? Anchor.TopCentre : Anchor.BottomCentre;
+        }
     }
 }

osu.Game.Rulesets.Mania/Objects/Drawables/DrawableNote.cs

@@ -78,7 +78,6 @@ protected override void OnApply()
         protected override void OnDirectionChanged(ValueChangedEvent<ScrollingDirection> e)
         {
             base.OnDirectionChanged(e);
-
             headPiece.Anchor = headPiece.Origin = e.NewValue == ScrollingDirection.Up ? Anchor.TopCentre : Anchor.BottomCentre;
         }
 

osu.Game/Skinning/LegacyManiaSkinConfiguration.cs

@@ -45,6 +45,7 @@ public class LegacyManiaSkinConfiguration : IHasCustomColours
         public bool KeysUnderNotes;
         public int LightFramePerSecond = 60;
 
+        public HoldNoteTailOrigin HoldNoteTailOrigin;
         public LegacyNoteBodyStyle? NoteBodyStyle;
 
         public LegacyManiaSkinConfiguration(int keys)

osu.Game/Skinning/LegacyManiaSkinConfigurationLookup.cs

@@ -57,6 +57,7 @@ public enum LegacyManiaSkinConfigurationLookups
         HoldNoteBodyImage,
         HoldNoteLightImage,
         HoldNoteLightScale,
+        HoldNoteTailOrigin,
         WidthForNoteHeightScale,
         ExplosionImage,
         ExplosionScale,
@@ -82,4 +83,10 @@ public enum LegacyManiaSkinConfigurationLookups
         NoteBodyStyle,
         LightFramePerSecond
     }
+
+    public enum HoldNoteTailOrigin
+    {
+        Regular = 0,
+        Inverted = 1
+    }
 }

osu.Game/Skinning/LegacyManiaSkinDecoder.cs

@@ -132,6 +132,11 @@ private void flushPendingLines()
                         currentConfig.LightFramePerSecond = lightFramePerSecond > 0 ? lightFramePerSecond : 24;
                         break;
 
+                    case "HoldNoteTailOrigin":
+                        if (Enum.TryParse<HoldNoteTailOrigin>(pair.Value, out var tailOrigin))
+                            currentConfig.HoldNoteTailOrigin = tailOrigin;
+                        break;
+
                     case string when pair.Key.StartsWith("Colour", StringComparison.Ordinal):
                         HandleColours(currentConfig, line, true);
                         break;

osu.Game/Skinning/LegacySkin.cs

@@ -241,6 +241,9 @@ protected override void ParseConfigurationStream(Stream stream)
 
                     return SkinUtils.As<TValue>(new Bindable<float>(existing.ColumnWidth[maniaLookup.ColumnIndex.Value] / LegacyManiaSkinConfiguration.DEFAULT_COLUMN_SIZE));
 
+                case LegacyManiaSkinConfigurationLookups.HoldNoteTailOrigin:
+                    return SkinUtils.As<TValue>(new Bindable<HoldNoteTailOrigin>(existing.HoldNoteTailOrigin));
+
                 case LegacyManiaSkinConfigurationLookups.KeyImage:
                     Debug.Assert(maniaLookup.ColumnIndex != null);
                     return SkinUtils.As<TValue>(getManiaImage(existing, $"KeyImage{maniaLookup.ColumnIndex}"));