Unity

Merge lp:~azzar1/unity/fix-spread-maximized-windows-keyboard-switch into lp:unity

  1. fix-spread-maximized-windows-keyboard-switch
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 4178
Proposed branch: lp:~azzar1/unity/fix-spread-maximized-windows-keyboard-switch
Merge into: lp:unity
Diff against target: 12 lines (+1/-1)
1 file modified
unity-shared/PluginAdapter.cpp (+1/-1)
To merge this branch: bzr merge lp:~azzar1/unity/fix-spread-maximized-windows-keyboard-switch
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+303139@code.launchpad.net

Commit message

Use compiz::Window::serverNext instead of compiz::Window::next in IsWindowObscured as the latter can be outdated just after scale/spread terminates.

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Good catch!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'unity-shared/PluginAdapter.cpp'
2--- unity-shared/PluginAdapter.cpp 2016-07-05 11:43:32 +0000
3+++ unity-shared/PluginAdapter.cpp 2016-08-17 14:23:28 +0000
4@@ -608,7 +608,7 @@
5
6 CompPoint window_vp = window->defaultViewport();
7 // Check if any windows above this one are blocking it
8- for (CompWindow* sibling = window->next; sibling != NULL; sibling = sibling->next)
9+ for (CompWindow* sibling = window->serverNext; sibling != NULL; sibling = sibling->serverNext)
10 {
11 if (sibling->defaultViewport() == window_vp
12 && !sibling->minimized()