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

[Windows] Improve non-exclusive full-screen mode. #72787

Closed
wants to merge 1 commit into from

Conversation

bruvzg
Copy link
Member

@bruvzg bruvzg commented Feb 6, 2023

Removes single pixel border in the non-exclusive full-screen mode.

Fixes #63500

Note: needs testing on a different GPUs to make sure it's not triggering exclusive full-screen, I have tested it on NVIDIA only. Testing on AMD and Intel is highly appreciated. The easiest way to test it is to use Shift+F11 combination in the editor and checking if menus are working fine, without blinking/fading to black (editor should be in multi window mode).

@pattlebass
Copy link
Contributor

pattlebass commented Feb 6, 2023

Video driver timeout after pressing Shift+F11.

Screenshot 2023-02-06 170200

gl_compatibility
RX 5600XT, driver version 22.11.1
Windows 10 21H2

On Forward+ there's no crash, but I get black flashes

@RedworkDE
Copy link
Member

On a laptop with Win10 21H2 and Intel HD Graphics 620, the border is gone but I get the described black flashes when opening menus.

@bruvzg
Copy link
Member Author

bruvzg commented Feb 6, 2023

Tested with integrated AMD Ryzen graphics, and it doesn't work (unlike on NVIDIA, menus are usable, but cause black flashes). So I guess adding border (or changing client area to not match screen size in some other way) is still the only way to prevent exclusive full-screen.

@bruvzg bruvzg closed this Feb 6, 2023
@bruvzg bruvzg deleted the win_fs_no_border branch February 6, 2023 15:21
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.

Enabling fullscreen via code shows window border at screen edges (DisplayServer regression)
4 participants