Merge lp:~smspillaz/compiz-core/compiz-core.fix_931958 into lp:compiz-core

Proposed by Sam Spilsbury
Status: Merged
Merged at revision: 3000
Proposed branch: lp:~smspillaz/compiz-core/compiz-core.fix_931958
Merge into: lp:compiz-core
Diff against target: 22 lines (+9/-1)
1 file modified
src/window.cpp (+9/-1)
To merge this branch: bzr merge lp:~smspillaz/compiz-core/compiz-core.fix_931958
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Review via email: mp+93114@code.launchpad.net

Description of the change

Always prefer to move the focus to windows that are pending a FocusIn because they requested it

Bug 931958

To post a comment you must log in.
Revision history for this message
Sam Spilsbury (smspillaz) wrote :

No tests yet, will put that in another branch

Revision history for this message
Tim Penhey (thumper) wrote :

Sam, can you talk with Alan about how to get this area under test?

Cheers.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/window.cpp'
2--- src/window.cpp 2012-01-31 14:52:20 +0000
3+++ src/window.cpp 2012-02-15 02:34:18 +0000
4@@ -2855,9 +2855,17 @@
5 priv->id == screen->priv->nextActiveWindow)
6 {
7 CompWindow *ancestor;
8+ CompWindow *nextActive = screen->findWindow (screen->priv->nextActiveWindow);
9 Window lastNextActiveWindow = screen->priv->nextActiveWindow;
10
11- if (priv->transientFor && priv->transientFor != screen->root ())
12+ /* Window pending focus */
13+ if (priv->id != screen->priv->nextActiveWindow &&
14+ nextActive &&
15+ nextActive->focus ())
16+ {
17+ nextActive->moveInputFocusTo ();
18+ }
19+ else if (priv->transientFor && priv->transientFor != screen->root ())
20 {
21 ancestor = screen->findWindow (priv->transientFor);
22 if (ancestor &&

Subscribers

People subscribed via source and target branches