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 mode cannot be toggled off #57787

Closed
Chaosus opened this issue Feb 8, 2022 · 5 comments · Fixed by #57790 or #57794
Closed

Fullscreen mode cannot be toggled off #57787

Chaosus opened this issue Feb 8, 2022 · 5 comments · Fixed by #57790 or #57794

Comments

@Chaosus
Copy link
Member

Chaosus commented Feb 8, 2022

Godot version

26facc0

System information

Windows 10

Issue description

After the editor screen entered to the fullscreen it cannot be turned back to windowed mode - it just flickers and become fullscreen again:

fullscreen_bug

This problem affects a debugged project as well.

Steps to reproduce

Toggle fullscreen in the editor on the current master (26facc0). It's a recent regression - it works correctly on Alpha 1.

Minimal reproduction project

No response

@Chaosus Chaosus added this to the 4.0 milestone Feb 8, 2022
@Chaosus Chaosus changed the title Fullscreen mode of the editor cannot be toggled off Fullscreen mode cannot be toggled off Feb 8, 2022
@Sauermann
Copy link
Contributor

Sauermann commented Feb 8, 2022

On Debian 11 + Xfce, the symptoms are the same.

@bruvzg
Copy link
Member

bruvzg commented Feb 8, 2022

Seems like regression from #54645, seems like this check is not always working:

} else if (window_client_rect.position == screen_position && window_client_rect.size == screen_size) {
window.fullscreen = true;
}

@bruvzg
Copy link
Member

bruvzg commented Feb 8, 2022

#57790 - fix for Windows.

On Debian 11 + Xfce, the symptoms are the same.

Should be a completely unrelated issue, was it working in alpha 1?

@Sauermann
Copy link
Contributor

Should be a completely unrelated issue, was it working in alpha 1?

You are right. It was not working on Linux in alpha 1. There is already a bug report for that: #55109

@bruvzg
Copy link
Member

bruvzg commented Feb 8, 2022

Fix for Linux/X11 - #57794, #55109 is not entirely the same issue, at least part about maximized windows behavior is probably unrelated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment