Fix pen events being handled twice

Wouldn't fail the test as it would be a nop state change therefore not propagated as events
Susko3 · Jan 13, 2025 · 3fc7237 · 3fc7237
1 parent 2792b5a
commit 3fc7237
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/osu.Framework/Input/PassThroughInputManager.cs b/osu.Framework/Input/PassThroughInputManager.cs
@@ -108,17 +108,17 @@ protected override bool Handle(UIEvent e)
                     case MouseDownEvent penDown:
                         Debug.Assert(penDown.Button == MouseButton.Left);
                         new MouseButtonInputFromPen(true) { DeviceType = penInput.DeviceType }.Apply(CurrentState, this);
-                        break;
+                        return false;
 
                     case MouseUpEvent penUp:
                         Debug.Assert(penUp.Button == MouseButton.Left);
                         new MouseButtonInputFromPen(false) { DeviceType = penInput.DeviceType }.Apply(CurrentState, this);
-                        break;
+                        return false;
 
                     case MouseMoveEvent penMove:
                         if (penMove.ScreenSpaceMousePosition != CurrentState.Mouse.Position)
                             new MousePositionAbsoluteInputFromPen { Position = penMove.ScreenSpaceMousePosition, DeviceType = penInput.DeviceType }.Apply(CurrentState, this);
-                        break;
+                        return false;
                 }
             }