Merge lp://qastaging/~canonical-dx-team/unity/unity.fix-675862 into lp://qastaging/unity
Proposed by
Mirco Müller
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Mirco Müller | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 893 | ||||
| Proposed branch: | lp://qastaging/~canonical-dx-team/unity/unity.fix-675862 | ||||
| Merge into: | lp://qastaging/unity | ||||
| Diff against target: |
170 lines (+107/-0) 2 files modified
src/unityshell.cpp (+99/-0) src/unityshell.h (+8/-0) |
||||
| To merge this branch: | bzr merge lp://qastaging/~canonical-dx-team/unity/unity.fix-675862 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Sam Spilsbury (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
This is based on Mathieu Trudel-Lapierre' branch lp:~mathieu-tl/unity/multimonitor and re-merged with trunk and adapted to the fixing-remarks Sam stated in his comments on the inital merge-proposal. See https:/
I left out point 5.) intentionally for later. This fixes LP: #675862
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Yeah, I think this is better.
I think though that the calls to EnableInputWindow before and after the relayout though are a problem because that's destroying and re-creating windows when you could just be reconfiguring them.