From eecb1ce9f55084d19eea03378b9cf6c34cfe9706 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Thu, 16 Dec 2021 20:20:02 +0900
Subject: [PATCH] Avoid applying mouse down effects to menu cursor when it
 isn't visible

Closes #16114.
---
 osu.Game/Graphics/Cursor/MenuCursor.cs | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/osu.Game/Graphics/Cursor/MenuCursor.cs b/osu.Game/Graphics/Cursor/MenuCursor.cs
index 3fa90e2330..8e272f637f 100644
--- a/osu.Game/Graphics/Cursor/MenuCursor.cs
+++ b/osu.Game/Graphics/Cursor/MenuCursor.cs
@@ -72,18 +72,21 @@ namespace osu.Game.Graphics.Cursor
 
         protected override bool OnMouseDown(MouseDownEvent e)
         {
-            // only trigger animation for main mouse buttons
-            activeCursor.Scale = new Vector2(1);
-            activeCursor.ScaleTo(0.90f, 800, Easing.OutQuint);
-
-            activeCursor.AdditiveLayer.Alpha = 0;
-            activeCursor.AdditiveLayer.FadeInFromZero(800, Easing.OutQuint);
-
-            if (cursorRotate.Value && dragRotationState != DragRotationState.Rotating)
+            if (State.Value == Visibility.Visible)
             {
-                // if cursor is already rotating don't reset its rotate origin
-                dragRotationState = DragRotationState.DragStarted;
-                positionMouseDown = e.MousePosition;
+                // only trigger animation for main mouse buttons
+                activeCursor.Scale = new Vector2(1);
+                activeCursor.ScaleTo(0.90f, 800, Easing.OutQuint);
+
+                activeCursor.AdditiveLayer.Alpha = 0;
+                activeCursor.AdditiveLayer.FadeInFromZero(800, Easing.OutQuint);
+
+                if (cursorRotate.Value && dragRotationState != DragRotationState.Rotating)
+                {
+                    // if cursor is already rotating don't reset its rotate origin
+                    dragRotationState = DragRotationState.DragStarted;
+                    positionMouseDown = e.MousePosition;
+                }
             }
 
             return base.OnMouseDown(e);