mirror of https://github.com/ppy/osu
Set `ClampExtension` to zero to stop jittering
This commit is contained in:
parent
8edb1cb98a
commit
c8643ed265
|
@ -42,6 +42,7 @@ protected GameplayLeaderboard()
|
|||
{
|
||||
scroll = new InputDisabledScrollContainer
|
||||
{
|
||||
ClampExtension = 0,
|
||||
RelativeSizeAxes = Axes.Both,
|
||||
Child = Flow = new FillFlowContainer<GameplayLeaderboardScore>
|
||||
{
|
||||
|
@ -92,14 +93,6 @@ public ILeaderboardScore Add(IUser? user, bool isTracked)
|
|||
|
||||
int displayCount = Math.Min(Flow.Count, max_panels);
|
||||
Height = displayCount * (GameplayLeaderboardScore.PANEL_HEIGHT + Flow.Spacing.Y);
|
||||
// Add extra margin space to flow equal to height of leaderboard.
|
||||
// This ensures the content is always on screen, but also accounts for the fact that scroll operations
|
||||
// without animation were actually forcing the local score to a location it can't usually reside at.
|
||||
//
|
||||
// Basically, the local score was in the scroll extension region (due to always trying to scroll the
|
||||
// local player to the middle of the display, but there being no other content below the local player
|
||||
// to scroll up by).
|
||||
Flow.Margin = new MarginPadding { Bottom = Height };
|
||||
requiresScroll = displayCount != Flow.Count;
|
||||
|
||||
return drawable;
|
||||
|
|
Loading…
Reference in New Issue