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

Fullscreen setting is broken #37588

Closed
Tracked by #37734
Anutrix opened this issue Apr 4, 2020 · 11 comments · Fixed by #40074
Closed
Tracked by #37734

Fullscreen setting is broken #37588

Anutrix opened this issue Apr 4, 2020 · 11 comments · Fixed by #40074

Comments

@Anutrix
Copy link
Contributor

Anutrix commented Apr 4, 2020

Godot version:
master 321ce4d

OS/device including version:
Windows 10 Pro

Issue description:
Setting the fullscreen option just makes the window of the game borderless. The size remains same and doesn't cover the whole screen.
Fullscreen unchecked:
Screenshot (656)
Fullscreen checked:
Screenshot (657)

Steps to reproduce:

  1. Check fullscreen option in Project Settings.
  2. Run the project.

Minimal reproduction project:
Any

@Anutrix
Copy link
Contributor Author

Anutrix commented Apr 8, 2020

Before: Screenshot (658)
After Shift+F11:Screenshot (659)

The issue is that input still works and it works on the window behind it. Can wreak havoc. Should be high-risk issue.

@KoBeWi
Copy link
Member

KoBeWi commented Jul 18, 2020

For some reason this still exists in 9e34ba4 (although the PR did fix it, so it just broke again)

@KoBeWi KoBeWi reopened this Jul 18, 2020
MarcusElg pushed a commit to MarcusElg/godot that referenced this issue Oct 19, 2020
@Anutrix
Copy link
Contributor Author

Anutrix commented Nov 1, 2020

Anyone worked/working on this?
It can still be reproduced in latest master 41f6676.

@DimitriyPS
Copy link

DimitriyPS commented Dec 11, 2020

@Anutrix Do you have multiple screens? What screen resolution?

@Anutrix
Copy link
Contributor Author

Anutrix commented Apr 15, 2022

@Anutrix Do you have multiple screens? What screen resolution?

Sorry about the super long delay, but no. I only have one display.

Haven't checked if it's reproducible in recent builds.

@KoBeWi
Copy link
Member

KoBeWi commented Apr 17, 2022

I checked in f7ca732. fullscreen makes window borderless and borderless makes the window fullscreen. Looks like the flags are swapped for some reason.

@Calinou
Copy link
Member

Calinou commented Apr 17, 2022

I checked in f7ca732. fullscreen makes window borderless and borderless makes the window fullscreen. Looks like the flags are swapped for some reason.

Good catch 🙂

See also my suggestion in godotengine/godot-proposals#4413, which could resolve this issue at the same time.

@KoBeWi
Copy link
Member

KoBeWi commented May 4, 2022

I tried to find out what causes the issue, but no luck. It's probably something in Window initialization code as setting fullscreen/borderless after initialization works fine.

Also it's not that the flags are swapped. Fullscreen makes the window borderless and it's correct, but the bug is that you need both fullscreen and borderless enabled for the window to be fullscreen at start.

@TokisanGames
Copy link
Contributor

alpha12, two 1920x1080 screens, win10/64 gtx1060

  • 1024x600, resizable => 1024x600 resizable
  • 1024x600, borderless => 1024x600 borderless
  • 1024x600, full screen => 1920x1080 moveable window w/ title !!!
  • 1024x600, resizable, full screen => 1024x600 borderless window !!!
  • 1024x600, borderless, full screen => 1920x1080 full screen borderless window
  • 1024x600, borderless, resizable, full screen => 1920x1080 full screen borderless window

There are two permutations that are wrong. Maybe all that is needed is for fullscreen to imply borderless, as it did in gd3. That could be a one line fix in the class that reads the options.

@bruvzg
Copy link
Member

bruvzg commented Jul 24, 2022

On Windows, this should be already fixed by #63194 (merged after alpha 12).

@KoBeWi
Copy link
Member

KoBeWi commented Aug 11, 2022

Can confirm it's fixed.

Although I noticed that when you have both borderless and fullscreen enabled, it causes a weird 1px outline at the borders. But that's another issue I guess.

@KoBeWi KoBeWi closed this as completed Aug 11, 2022
Repository owner moved this from Todo to Done in 4.x Priority Issues Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants