Merge lp:~charon030/compiz/fix-751605 into lp:compiz/0.9.8
Status: | Merged |
---|---|
Approved by: | Daniel van Vugt on 2012-12-11 |
Approved revision: | 3427 |
Merged at revision: | 3428 |
Proposed branch: | lp:~charon030/compiz/fix-751605 |
Merge into: | lp:compiz/0.9.8 |
Diff against target: |
126 lines (+19/-87) 2 files modified
src/screen.cpp (+5/-22) src/window.cpp (+14/-65) |
To merge this branch: | bzr merge lp:~charon030/compiz/fix-751605 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | 2012-12-08 | Approve on 2012-12-11 | |
Review via email:
|
Commit message
Prevent windows from maximizing on the wrong monitor (LP: #751605)
This fixes two causes I have found:
1. outputDeviceFor
top or left) if a window was mostly off the bottom/right of the screen.
2. Even when outputDeviceFor
incorrect output in the case where the old size of a window exceeds
the dimensions of the smaller monitor you're trying to maximize it on.
.
Description of the change
This is the same bugfix as it was integrated into mainline by Daniel van Vugt before in:
https:/
Michael Terry (mterry) wrote : | # |
Daniel van Vugt (vanvugt) wrote : | # |
The automated tests are in the upstream version (lp:compiz) and should not be backported here because they required some architectural change.
Daniel van Vugt (vanvugt) wrote : | # |
Yep, it's identical to my upstream fix. Just minus the test cases which are not feasible to backport due to architectural changes.
Seems like we should have a test for this case.