Merge lp:~compiz-team/compiz/compiz.fix_1159324 into lp:compiz/0.9.9
Proposed by
Sam Spilsbury
Status: | Merged |
---|---|
Approved by: | Michael Terry |
Approved revision: | 3636 |
Merged at revision: | 3636 |
Proposed branch: | lp:~compiz-team/compiz/compiz.fix_1159324 |
Merge into: | lp:compiz/0.9.9 |
Diff against target: |
476 lines (+312/-93) 4 files modified
plugins/place/src/constrain-to-workarea/include/constrain-to-workarea.h (+6/-0) plugins/place/src/constrain-to-workarea/src/constrain-to-workarea.cpp (+62/-5) plugins/place/src/constrain-to-workarea/tests/constrain-to-workarea/src/test-place-constrain-to-workarea.cpp (+235/-58) plugins/place/src/place.cpp (+9/-30) |
To merge this branch: | bzr merge lp:~compiz-team/compiz/compiz.fix_1159324 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
MC Return | Approve | ||
Compiz Maintainers | Pending | ||
Review via email: mp+155130@code.launchpad.net |
Commit message
Don't offset placement by window extents for windows that have a static
gravity.
Those windows want to be placed as though they didn't have decorations.
Get cp::constrainPo
place tests too, so that they only test one thing at a time.
(LP: #1159324)
Description of the change
Don't offset placement by window extents for windows that have a static
gravity.
Those windows want to be placed as though they didn't have decorations.
Get cp::constrainPo
place tests too, so that they only test one thing at a time.
(LP: #1159324)
To post a comment you must log in.
Confirming this fixes bug #1159324. \o/
All regressions recently introduced are gone.
Tested Guake, Terra and Yakuake drop-down terminals.
All open with right sizes and in correct positions.
Thanks a lot, Sam. +1
P.S.: bazaar. launchpad. net/~compiz- team/compiz/ raring/ revision/ 3639
Raring would need this fix also, because of:
http://
It is a bit painful to suddenly have to deal with a trunk != raring situation...