1
0
mirror of https://github.com/ppy/osu.git synced 2025-03-28 01:37:46 +08:00

Reorder StatisticsItem constructor to make a touch more sense

This commit is contained in:
Dean Herbert 2022-02-02 23:02:38 +09:00
parent 3c2a6fe208
commit 19eb9ad8a7
4 changed files with 34 additions and 41 deletions

View File

@ -370,23 +370,21 @@ namespace osu.Game.Rulesets.Mania
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem("Timing Distribution", true, new StatisticItem("Timing Distribution", () => new HitEventTimingDistributionGraph(score.HitEvents)
() => new HitEventTimingDistributionGraph(score.HitEvents) {
{ RelativeSizeAxes = Axes.X,
RelativeSizeAxes = Axes.X, Height = 250
Height = 250 }, true),
}),
} }
}, },
new StatisticRow new StatisticRow
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem(string.Empty, true, new StatisticItem(string.Empty, () => new SimpleStatisticTable(3, new SimpleStatisticItem[]
() => new SimpleStatisticTable(3, new SimpleStatisticItem[] {
{ new UnstableRate(score.HitEvents)
new UnstableRate(score.HitEvents) }), true)
}))
} }
} }
}; };

View File

@ -277,35 +277,32 @@ namespace osu.Game.Rulesets.Osu
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem("Timing Distribution", true, new StatisticItem("Timing Distribution", () => new HitEventTimingDistributionGraph(timedHitEvents)
() => new HitEventTimingDistributionGraph(timedHitEvents) {
{ RelativeSizeAxes = Axes.X,
RelativeSizeAxes = Axes.X, Height = 250
Height = 250 }, true),
}),
} }
}, },
new StatisticRow new StatisticRow
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem("Accuracy Heatmap", true, new StatisticItem("Accuracy Heatmap", () => new AccuracyHeatmap(score, playableBeatmap)
() => new AccuracyHeatmap(score, playableBeatmap) {
{ RelativeSizeAxes = Axes.X,
RelativeSizeAxes = Axes.X, Height = 250
Height = 250 }, true),
}),
} }
}, },
new StatisticRow new StatisticRow
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem(string.Empty, true, new StatisticItem(string.Empty, () => new SimpleStatisticTable(3, new SimpleStatisticItem[]
() => new SimpleStatisticTable(3, new SimpleStatisticItem[] {
{ new UnstableRate(timedHitEvents)
new UnstableRate(timedHitEvents) }), true)
}))
} }
} }
}; };

View File

@ -213,23 +213,21 @@ namespace osu.Game.Rulesets.Taiko
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem("Timing Distribution", true, new StatisticItem("Timing Distribution", () => new HitEventTimingDistributionGraph(timedHitEvents)
() => new HitEventTimingDistributionGraph(timedHitEvents) {
{ RelativeSizeAxes = Axes.X,
RelativeSizeAxes = Axes.X, Height = 250
Height = 250 }, true),
}),
} }
}, },
new StatisticRow new StatisticRow
{ {
Columns = new[] Columns = new[]
{ {
new StatisticItem(string.Empty, true, new StatisticItem(string.Empty, () => new SimpleStatisticTable(3, new SimpleStatisticItem[]
() => new SimpleStatisticTable(3, new SimpleStatisticItem[] {
{ new UnstableRate(timedHitEvents)
new UnstableRate(timedHitEvents) }), true)
}))
} }
} }
}; };

View File

@ -37,10 +37,10 @@ namespace osu.Game.Screens.Ranking.Statistics
/// Creates a new <see cref="StatisticItem"/>, to be displayed inside a <see cref="StatisticRow"/> in the results screen. /// Creates a new <see cref="StatisticItem"/>, to be displayed inside a <see cref="StatisticRow"/> in the results screen.
/// </summary> /// </summary>
/// <param name="name">The name of the item. Can be <see cref="string.Empty"/> to hide the item header.</param> /// <param name="name">The name of the item. Can be <see cref="string.Empty"/> to hide the item header.</param>
/// <param name="requiresHitEvents">Whether this item requires hit events. If true, <see cref="CreateContent"/> will not be called if no hit events are available.</param>
/// <param name="createContent">A function returning the <see cref="Drawable"/> content to be displayed.</param> /// <param name="createContent">A function returning the <see cref="Drawable"/> content to be displayed.</param>
/// <param name="requiresHitEvents">Whether this item requires hit events. If true, <see cref="CreateContent"/> will not be called if no hit events are available.</param>
/// <param name="dimension">The <see cref="Dimension"/> of this item. This can be thought of as the column dimension of an encompassing <see cref="GridContainer"/>.</param> /// <param name="dimension">The <see cref="Dimension"/> of this item. This can be thought of as the column dimension of an encompassing <see cref="GridContainer"/>.</param>
public StatisticItem([NotNull] string name, bool requiresHitEvents, [NotNull] Func<Drawable> createContent, [CanBeNull] Dimension dimension = null) public StatisticItem([NotNull] string name, [NotNull] Func<Drawable> createContent, bool requiresHitEvents = false, [CanBeNull] Dimension dimension = null)
{ {
Name = name; Name = name;
RequiresHitEvents = requiresHitEvents; RequiresHitEvents = requiresHitEvents;