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

unmaximize: use window's current screen for positioning #69

Merged
merged 1 commit into from
May 9, 2020

Conversation

ThomasAdam
Copy link
Member

When moving a maximized window between pages or screens, FVWM records
the previous geometry of the unmaximized window, so it can restore
correctly.

However, this doesn't work well when the maximized window itself has
moved between screens or different pages, since the unmaximized geometry
from the point of view of the maximized window is now incorrect.
Subsequently, unmaximizing a window will put that window in to the
ether.

For now, the approach is to make unmaximizing a window imply
MoveToScreen (relative to the window's screen) such that it remains
visible, and should largely do what the user expects.

Fixes #68

When moving a maximized window between pages or screens, FVWM records
the previous geometry of the unmaximized window, so it can restore
correctly.

However, this doesn't work well when the maximized window itself has
moved between screens or different pages, since the unmaximized geometry
from the point of view of the maximized window is now incorrect.
Subsequently, unmaximizing a window will put that window in to the
ether.

For now, the approach is to make unmaximizing a window imply
MoveToScreen (relative to the window's screen) such that it remains
visible, and should largely do what the user expects.

Fixes #68
@ThomasAdam ThomasAdam self-assigned this May 9, 2020
@ThomasAdam ThomasAdam added the type:bug Something's broken! label May 9, 2020
@ThomasAdam ThomasAdam added this to the 1.0 milestone May 9, 2020
@ThomasAdam ThomasAdam merged commit 29e9a8a into master May 9, 2020
@ThomasAdam ThomasAdam deleted the ta/gh-68 branch May 9, 2020 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something's broken!
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Unmaximizing windows can sometimes vanish
1 participant