Restoring a maximized Terminal with an auto-hide taskbar hides the taskbar #7246

Closed
opened 2026-01-31 00:58:54 +00:00 by claunia · 2 comments
Owner

Originally created by @zadjii-msft on GitHub (Apr 1, 2020).

I have v0.10.781.0 installed. The issue seems to have been partially fixed. After minimizing then maximizing again the issue returns.

Originally posted by @ARabinovich in https://github.com/microsoft/terminal/issues/1438#issuecomment-607372266

Originally created by @zadjii-msft on GitHub (Apr 1, 2020). I have v0.10.781.0 installed. The issue seems to have been partially fixed. After minimizing then maximizing again the issue returns. _Originally posted by @ARabinovich in https://github.com/microsoft/terminal/issues/1438#issuecomment-607372266_
Author
Owner

@zadjii-msft commented on GitHub (Apr 1, 2020):

Found the bug.
a12a6285f5/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp (L366-L371)

When we're restoring from minimized, that MonitorFromWindow call is returning null. Curious that we're getting null here, when MSDN states:

If the window is currently minimized, MonitorFromWindow uses the rectangle of the window before it was minimized.

Hopefully MONITOR_DEFAULTTONEAREST might just fix this? Otherwise we could manually cache the window pos when we minimize...

EDIT: It did.

@zadjii-msft commented on GitHub (Apr 1, 2020): Found the bug. https://github.com/microsoft/terminal/blob/a12a6285f54a02309da930c7de0682c2d44cf243/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp#L366-L371 When we're restoring from minimized, that `MonitorFromWindow` call is returning null. Curious that we're getting null here, when MSDN states: > If the window is currently minimized, MonitorFromWindow uses the rectangle of the window before it was minimized. Hopefully `MONITOR_DEFAULTTONEAREST` might just fix this? Otherwise we could manually cache the window pos when we minimize... EDIT: It did.
Author
Owner

@ghost commented on GitHub (Apr 22, 2020):

:tada:This issue was addressed in #5213, which has now been successfully released as Windows Terminal Preview v0.11.1121.0.🎉

Handy links:

@ghost commented on GitHub (Apr 22, 2020): :tada:This issue was addressed in #5213, which has now been successfully released as `Windows Terminal Preview v0.11.1121.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v0.11.1121.0) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#7246