1
0
mirror of https://github.com/ppy/osu.git synced 2024-09-22 04:47:24 +08:00

Fix using right mouse button to drag at song select seeking incorrectly with many beatmaps loaded

Closes #5195
This commit is contained in:
Dean Herbert 2019-07-30 07:03:59 +09:00
parent 5fa820cdd4
commit cec26a270e

View File

@ -27,11 +27,12 @@ namespace osu.Game.Graphics.Containers
private bool shouldPerformRightMouseScroll(MouseButtonEvent e) => RightMouseScrollbar && e.Button == MouseButton.Right; private bool shouldPerformRightMouseScroll(MouseButtonEvent e) => RightMouseScrollbar && e.Button == MouseButton.Right;
private void scrollToRelative(float value) => ScrollTo(Clamp((value - Scrollbar.DrawSize[ScrollDim] / 2) / Scrollbar.Size[ScrollDim]), true, DistanceDecayOnRightMouseScrollbar); private void scrollFromMouseEvent(MouseEvent e) =>
ScrollTo(Clamp(ToLocalSpace(e.ScreenSpaceMousePosition)[ScrollDim] / DrawSize[ScrollDim]) * Content.DrawSize[ScrollDim], true, DistanceDecayOnRightMouseScrollbar);
private bool mouseScrollBarDragging; private bool rightMouseDragging;
protected override bool IsDragging => base.IsDragging || mouseScrollBarDragging; protected override bool IsDragging => base.IsDragging || rightMouseDragging;
public OsuScrollContainer(Direction scrollDirection = Direction.Vertical) public OsuScrollContainer(Direction scrollDirection = Direction.Vertical)
: base(scrollDirection) : base(scrollDirection)
@ -42,7 +43,7 @@ namespace osu.Game.Graphics.Containers
{ {
if (shouldPerformRightMouseScroll(e)) if (shouldPerformRightMouseScroll(e))
{ {
scrollToRelative(e.MousePosition[ScrollDim]); scrollFromMouseEvent(e);
return true; return true;
} }
@ -51,9 +52,9 @@ namespace osu.Game.Graphics.Containers
protected override bool OnDrag(DragEvent e) protected override bool OnDrag(DragEvent e)
{ {
if (mouseScrollBarDragging) if (rightMouseDragging)
{ {
scrollToRelative(e.MousePosition[ScrollDim]); scrollFromMouseEvent(e);
return true; return true;
} }
@ -64,7 +65,7 @@ namespace osu.Game.Graphics.Containers
{ {
if (shouldPerformRightMouseScroll(e)) if (shouldPerformRightMouseScroll(e))
{ {
mouseScrollBarDragging = true; rightMouseDragging = true;
return true; return true;
} }
@ -73,9 +74,9 @@ namespace osu.Game.Graphics.Containers
protected override bool OnDragEnd(DragEndEvent e) protected override bool OnDragEnd(DragEndEvent e)
{ {
if (mouseScrollBarDragging) if (rightMouseDragging)
{ {
mouseScrollBarDragging = false; rightMouseDragging = false;
return true; return true;
} }