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

Taskbar is still there in full screen mode entered from maximized window #2577

Closed
kingcean opened this issue Jun 5, 2022 · 9 comments
Closed
Assignees
Labels
area-Windowing bug Something isn't working

Comments

@kingcean
Copy link

kingcean commented Jun 5, 2022

Describe the bug

The taskbar is not covered if the window was maximized before entering full screen.

Steps to reproduce the bug

  1. Maximize the window.
  2. Enter full screen.

Expected behavior

The taskbar should disappear until the window closes or exits from full screen.

Screenshots

Maximize window
↑ Maximize window

Full screen
↑ Then enter full screen (but you can still see the taskbar)

NuGet package version

1.1.0

Packaging type

Packaged (MSIX)

Windows version

Windows 11 version 21H2 (22000)

IDE

Visual Studio 2022

Additional context

No response

@marb2000
Copy link

marb2000 commented Jun 6, 2022

Hey @kingcean do you have a repro so we can take a look at the APIs that you are using?

@kingcean
Copy link
Author

kingcean commented Jun 7, 2022

Yes. You can maximize the window first (by programming or by clicking/tapping the caption button), and then enter full screen to have a look.

// var appWindow = ... The AppWindow instance of MainWindow.
var presenter = appWindow.Presenter as Microsoft.UI.Windowing.OverlappedPresenter;
presenter.Maximize();
appWindow.SetPresenter(Microsoft.UI.Windowing.AppWindowPresenterKind.FullScreen);

@riverar
Copy link
Contributor

riverar commented Jun 7, 2022

This repros 100% if window is in maximized state before fullscreen presenter is set.

Repro project:
was_issue_2577.zip

AppWindow seems to really unreliable and a continuous source of bugs. Would recommend increasing test suite coverage and engineering scrutiny on AppWindow classes.

@dotMorten
Copy link
Contributor

I can repro this with the WinUIEx Sample app. Just run the app, maximize window, then select the "Fullscreen" presenter in the presenter dropdown. My app restores to the non-maximized state when I do this, yet claims to be fullscreen.

@rkarman
Copy link

rkarman commented Jun 7, 2022

Yeah, this one should not have been able to slip through the automated tests. 😞 I've logged a task for the dev team and we'll be adding it to a servicing release as soon as we have a fix.

MSFT dev team tracking: http://task.ms/39908320

@kingcean
Copy link
Author

I can still repro this on Windows App SDK 1.1.1.

@koenvd
Copy link

koenvd commented Sep 21, 2022

Still reproducible with Windows App SDK 1.1.4.

@stevewri
Copy link
Contributor

stevewri commented Nov 1, 2022 via email

@kingcean
Copy link
Author

kingcean commented Nov 11, 2022

Fixed - v1.2.
However, if you exit full screen and restore, the window size is not the one before maximizing and entering full screen. For example, set window size to 1000x600 → maximize → enter full screen → exit full screen → restore, then the window size is not 1000x600 anymore but fill the screen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Windowing bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants