mirror of
https://github.com/ppy/osu
synced 2025-01-10 16:19:47 +00:00
Highlight perfect slider tick/end values in beatmap info leaderboards
This commit is contained in:
parent
87afa7317b
commit
1270abdf42
@ -23,6 +23,7 @@ using osuTK;
|
|||||||
using osuTK.Graphics;
|
using osuTK.Graphics;
|
||||||
using osu.Framework.Localisation;
|
using osu.Framework.Localisation;
|
||||||
using osu.Framework.Extensions.LocalisationExtensions;
|
using osu.Framework.Extensions.LocalisationExtensions;
|
||||||
|
using osu.Framework.Graphics.Cursor;
|
||||||
using osu.Game.Resources.Localisation.Web;
|
using osu.Game.Resources.Localisation.Web;
|
||||||
|
|
||||||
namespace osu.Game.Overlays.BeatmapSet.Scores
|
namespace osu.Game.Overlays.BeatmapSet.Scores
|
||||||
@ -38,8 +39,6 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
|
|
||||||
private readonly FillFlowContainer backgroundFlow;
|
private readonly FillFlowContainer backgroundFlow;
|
||||||
|
|
||||||
private Color4 highAccuracyColour;
|
|
||||||
|
|
||||||
public ScoreTable()
|
public ScoreTable()
|
||||||
{
|
{
|
||||||
RelativeSizeAxes = Axes.X;
|
RelativeSizeAxes = Axes.X;
|
||||||
@ -57,12 +56,6 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
[BackgroundDependencyLoader]
|
|
||||||
private void load(OsuColour colours)
|
|
||||||
{
|
|
||||||
highAccuracyColour = colours.GreenLight;
|
|
||||||
}
|
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// The statistics that appear in the table, in order of appearance.
|
/// The statistics that appear in the table, in order of appearance.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
@ -158,12 +151,10 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
Current = scoreManager.GetBindableTotalScoreString(score),
|
Current = scoreManager.GetBindableTotalScoreString(score),
|
||||||
Font = OsuFont.GetFont(size: text_size, weight: index == 0 ? FontWeight.Bold : FontWeight.Medium)
|
Font = OsuFont.GetFont(size: text_size, weight: index == 0 ? FontWeight.Bold : FontWeight.Medium)
|
||||||
},
|
},
|
||||||
new OsuSpriteText
|
new StatisticText(score.Accuracy, 1, showTooltip: false)
|
||||||
{
|
{
|
||||||
Margin = new MarginPadding { Right = horizontal_inset },
|
Margin = new MarginPadding { Right = horizontal_inset },
|
||||||
Text = score.DisplayAccuracy,
|
Text = score.DisplayAccuracy,
|
||||||
Font = OsuFont.GetFont(size: text_size),
|
|
||||||
Colour = score.Accuracy == 1 ? highAccuracyColour : Color4.White
|
|
||||||
},
|
},
|
||||||
new UpdateableFlag(score.User.CountryCode)
|
new UpdateableFlag(score.User.CountryCode)
|
||||||
{
|
{
|
||||||
@ -171,14 +162,9 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
ShowPlaceholderOnUnknown = false,
|
ShowPlaceholderOnUnknown = false,
|
||||||
},
|
},
|
||||||
username,
|
username,
|
||||||
new OsuSpriteText
|
|
||||||
{
|
|
||||||
Text = score.MaxCombo.ToLocalisableString(@"0\x"),
|
|
||||||
Font = OsuFont.GetFont(size: text_size),
|
|
||||||
#pragma warning disable 618
|
#pragma warning disable 618
|
||||||
Colour = score.MaxCombo == score.BeatmapInfo.MaxCombo ? highAccuracyColour : Color4.White
|
new StatisticText(score.MaxCombo, score.BeatmapInfo.MaxCombo, @"0\x"),
|
||||||
#pragma warning restore 618
|
#pragma warning restore 618
|
||||||
}
|
|
||||||
};
|
};
|
||||||
|
|
||||||
var availableStatistics = score.GetStatisticsForDisplay().ToDictionary(tuple => tuple.Result);
|
var availableStatistics = score.GetStatisticsForDisplay().ToDictionary(tuple => tuple.Result);
|
||||||
@ -188,23 +174,13 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
if (!availableStatistics.TryGetValue(result.result, out var stat))
|
if (!availableStatistics.TryGetValue(result.result, out var stat))
|
||||||
stat = new HitResultDisplayStatistic(result.result, 0, null, result.displayName);
|
stat = new HitResultDisplayStatistic(result.result, 0, null, result.displayName);
|
||||||
|
|
||||||
content.Add(new OsuSpriteText
|
content.Add(new StatisticText(stat.Count, stat.MaxCount, @"N0") { Colour = stat.Count == 0 ? Color4.Gray : Color4.White });
|
||||||
{
|
|
||||||
Text = stat.MaxCount == null ? stat.Count.ToLocalisableString(@"N0") : (LocalisableString)$"{stat.Count}/{stat.MaxCount}",
|
|
||||||
Font = OsuFont.GetFont(size: text_size),
|
|
||||||
Colour = stat.Count == 0 ? Color4.Gray : Color4.White
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (showPerformancePoints)
|
if (showPerformancePoints)
|
||||||
{
|
{
|
||||||
Debug.Assert(score.PP != null);
|
Debug.Assert(score.PP != null);
|
||||||
|
content.Add(new StatisticText(score.PP.Value, format: @"N0"));
|
||||||
content.Add(new OsuSpriteText
|
|
||||||
{
|
|
||||||
Text = score.PP.ToLocalisableString(@"N0"),
|
|
||||||
Font = OsuFont.GetFont(size: text_size)
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
content.Add(new ScoreboardTime(score.Date, text_size)
|
content.Add(new ScoreboardTime(score.Date, text_size)
|
||||||
@ -243,5 +219,31 @@ namespace osu.Game.Overlays.BeatmapSet.Scores
|
|||||||
Colour = colourProvider.Foreground1;
|
Colour = colourProvider.Foreground1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private class StatisticText : OsuSpriteText, IHasTooltip
|
||||||
|
{
|
||||||
|
private readonly double count;
|
||||||
|
private readonly double? maxCount;
|
||||||
|
private readonly bool showTooltip;
|
||||||
|
|
||||||
|
public LocalisableString TooltipText => maxCount == null || !showTooltip ? string.Empty : $"{count}/{maxCount}";
|
||||||
|
|
||||||
|
public StatisticText(double count, double? maxCount = null, string format = null, bool showTooltip = true)
|
||||||
|
{
|
||||||
|
this.count = count;
|
||||||
|
this.maxCount = maxCount;
|
||||||
|
this.showTooltip = showTooltip;
|
||||||
|
|
||||||
|
Text = count.ToLocalisableString(format);
|
||||||
|
Font = OsuFont.GetFont(size: text_size);
|
||||||
|
}
|
||||||
|
|
||||||
|
[BackgroundDependencyLoader]
|
||||||
|
private void load(OsuColour colours)
|
||||||
|
{
|
||||||
|
if (count == maxCount)
|
||||||
|
Colour = colours.GreenLight;
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user