Move implementation of drag handle operations to concrete classes

osu.Game/Screens/Edit/Compose/Components/SelectionBox.cs

@@ -15,6 +15,7 @@ using osuTK.Input;
 
 namespace osu.Game.Screens.Edit.Compose.Components
 {
+    [Cached]
     public class SelectionBox : CompositeDrawable
     {
         public const float BORDER_RADIUS = 3;
@@ -306,7 +307,7 @@ namespace osu.Game.Screens.Edit.Compose.Components
             var handle = new SelectionBoxScaleHandle
             {
                 Anchor = anchor,
-                HandleDrag = e => OnScale?.Invoke(e.Delta, anchor)
+                HandleScale = (delta, a) => OnScale?.Invoke(delta, a)
             };
 
             handle.OperationStarted += operationStarted;
@@ -319,7 +320,7 @@ namespace osu.Game.Screens.Edit.Compose.Components
             var handle = new SelectionBoxRotationHandle
             {
                 Anchor = anchor,
-                HandleDrag = e => OnRotation?.Invoke(convertDragEventToAngleOfRotation(e))
+                HandleRotate = angle => OnRotation?.Invoke(angle)
             };
 
             handle.OperationStarted += operationStarted;

osu.Game/Screens/Edit/Compose/Components/SelectionBoxDragHandle.cs

@@ -8,20 +8,12 @@ namespace osu.Game.Screens.Edit.Compose.Components
 {
     public abstract class SelectionBoxDragHandle : SelectionBoxControl
     {
-        public Action<DragEvent> HandleDrag { get; set; }
-
         protected override bool OnDragStart(DragStartEvent e)
         {
             TriggerOperationStarted();
             return true;
         }
 
-        protected override void OnDrag(DragEvent e)
-        {
-            HandleDrag?.Invoke(e);
-            base.OnDrag(e);
-        }
-
         protected override void OnDragEnd(DragEndEvent e)
         {
             TriggerOperationEnded();

osu.Game/Screens/Edit/Compose/Components/SelectionBoxRotationHandle.cs

@@ -1,10 +1,12 @@
 // 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.
 
+using System;
 using osu.Framework.Allocation;
 using osu.Framework.Extensions.EnumExtensions;
 using osu.Framework.Graphics;
 using osu.Framework.Graphics.Sprites;
+using osu.Framework.Input.Events;
 using osuTK;
 using osuTK.Graphics;
 
@@ -12,8 +14,13 @@ namespace osu.Game.Screens.Edit.Compose.Components
 {
     public class SelectionBoxRotationHandle : SelectionBoxDragHandle
     {
+        public Action<float> HandleRotate { get; set; }
+
         private SpriteIcon icon;
 
+        [Resolved]
+        private SelectionBox selectionBox { get; set; }
+
         [BackgroundDependencyLoader]
         private void load()
         {
@@ -38,5 +45,20 @@ namespace osu.Game.Screens.Edit.Compose.Components
             base.UpdateHoverState();
             icon.FadeColour(!IsHeld && IsHovered ? Color4.White : Color4.Black, TRANSFORM_DURATION, Easing.OutQuint);
         }
+
+        protected override void OnDrag(DragEvent e)
+        {
+            base.OnDrag(e);
+            HandleRotate?.Invoke(convertDragEventToAngleOfRotation(e));
+        }
+
+        private float convertDragEventToAngleOfRotation(DragEvent e)
+        {
+            // Adjust coordinate system to the center of SelectionBox
+            float startAngle = MathF.Atan2(e.LastMousePosition.Y - selectionBox.DrawHeight / 2, e.LastMousePosition.X - selectionBox.DrawWidth / 2);
+            float endAngle = MathF.Atan2(e.MousePosition.Y - selectionBox.DrawHeight / 2, e.MousePosition.X - selectionBox.DrawWidth / 2);
+
+            return (endAngle - startAngle) * 180 / MathF.PI;
+        }
     }
 }

osu.Game/Screens/Edit/Compose/Components/SelectionBoxScaleHandle.cs

@@ -1,17 +1,28 @@
 // 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.
 
+using System;
 using osu.Framework.Allocation;
+using osu.Framework.Graphics;
+using osu.Framework.Input.Events;
 using osuTK;
 
 namespace osu.Game.Screens.Edit.Compose.Components
 {
     public class SelectionBoxScaleHandle : SelectionBoxDragHandle
     {
+        public Action<Vector2, Anchor> HandleScale { get; set; }
+
         [BackgroundDependencyLoader]
         private void load()
         {
             Size = new Vector2(10);
         }
+
+        protected override void OnDrag(DragEvent e)
+        {
+            HandleScale?.Invoke(e.Delta, Anchor);
+            base.OnDrag(e);
+        }
     }
 }