Merge lp:~smspillaz/unity/unity.fix_843958 into lp:unity

Proposed by Sam Spilsbury
Status: Merged
Merged at revision: 1714
Proposed branch: lp:~smspillaz/unity/unity.fix_843958
Merge into: lp:unity
Diff against target: 28 lines (+9/-1)
1 file modified
plugins/unityshell/src/PanelMenuView.cpp (+9/-1)
To merge this branch: bzr merge lp:~smspillaz/unity/unity.fix_843958
Reviewer Review Type Date Requested Status
Robert Carr (community) Approve
Jason Smith (community) Approve
Review via email: mp+79011@code.launchpad.net

Description of the change

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

works for me

review: Approve
Revision history for this message
Robert Carr (robertcarr) wrote :

+1

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/PanelMenuView.cpp'
2--- plugins/unityshell/src/PanelMenuView.cpp 2011-10-10 22:49:01 +0000
3+++ plugins/unityshell/src/PanelMenuView.cpp 2011-10-11 16:57:23 +0000
4@@ -1258,6 +1258,7 @@
5 void
6 PanelMenuView::OnMaximizedGrabStart(int x, int y, unsigned long button_flags, unsigned long)
7 {
8+ Window maximized_win;
9 if (nux::GetEventButton(button_flags) != 1 || _places_showing)
10 return;
11
12@@ -1266,8 +1267,15 @@
13 //
14 // This is a workaround to avoid that the grid plugin would be fired
15 // showing the window shape preview effect. See bug #838923
16- if (GetMaximizedWindow() != 0)
17+
18+ maximized_win = GetMaximizedWindow ();
19+
20+ if (maximized_win != 0)
21+ {
22+ /* Always activate the window in case it is on another monitor */
23+ WindowManager::Default ()->Activate (maximized_win);
24 _panel_titlebar_grab_area->SetGrabbed(true);
25+ }
26 }
27
28 void