Merge lp:~dbarth/compiz-core/mumble-fix into lp:compiz-core/oneiric

Proposed by David Barth
Status: Merged
Approved by: Sam Spilsbury
Approved revision: 2802
Merged at revision: 2802
Proposed branch: lp:~dbarth/compiz-core/mumble-fix
Merge into: lp:compiz-core/oneiric
Diff against target: 23 lines (+5/-1)
1 file modified
src/window.cpp (+5/-1)
To merge this branch: bzr merge lp:~dbarth/compiz-core/mumble-fix
Reviewer Review Type Date Requested Status
Compiz Maintainers Pending
Review via email: mp+73857@code.launchpad.net

Description of the change

I have tested that fix backported from the dev. trunk and confirmed it to fix the issue with mumble.

Test case:
- start mumble
- open the dash at the same time

Issue: the dash appears under other windows; the alt-tab switcher too

With the patch applied, the dash correctly appears on top of the mumble window; that alt-tab switcher as well

To post a comment you must log in.

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 2011-08-23 13:28:57 +0000
+++ src/window.cpp 2011-09-02 17:04:23 +0000
@@ -2182,6 +2182,9 @@
2182 if (w->overrideRedirect ())2182 if (w->overrideRedirect ())
2183 return true;2183 return true;
21842184
2185 if (w->destroyed ())
2186 return true;
2187
2185 if (!w->priv->shaded && !w->priv->pendingMaps)2188 if (!w->priv->shaded && !w->priv->pendingMaps)
2186 {2189 {
2187 if (!w->isViewable () || !w->isMapped ())2190 if (!w->isViewable () || !w->isMapped ())
@@ -3470,7 +3473,8 @@
3470 }3473 }
3471 }3474 }
34723475
3473 mask |= priv->addWindowStackChanges (&xwc, sibling);3476 if (sibling)
3477 mask |= priv->addWindowStackChanges (&xwc, sibling);
3474 }3478 }
34753479
3476 mask |= priv->addWindowSizeChanges (&xwc, priv->serverGeometry);3480 mask |= priv->addWindowSizeChanges (&xwc, priv->serverGeometry);

Subscribers

People subscribed via source and target branches