From 45a6a743a226becf932488075fd91617f4a7f440 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= <dach.bartlomiej@gmail.com>
Date: Fri, 4 Oct 2024 13:12:25 +0200
Subject: [PATCH] Fix improper handling of decimal separator in "form" number
 boxes / sliders

Spotted in passing in
https://discord.com/channels/188630481301012481/1097318920991559880/1291693852981329981.
---
 osu.Game/Graphics/UserInterfaceV2/FormNumberBox.cs |  4 +++-
 osu.Game/Graphics/UserInterfaceV2/FormSliderBar.cs | 13 ++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/osu.Game/Graphics/UserInterfaceV2/FormNumberBox.cs b/osu.Game/Graphics/UserInterfaceV2/FormNumberBox.cs
index 8ce6c85fa986..c3256e0038c5 100644
--- a/osu.Game/Graphics/UserInterfaceV2/FormNumberBox.cs
+++ b/osu.Game/Graphics/UserInterfaceV2/FormNumberBox.cs
@@ -1,6 +1,8 @@
 // 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.Globalization;
+
 namespace osu.Game.Graphics.UserInterfaceV2
 {
     public partial class FormNumberBox : FormTextBox
@@ -18,7 +20,7 @@ internal partial class InnerNumberBox : InnerTextBox
             public bool AllowDecimals { get; init; }
 
             protected override bool CanAddCharacter(char character)
-                => char.IsAsciiDigit(character) || (AllowDecimals && character == '.');
+                => char.IsAsciiDigit(character) || (AllowDecimals && CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator.Contains(character));
         }
     }
 }
diff --git a/osu.Game/Graphics/UserInterfaceV2/FormSliderBar.cs b/osu.Game/Graphics/UserInterfaceV2/FormSliderBar.cs
index 78c197dee6f8..a29e33a42146 100644
--- a/osu.Game/Graphics/UserInterfaceV2/FormSliderBar.cs
+++ b/osu.Game/Graphics/UserInterfaceV2/FormSliderBar.cs
@@ -17,6 +17,7 @@
 using osu.Framework.Input.Events;
 using osu.Framework.Localisation;
 using osu.Game.Graphics.UserInterface;
+using osu.Game.Localisation;
 using osu.Game.Overlays;
 
 namespace osu.Game.Graphics.UserInterfaceV2
@@ -83,8 +84,10 @@ public CompositeDrawable? TabbableContentContainer
         [Resolved]
         private OverlayColourProvider colourProvider { get; set; } = null!;
 
+        private readonly Bindable<Language> currentLanguage = new Bindable<Language>();
+
         [BackgroundDependencyLoader]
-        private void load(OsuColour colours)
+        private void load(OsuColour colours, OsuGame? game)
         {
             RelativeSizeAxes = Axes.X;
             Height = 50;
@@ -150,6 +153,9 @@ private void load(OsuColour colours)
                     },
                 },
             };
+
+            if (game != null)
+                currentLanguage.BindTo(game.CurrentLanguage);
         }
 
         protected override void LoadComplete()
@@ -164,10 +170,11 @@ protected override void LoadComplete()
 
             slider.IsDragging.BindValueChanged(_ => updateState());
 
+            currentLanguage.BindValueChanged(_ => Schedule(updateValueDisplay));
             current.BindValueChanged(_ =>
             {
                 updateState();
-                updateTextBoxFromSlider();
+                updateValueDisplay();
             }, true);
         }
 
@@ -258,7 +265,7 @@ private void updateState()
                 background.Colour = colourProvider.Background5;
         }
 
-        private void updateTextBoxFromSlider()
+        private void updateValueDisplay()
         {
             if (updatingFromTextBox) return;