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

handle unmaximize #537

Closed
totaam opened this issue Mar 19, 2014 · 4 comments
Closed

handle unmaximize #537

totaam opened this issue Mar 19, 2014 · 4 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Mar 19, 2014

Issue migrated from trac ticket # 537

component: server | priority: minor | resolution: fixed

2014-03-19 06:16:04: totaam created the issue


Split from #212: support applications which request that we unmaximize them.

I am not quite sure how this is supposed to work, and we don't really care too much about it: the user can unmaximize client side using the operating system's window controls.

@totaam
Copy link
Collaborator Author

totaam commented Dec 26, 2014

2014-12-26 15:03:30: totaam changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Dec 26, 2014

2014-12-26 15:03:30: totaam changed resolution from ** to fixed

@totaam
Copy link
Collaborator Author

totaam commented Dec 26, 2014

2014-12-26 15:03:30: totaam commented


r8294 fixes the test application, removes some crappy code that was added as a workaround for the buggy test class, and now correctly supports the add / remove / toggle modes.

r8295 synchronizes the client and server side maximized state.

Can be tested using the "test window maximize" test application.
We now also show the window as maximized via "xpra info" when it has been maximized by the client (this was not the case previously).

Backports in 8297 + 8298.

@totaam totaam closed this as completed Dec 26, 2014
@totaam
Copy link
Collaborator Author

totaam commented May 1, 2015

2015-05-01 23:56:17: afarr commented


Re-tested with win32 & osx 0.15.0 9202 (against fedora 21 0.15.0 r9137).

Still works wonderfully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant