-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Fix player settings overlay cog overlapping skin elements #31106
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
Fix player settings overlay cog overlapping skin elements #31106
Conversation
This brings it down to be in line with the flowing elements that usually do their best to not get in the way. Decided against putting it in the `HUDOverlay` flow for simplicity. It will work fine until it doesn't.
I don't know why this is doing its own thing and not just a skinnable element. I struggle to imagine this is going to last given that there are now three places doing very similar things, the others being all this silliness: osu/osu.Game/Skinning/LegacySkin.cs Lines 383 to 410 in c29c3d4
osu/osu.Game/Screens/Play/HUDOverlay.cs Lines 257 to 290 in e2af882
|
This brings it down to be in line with the flowing elements that usually do their best to not get in the way.
Decided against putting it in the
HUDOverlay
flow for simplicity. It will work fine until it doesn't.osu.2024-12-13.at.08.29.32.mp4