h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-02-13 16:02:58 +08:00
Code Issues Releases Wiki Activity

Remove nullability and transition support from star rating display

Browse Source
This commit is contained in:
Salman Ahmed 2021-05-14 16:01:25 +03:00
parent 34aab11ff3
commit 6cc678f497
2 changed files with 17 additions and 39 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
osu.Game.Tests/Visual/Ranking/TestSceneStarRatingDisplay.cs
View File

@ -33,33 +33,28 @@ namespace osu.Game.Tests.Visual.Ranking
}); });
} }
[Test]
public void TestNullStarRatingDisplay()
{
AddStep("load null", () => Child = new StarRatingDisplay
{
Anchor = Anchor.Centre,
Origin = Anchor.Centre,
Scale = new Vector2(3f),
});
}
[Test] [Test]
public void TestChangingStarRatingDisplay() public void TestChangingStarRatingDisplay()
{ {
StarRatingDisplay starRating = null; StarRatingDisplay starRating = null;
AddStep("load display", () => Child = starRating = new StarRatingDisplay AddStep("load display", () => Child = starRating = new StarRatingDisplay(new StarDifficulty(5.55, 1))
{ {
Anchor = Anchor.Centre, Anchor = Anchor.Centre,
Origin = Anchor.Centre, Origin = Anchor.Centre,
Scale = new Vector2(3f), Scale = new Vector2(3f),
}); });
AddRepeatStep("change display value", () => AddRepeatStep("set random value", () =>
{ {
starRating.Current.Value = new StarDifficulty(RNG.NextDouble(0.0, 11.0), RNG.Next(2000)); starRating.Current.Value = new StarDifficulty(RNG.NextDouble(0.0, 11.0), 1);
}, 10); }, 10);
AddSliderStep("set exact stars", 0.0, 11.0, 5.55, d =>
{
if (starRating != null)
starRating.Current.Value = new StarDifficulty(d, 1);
});
} }
} }
} }

33
osu.Game/Screens/Ranking/Expanded/StarRatingDisplay.cs
View File

@ -22,7 +22,7 @@ namespace osu.Game.Screens.Ranking.Expanded
/// <summary> /// <summary>
/// A pill that displays the star rating of a <see cref="BeatmapInfo"/>. /// A pill that displays the star rating of a <see cref="BeatmapInfo"/>.
/// </summary> /// </summary>
public class StarRatingDisplay : CompositeDrawable, IHasCurrentValue<StarDifficulty?> public class StarRatingDisplay : CompositeDrawable, IHasCurrentValue<StarDifficulty>
{ {
private Box background; private Box background;
private OsuTextFlowContainer textFlow; private OsuTextFlowContainer textFlow;
@ -30,21 +30,14 @@ namespace osu.Game.Screens.Ranking.Expanded
[Resolved] [Resolved]
private OsuColour colours { get; set; } private OsuColour colours { get; set; }
private readonly BindableWithCurrent<StarDifficulty?> current = new BindableWithCurrent<StarDifficulty?>(); private readonly BindableWithCurrent<StarDifficulty> current = new BindableWithCurrent<StarDifficulty>();
public Bindable<StarDifficulty?> Current public Bindable<StarDifficulty> Current
{ {
get => current.Current; get => current.Current;
set => current.Current = value; set => current.Current = value;
} }
/// <summary>
/// Creates a new <see cref="StarRatingDisplay"/> without any <see cref="StarDifficulty"/> set, displaying a placeholder until <see cref="Current"/> is changed.
/// </summary>
public StarRatingDisplay()
{
}
/// <summary> /// <summary>
/// Creates a new <see cref="StarRatingDisplay"/> using an already computed <see cref="StarDifficulty"/>. /// Creates a new <see cref="StarRatingDisplay"/> using an already computed <see cref="StarDifficulty"/>.
/// </summary> /// </summary>
@ -112,26 +105,16 @@ namespace osu.Game.Screens.Ranking.Expanded
private void updateDisplay() private void updateDisplay()
{ {
const double duration = 400; var starRatingParts = Current.Value.Stars.ToString("0.00", CultureInfo.InvariantCulture).Split('.');
const Easing easing = Easing.OutQuint;
double stars = Current.Value?.Stars ?? 0.00f;
var starRatingParts = stars.ToString("0.00", CultureInfo.InvariantCulture).Split('.');
string wholePart = starRatingParts[0]; string wholePart = starRatingParts[0];
string fractionPart = starRatingParts[1]; string fractionPart = starRatingParts[1];
string separator = CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator; string separator = CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator;
if (Current.Value == null) var rating = Current.Value.DifficultyRating;
background.FadeColour(Color4.SlateGray.Opacity(0.3f));
else
{
var rating = Current.Value.Value.DifficultyRating;
background.FadeColour(rating == DifficultyRating.ExpertPlus background.Colour = rating == DifficultyRating.ExpertPlus
? ColourInfo.GradientVertical(Color4Extensions.FromHex("#C1C1C1"), Color4Extensions.FromHex("#595959")) ? ColourInfo.GradientVertical(Color4Extensions.FromHex("#C1C1C1"), Color4Extensions.FromHex("#595959"))
: (ColourInfo)colours.ForDifficultyRating(rating), duration, easing); : (ColourInfo)colours.ForDifficultyRating(rating);
}
textFlow.Clear(); textFlow.Clear();