Merge lp:~brandontschaefer/unity/lp.1231679-fix into lp:unity

Proposed by Brandon Schaefer
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 3537
Proposed branch: lp:~brandontschaefer/unity/lp.1231679-fix
Merge into: lp:unity
Diff against target: 15 lines (+4/-1)
1 file modified
plugins/unityshell/src/unityshell.cpp (+4/-1)
To merge this branch: bzr merge lp:~brandontschaefer/unity/lp.1231679-fix
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+187912@code.launchpad.net

Commit message

If we are switching view ports tell nux to process the events. Even though the viewport switcher does grab the mouse, we will still feed nux the events so we can correctly update where the mouse is for the launcher

Description of the change

If we are switching view ports tell nux to process the events. Even though the viewport switcher does grab the mouse, we will still feed nux the events so we can correctly update where the mouse is for the launcher

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

nice

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/unityshell.cpp'
2--- plugins/unityshell/src/unityshell.cpp 2013-09-24 20:18:42 +0000
3+++ plugins/unityshell/src/unityshell.cpp 2013-09-26 22:46:23 +0000
4@@ -1695,7 +1695,10 @@
5 break;
6 }
7
8- if (!skip_other_plugins && !switcher_controller_->IsMouseDisabled() &&
9+ if (switcher_controller_->IsMouseDisabled() && switcher_controller_->Visible())
10+ skip_other_plugins = true;
11+
12+ if (!skip_other_plugins &&
13 screen->otherGrabExist("deco", "move", "switcher", "resize", nullptr))
14 {
15 wt->ProcessForeignEvent(event, nullptr);