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
=== modified file 'src/window.cpp'
--- src/window.cpp 2012-01-31 14:52:20 +0000
+++ src/window.cpp 2012-02-15 02:34:18 +0000
@@ -2855,9 +2855,17 @@
2855 priv->id == screen->priv->nextActiveWindow)2855 priv->id == screen->priv->nextActiveWindow)
2856 {2856 {
2857 CompWindow *ancestor;2857 CompWindow *ancestor;
2858 CompWindow *nextActive = screen->findWindow (screen->priv->nextActiveWindow);
2858 Window lastNextActiveWindow = screen->priv->nextActiveWindow;2859 Window lastNextActiveWindow = screen->priv->nextActiveWindow;
28592860
2860 if (priv->transientFor && priv->transientFor != screen->root ())2861 /* Window pending focus */
2862 if (priv->id != screen->priv->nextActiveWindow &&
2863 nextActive &&
2864 nextActive->focus ())
2865 {
2866 nextActive->moveInputFocusTo ();
2867 }
2868 else if (priv->transientFor && priv->transientFor != screen->root ())
2861 {2869 {
2862 ancestor = screen->findWindow (priv->transientFor);2870 ancestor = screen->findWindow (priv->transientFor);
2863 if (ancestor &&2871 if (ancestor &&

Subscribers

People subscribed via source and target branches