Skip to content

Minor design improvements to the extended result screen#37144

Open
stanriders wants to merge 5 commits intoppy:masterfrom
stanriders:result-screen-minor-improvements
Open

Minor design improvements to the extended result screen#37144
stanriders wants to merge 5 commits intoppy:masterfrom
stanriders:result-screen-minor-improvements

Conversation

@stanriders
Copy link
Copy Markdown
Member

@stanriders stanriders commented Mar 29, 2026

I am aware of the possibly-one-day-definitely-upcoming redesign, but while it's not yet ready I've made a couple of small design improvements to the current result screen.

This includes:

  • Slightly tighter and more consistent paddings
  • StatisticPanel's header being consistent with other similar panels (it's using subsection design right now which probably made sense back when it was all one panel)
  • More compact PerformanceBreakdown section
  • Slightly less background blur and slightly more transparent statistics panels

Before:
image
image

After:
image
image

@peppy peppy self-requested a review March 30, 2026 14:48
@peppy
Copy link
Copy Markdown
Member

peppy commented Mar 30, 2026

I dunno if this is worth it when it still looks like shit, but I'll give it 10 minutes.

@peppy
Copy link
Copy Markdown
Member

peppy commented Mar 30, 2026

I don't like this display format.

2026-03-31 03 42 37@2x

I don't think PP should be a 1/2 fraction thing.

I'd rather a second gray line saying "Maximum 102pp" beneath like it was before.

Copy link
Copy Markdown
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't mind making padding changes and stuff but let's not invent new display methods from nowhere.

@stanriders
Copy link
Copy Markdown
Member Author

I don't like this display format.

I don't think PP should be a 1/2 fraction thing.

I'd rather a second gray line saying "Maximum 102pp" beneath like it was before.

This isn't really a completely new display method as we already have these things which show max value separated by a /
image

Would you prefer something like this or should I just revert to the original version?
image
image

@peppy
Copy link
Copy Markdown
Member

peppy commented Apr 2, 2026

This isn't really a completely new display method as we already have these things which show max value separated by a

I fully understand that, but it's new for PP and I don't like it. That's never how PP was envisioned (there shouldn't be a "MAX" to aim for).

Yes, the new version you propose is ok.

While you're there make the bars on the rigth not take up 1000pixels. Maybe limit to 50% of width or less.

@stanriders
Copy link
Copy Markdown
Member Author

image Okay this is the best I could come up with, it's so long and unbalanced it's kinda hard to make it look any good lol. Looks good on 4:3 though so that's something i guess image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants