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

removes SDL_WINDOW_BORDERLESS flag #11546

Merged
merged 1 commit into from Sep 11, 2023
Merged

Conversation

Davidian1024
Copy link
Contributor

Meant to address the issue reported here:
#7532 (comment) And possibly here:
#7922 (comment)

Meant to address the issue reported here:
mamedev#7532 (comment)
And possibly here:
mamedev#7922 (comment)
@cuavas
Copy link
Member

cuavas commented Sep 11, 2023

Does this have a detrimental effect on any platforms? What happens on Windows and macOS when this flag is not specified?

@Davidian1024
Copy link
Contributor Author

I honestly don't know. To be completely honest, I don't fully understand what a borderless window is.

All that said, these days I see borderless as optional in probably most games nowadays.

Currently MAME seems to force it's fullscreen window to always be of the borderless variety at least when SDL is being used.

@cuavas cuavas merged commit c9b6c43 into mamedev:master Sep 11, 2023
@cuavas
Copy link
Member

cuavas commented Sep 11, 2023

I tested this on Windows with two displays, and it seemed to work. I think you’re supposed to either specify full screen, or the combination of borderless and maximised, but not a combination.

A borderless window won’t have “decorations” like a title bar, window frame, shadow, etc. Depending on the OS and window manager, there may be a distinction between a borderless window that covers an entire display, and an application taking exclusive control of a display.

@Davidian1024
Copy link
Contributor Author

Oh ok cool. :)

@rb6502
Copy link
Contributor

rb6502 commented Sep 11, 2023

FWIW, non-borderless full screen is deprecated by both Microsoft and Apple and we no longer use it at work. I think this is probably an SDL bug and I greatly oppose this change.

@cuavas
Copy link
Member

cuavas commented Sep 12, 2023

FWIW, non-borderless full screen is deprecated by both Microsoft and Apple and we no longer use it at work. I think this is probably an SDL bug and I greatly oppose this change.

You're missing the point - SDL always creates a borderless window when you specify fullscreen if it isn't using exclusive mode. If you want it to always use borderless fullscreen (never use exclusive mode, never change resolution), you need to specify borderless and maximised. Specifying a combination of fullscreen and borderless confuses it.

@Davidian1024 Davidian1024 deleted the patch-1 branch September 24, 2023 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants