sim1222/osukey
1
1
Fork 0
mirror of https://github.com/osukey/osukey.git synced 2025-08-07 16:43:52 +09:00
Code Issues Packages Projects Releases Wiki Activity

Fix remaining feedback loops

Browse Source
This commit is contained in:
Dean Herbert
2021-03-16 18:14:29 +09:00
parent e3bed4c97d
commit 932745e5c4
1 changed files with 33 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
osu.Game/Overlays/Settings/Sections/Input/TabletSettings.cs
View File

@ -1,7 +1,6 @@
// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence. // 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. // See the LICENCE file in the repository root for full licence text.
using System.ComponentModel;
using System.Drawing; using System.Drawing;
using osu.Framework.Allocation; using osu.Framework.Allocation;
using osu.Framework.Bindables; using osu.Framework.Bindables;
@ -80,7 +79,8 @@ namespace osu.Game.Overlays.Settings.Sections.Input
aspectRatio.BindValueChanged(aspect => aspectRatio.BindValueChanged(aspect =>
{ {
forceAspectRatio(aspect.NewValue); aspectRatioApplication?.Cancel();
aspectRatioApplication = Schedule(() => forceAspectRatio(aspect.NewValue));
}); });
((IBindable<Size>)tabletSize).BindTo(tabletHandler.TabletSize); ((IBindable<Size>)tabletSize).BindTo(tabletHandler.TabletSize);
@ -102,18 +102,23 @@ namespace osu.Game.Overlays.Settings.Sections.Input
}, true); }, true);
} }
private float curentAspectRatio => (float)sizeX.Value / sizeY.Value;
private void applyAspectRatio(ValueChangedEvent<Size> sizeChanged) private void applyAspectRatio(ValueChangedEvent<Size> sizeChanged)
{ {
float proposedAspectRatio = (float)sizeX.Value / sizeY.Value; float proposedAspectRatio = curentAspectRatio;
try
{
if (!aspectLock.Value) if (!aspectLock.Value)
{ {
aspectRatio.Value = proposedAspectRatio;
// aspect ratio was in a valid range. // aspect ratio was in a valid range.
if (proposedAspectRatio >= aspectRatio.MinValue && proposedAspectRatio <= aspectRatio.MaxValue) if (proposedAspectRatio >= aspectRatio.MinValue && proposedAspectRatio <= aspectRatio.MaxValue)
{
updateAspectRatio();
return; return;
} }
}
if (sizeChanged.NewValue.Width != sizeChanged.OldValue.Width) if (sizeChanged.NewValue.Width != sizeChanged.OldValue.Width)
{ {
@ -123,11 +128,19 @@ namespace osu.Game.Overlays.Settings.Sections.Input
{ {
areaSize.Value = new Size((int)(areaSize.Value.Height * aspectRatio.Value), areaSize.Value.Height); areaSize.Value = new Size((int)(areaSize.Value.Height * aspectRatio.Value), areaSize.Value.Height);
} }
}
finally
{
// cancel any event which may have fired while updating variables as a result of aspect ratio limitations. // cancel any event which may have fired while updating variables as a result of aspect ratio limitations.
// this avoids a potential feedback loop. // this avoids a potential feedback loop.
aspectRatioApplication?.Cancel(); aspectRatioApplication?.Cancel();
} }
}
private void updateAspectRatio()
{
aspectRatio.Value = curentAspectRatio;
}
private void updateDisplay() private void updateDisplay()
{ {
@ -204,6 +217,8 @@ namespace osu.Game.Overlays.Settings.Sections.Input
else else
sizeX.Value = (int)(sizeY.Value * aspectRatio); sizeX.Value = (int)(sizeY.Value * aspectRatio);
updateAspectRatio();
aspectRatioApplication?.Cancel(); aspectRatioApplication?.Cancel();
aspectLock.Value = true; aspectLock.Value = true;
} }