h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2024-11-11 17:07:38 +08:00
Code Issues Releases Wiki Activity

Change WarningText to accept LocalisableString

Browse Source 
Can't work just yet, but best to have the flow in place to maintain 100%
localisation on classes which were already localised.
This commit is contained in:
Dean Herbert 2021-07-26 16:59:27 +09:00
parent 488dd3ea52
commit af9f910a12
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
osu.Game/Overlays/Settings/SettingsItem.cs
View File

@ -61,12 +61,17 @@ namespace osu.Game.Overlays.Settings
/// Text to be displayed at the bottom of this <see cref="SettingsItem{T}"/>.
/// Generally used to recommend the user change their setting as the current one is considered sub-optimal.
/// </summary>
public string WarningText
public LocalisableString? WarningText
{
set
{
bool hasValue = string.IsNullOrWhiteSpace(value.ToString());
if (warningText == null)
{
if (!hasValue)
return;
// construct lazily for cases where the label is not needed (may be provided by the Control).
FlowContent.Add(warningText = new OsuTextFlowContainer
{
@ -77,8 +82,8 @@ namespace osu.Game.Overlays.Settings
});
}
warningText.Alpha = string.IsNullOrWhiteSpace(value) ? 0 : 1;
warningText.Text = value;
warningText.Alpha = hasValue ? 0 : 1;
warningText.Text = value.ToString(); // TODO: Add localisation support after https://github.com/ppy/osu-framework/pull/4603 is merged.
}
}