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

Remove macOS "borderless" recommendation #30636

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

peppy
Copy link
Member

@peppy peppy commented Nov 15, 2024

As of SDL3, this is no longer a thing, and fullscreen should be the preferred execution mode.

Probably hold off merging this until we're sure that macOS isn't broken for others in this mode (I had issues locally, such as alt-tabbing being broken sooo...)

As of SDL3, this is no longer a thing, and fullscreen should be the
preferred execution mode.

Probably hold off merging this until we're sure that macOS isn't broken
for others in this mode (I had issues locally, such as alt-tabbing being
broken sooo...)
@@ -269,14 +269,7 @@ private void updateDisplaySettingsVisibility()
private void updateScreenModeWarning()
{
if (RuntimeInfo.OS == RuntimeInfo.Platform.macOS)
{
if (windowModeDropdown.Current.Value == WindowMode.Fullscreen)
Copy link
Member

Choose a reason for hiding this comment

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

Can probably just condition this on FrameworkEnvironment.UseSDL3 instead of outright removing the code. Even if SDL3 is/will be the default, OSU_SDL3=0 is still an option.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, applied.

@koopafou
Copy link

I'm reading the SDL3 doc, you still have the choice to use exclusive mode (must be explicit) or borderless (default), it's not even marked as deprecated. For some reason the borderless option disappeared on Linux in the last update, and the remaining fullscreen option is using exclusive mode rather than being a secret borderless window to silently get rid of people's issues with alt-tabbing, you messed up something here.

@bdach
Copy link
Collaborator

bdach commented Nov 15, 2024

probably gonna link these in here before the above point of discussion gets too heated (started off pretty heated already what with the "you messed up something here")

@koopafou
Copy link

Sorry I wasn't trying to get the discussion heated, I was pointing out that maybe the issue came from the selection of the fullscreen mode rather than trying to handle user-facing issues that aren't even an osu or SDL problem. Fixing exclusive mode alt-tab is a lost cause in modern days, it's not even a macOS thing, it also occurs on Windows and Linux in any video game because the hardware is designed to force it that way. So yeah while you could talk about the performance improvements and be technically true, in practice window managers should be fast enough that the performance hit of compositing is probably placebo. But even if you prove it's placebo, some players will still want legacy fullscreen mode, and you can't remove the borderless fullscreen mode that just works. So might as well keep both and tell people to go borderless if they have issues.

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.

4 participants