Merge lp:~sao/unity/spread_windows_workspace into lp:unity

Proposed by Oliver Sauder
Status: Merged
Merge reported by: Jason Smith
Merged at revision: not available
Proposed branch: lp:~sao/unity/spread_windows_workspace
Merge into: lp:unity
Diff against target: 17 lines (+6/-1)
1 file modified
plugins/unityshell/src/BamfLauncherIcon.cpp (+6/-1)
To merge this branch: bzr merge lp:~sao/unity/spread_windows_workspace
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+66200@code.launchpad.net

Description of the change

This fixes the issue that the app spread will only show applications on the current workspace.

Not sure if this should be merged before Bug 690143 is fixed as well.

To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) wrote :

Excellent

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/src/BamfLauncherIcon.cpp'
2--- plugins/unityshell/src/BamfLauncherIcon.cpp 2011-06-28 11:49:06 +0000
3+++ plugins/unityshell/src/BamfLauncherIcon.cpp 2011-06-28 19:46:13 +0000
4@@ -504,8 +504,13 @@
5 if (BAMF_IS_WINDOW (view))
6 {
7 guint32 xid = bamf_window_get_xid (BAMF_WINDOW (view));
8+ CompWindow *window = m_Screen->findWindow ((Window) xid);
9
10- windowList.push_back ((Window) xid);
11+ // filter by workspace
12+ if (window && window->defaultViewport () == m_Screen->vp ())
13+ {
14+ windowList.push_back ((Window) xid);
15+ }
16 }
17 }
18