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

Kicked out of fullscreen on AppSDK 1.6 #10152

Open
tom-huntington opened this issue Nov 11, 2024 · 1 comment
Open

Kicked out of fullscreen on AppSDK 1.6 #10152

tom-huntington opened this issue Nov 11, 2024 · 1 comment
Labels
needs-triage Issue needs to be triaged by the area owners

Comments

@tom-huntington
Copy link

tom-huntington commented Nov 11, 2024

Describe the bug

I upgraded to AppSDK 1.6 and my app would not go fullscreen (Boarderless).

Specifically this is a problem when the task bar is "minimized" (one or two pixels). There is no problem when the task bar is fixed. And you are using SetWindowLong, SetWindowPos and ShowWindow to enter fullscreen.

If you already are in fullscreen, then AudioGraph.CreateDeviceInputNodeAsync will cause the 2 pixels of the minimized task bar to be shown.

If first you call AudioGraph.CreateDeviceInputNodeAsync then after that you are unable to go fullscreen. The 2 pixels of the minimized task bar will still be there.

Steps to reproduce the bug

I have made a MRE with a fullscreen button, and a button that calls AudioGraph.CreateDeviceInputNodeAsync

https://github.com/tom-huntington/SetWindowLongMRE

NuGet package version

Windows App SDK 1.6.1: 1.6.240923002

Packaging type

Packaged (MSIX)

Windows version

Windows 11 version 21H2 (22000)

IDE

Visual Studio 2022

Additional context

Previously to upgrading to 1.6 the debugger did not break of exceptions in SetWindowLong. After upgrading it breaks on exceptions in SetWindowLong. (Its removing WS_CAPTION that causes the exceptions)

Note: This AudioGraph.CreateDeviceInputNodeAsync behaviour only happens when launching F5/ctrl+F5. When starting from start menu the fullscreen is messed up from the start)

@microsoft-github-policy-service microsoft-github-policy-service bot added the needs-triage Issue needs to be triaged by the area owners label Nov 11, 2024
@RDMacLachlan RDMacLachlan transferred this issue from microsoft/WindowsAppSDK Nov 11, 2024
@tom-huntington tom-huntington changed the title AudioGraph.CreateDeviceInputNodeAsync breaks fullscreen on AppSDK 1.6 Kicked out of fullscreen on AppSDK 1.6 Nov 11, 2024
@tom-huntington
Copy link
Author

tom-huntington commented Nov 11, 2024

This is only a problem with the Win32 Fullscreen APIs. The AppSDK Fullscreen API works fine

AppWindow.SetPresenter(Microsoft::UI::Windowing::AppWindowPresenterKind::FullScreen);

I wonder what AppWindow.SetPresenter is doing differently?

The minimized task bar will go away with HWND_TOPMOST

@tom-huntington tom-huntington changed the title Kicked out of fullscreen on AppSDK 1.6 Z order is changed AudioGraph.CreateDeviceInputNodeAsync on AppSDK 1.6 Nov 11, 2024
@tom-huntington tom-huntington changed the title Z order is changed AudioGraph.CreateDeviceInputNodeAsync on AppSDK 1.6 Z order is changed by AudioGraph.CreateDeviceInputNodeAsync on AppSDK 1.6 Nov 12, 2024
@tom-huntington tom-huntington changed the title Z order is changed by AudioGraph.CreateDeviceInputNodeAsync on AppSDK 1.6 Kicked out of fullscreen on AppSDK 1.6 Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Issue needs to be triaged by the area owners
Projects
None yet
Development

No branches or pull requests

1 participant