Merge lp:~mc-return/compiz/compiz.merge-fix1068173-grid-does-not-follow-preview into lp:compiz/0.9.9
Status: | Merged |
---|---|
Approved by: | Daniel van Vugt |
Approved revision: | 3441 |
Merged at revision: | 3448 |
Proposed branch: | lp:~mc-return/compiz/compiz.merge-fix1068173-grid-does-not-follow-preview |
Merge into: | lp:compiz/0.9.9 |
Diff against target: |
51 lines (+17/-17) 1 file modified
plugins/grid/src/grid.cpp (+17/-17) |
To merge this branch: | bzr merge lp:~mc-return/compiz/compiz.merge-fix1068173-grid-does-not-follow-preview |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Approve | ||
Sam Spilsbury | Approve | ||
PS Jenkins bot | continuous-integration | Pending | |
Review via email: mp+131227@code.launchpad.net |
Commit message
Check on which display your pointer is at, before calculating edges/corners for the snap preview.
The problem here is that detection (if the cursor enters another output) is done after the trigger edges have already been detected and calculated, so if you enter a new screen the preview will still have the old values and not follow the mousepointer to the new monitor. Then if you release the mousebutton the resize won't follow the preview, but resize correctly on the monitor your cursor is currently at, while still showing the old preview.
(LP: #1068173)
Description of the change
Videos are showing the situation best :)
Before the fix: https:/
After the fix:
https:/
This looks fine to me, any thoughts Daniel?