-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Change menus to fade out with a slight delay so settings changes are visible #31779
Conversation
…visible Useful for cases like ppy#31778, where a change to one setting can affect another.
this.Delay(50) | ||
.FadeOut(300, Easing.OutQuint); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did / do you want this change to also apply to context menus? Asking because OsuContextMenu
overrides AnimateClose()
and does not call base:
osu/osu.Game/Graphics/UserInterface/OsuContextMenu.cs
Lines 60 to 68 in b60a834
protected override void AnimateClose() | |
{ | |
this.FadeOut(fade_duration, Easing.OutQuint); | |
if (wasOpened) | |
menuSamples.PlayCloseSample(); | |
wasOpened = false; | |
} |
so closes on context menus are still not delayed on this branch as it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have applied this, good catch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This somewhat resolves #12894, but I can still see the illusion of two checkmarks if I'm focusing hard enough. FWIW the OP video that had the 300ms fade out before improves that situation for me.
This also makes it possible to double-click a menu, but maybe not a problem?
Potentially a monitor ghosting issue. Not really much can be done here, it's already an immediate hide.
Doesn't really change that, just makes it more possible. I'll see what I can do. |
I tried to fix the clicking-during-fade but it's not easy (the click handling is in the item itself which has no knowledge of the menu's state). Probably best left for another time. I did end up doing a refactor to avoid the mind-boggling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
audibly the refactor commit seems correct.
i give it a ~20% chance that someone is going to complain about the delay but let's see if it actually happens
Useful for cases like #31778, where a change to one setting can affect another.
osu.2025-02-03.at.08.36.58.mp4