Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce semi-opaque layers at song select #30580

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

peppy
Copy link
Member

@peppy peppy commented Nov 11, 2024

I made these changes while working on #30579. Basically, it's hard to fix the ranks not loading while underneath the footer, and the transparency both looks bad, and is going away in the redesign.

I've chosen values here that are moving in the direction of the new design without overhauling everything. I figure this will if anything lead to less unexpected change when we finally drop the new design.

  • I know that there's still some transparency. I did this because it helps keep all current elements / colours contrasting without too much effort.
  • I completely removed the transparency adjustments on the beatmap panels. This always looked bad due to being applied per-layer, and I don't think it added much.
Before After
osu! 2024-11-11 at 08 58 57 osu! 2024-11-11 at 08 58 04

I made these changes while working on
ppy#30579. Basically, it's hard to fix the
ranks not loading while underneath the footer, and the transparency both
looks bad, and is going away in the redesign.

I've chosen values here that are moving *in the direction* of the new
design without overhauling everything.

- I know that there's still some transparency. I did this because it
helps keep all current elements / colours contrasting without too much
effort.
- I completely removed the transparency adjustments on the beatmap
panels. This always looked bad due to being applied per-layer, and I
don't think it added much.
Copy link
Collaborator

@bdach bdach left a comment

Choose a reason for hiding this comment

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

inb4 "claustrophobia" comments

@peppy
Copy link
Member Author

peppy commented Nov 11, 2024

inb4 "claustrophobia" comments

potentially a good test run since the new design will be roughly the same.

@bdach bdach merged commit bca42e9 into ppy:master Nov 11, 2024
10 checks passed
@peppy peppy deleted the song-select-less-semi-opaque branch November 15, 2024 03:19
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.

2 participants