From f4f732ca4381768d5f54508b90fe06683f2bad0c Mon Sep 17 00:00:00 2001 From: Unknown Date: Sun, 26 Nov 2017 11:19:42 +0530 Subject: [PATCH] Remove unnecessary null check and tweak transform a bit. --- .../Online/API/Requests/GetScoresRequest.cs | 31 +++++++++---------- .../Select/Leaderboards/Leaderboard.cs | 8 ++--- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git a/osu.Game/Online/API/Requests/GetScoresRequest.cs b/osu.Game/Online/API/Requests/GetScoresRequest.cs index 534a209609..b9db836898 100644 --- a/osu.Game/Online/API/Requests/GetScoresRequest.cs +++ b/osu.Game/Online/API/Requests/GetScoresRequest.cs @@ -70,27 +70,24 @@ namespace osu.Game.Online.API.Requests break; } - if (ruleset != null) + switch (ruleset?.Name) { - switch (ruleset.Name) - { - default: - case @"osu!": - req.AddParameter(@"mode", @"osu"); - break; + default: + case @"osu!": + req.AddParameter(@"mode", @"osu"); + break; - case @"osu!taiko": - req.AddParameter(@"mode", @"taiko"); - break; + case @"osu!taiko": + req.AddParameter(@"mode", @"taiko"); + break; - case @"osu!catch": - req.AddParameter(@"mode", @"catch"); - break; + case @"osu!catch": + req.AddParameter(@"mode", @"catch"); + break; - case @"osu!mania": - req.AddParameter(@"mode", @"mania"); - break; - } + case @"osu!mania": + req.AddParameter(@"mode", @"mania"); + break; } return req; diff --git a/osu.Game/Screens/Select/Leaderboards/Leaderboard.cs b/osu.Game/Screens/Select/Leaderboards/Leaderboard.cs index 3bea1d4bfd..ef6ba01393 100644 --- a/osu.Game/Screens/Select/Leaderboards/Leaderboard.cs +++ b/osu.Game/Screens/Select/Leaderboards/Leaderboard.cs @@ -232,6 +232,8 @@ namespace osu.Game.Screens.Select.Leaderboards if (api == null || Beatmap?.OnlineBeatmapID == null) return; + loading.Show(); + if (Scope == LeaderboardScope.Local) { // TODO: get local scores from wherever here. @@ -239,8 +241,6 @@ namespace osu.Game.Screens.Select.Leaderboards return; } - loading.Show(); - getScoresRequest = new GetScoresRequest(Beatmap, osuGame?.Ruleset.Value, Scope); getScoresRequest.Success += r => { @@ -323,13 +323,13 @@ namespace osu.Game.Screens.Select.Leaderboards protected override bool OnMouseDown(Framework.Input.InputState state, Framework.Input.MouseDownEventArgs args) { - icon.ScaleTo(0.8f, 200, Easing.InElastic); + icon.ScaleTo(0.8f, 800, Easing.InElastic); return base.OnMouseDown(state, args); } protected override bool OnMouseUp(Framework.Input.InputState state, Framework.Input.MouseUpEventArgs args) { - icon.ScaleTo(1.2f, 400, Easing.OutElastic).Then().ScaleTo(1f, 400, Easing.OutElastic); + icon.ScaleTo(1.2f, 800, Easing.OutElastic).Then().ScaleTo(1f, 800, Easing.OutElastic); return base.OnMouseUp(state, args); } }