Merge lp:~townsend/unity/fix-lp1064155-7.0 into lp:unity/7.0

Proposed by Christopher Townsend on 2013-05-09
Status: Merged
Approved by: Brandon Schaefer on 2013-05-14
Approved revision: 3316
Merged at revision: 3316
Proposed branch: lp:~townsend/unity/fix-lp1064155-7.0
Merge into: lp:unity/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
plugins/unityshell/src/unityshell.cpp (+1/-1)
To merge this branch: bzr merge lp:~townsend/unity/fix-lp1064155-7.0
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) 2013-05-09 Approve on 2013-05-14
PS Jenkins bot (community) continuous-integration Approve on 2013-05-09
Review via email: mp+163197@code.launchpad.net

Commit message

Fixes issue where minimizing a full screen window would not draw the Launcher and Panel after minimization. This is a backport of lp:unity revno. 3320.

Description of the change

Note: This is a backport of lp:unity revno. 3320.

= Issue =
Upon minimizing a full screen window, the Launcher and Panel are not visible.

= Fix =
Add logic to check if full screen windows are minimized.

= Test =
This is a visual issue. Reproduce issue by minimizing a full screen window and observe Launcher and Panel is not visible. Apply this merge and observe that minimizing a full screen window will display the Launcher and Panel.

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Brandon Schaefer (brandontschaefer) wrote :

Sweet, LGTM

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-04-23 16:59:52 +0000
3+++ plugins/unityshell/src/unityshell.cpp 2013-05-09 18:14:24 +0000
4@@ -2505,7 +2505,7 @@
5 uScreen->windows_for_monitor_[monitor] = 1;
6
7 if (!(mask & nonOcclusionBits) &&
8- (window->state() & CompWindowStateFullscreenMask) &&
9+ (window->state() & CompWindowStateFullscreenMask && !window->minimized()) &&
10 uScreen->windows_for_monitor_[monitor] == 1)
11 // And I've been advised to test other things, but they don't work:
12 // && (attrib.opacity == OPAQUE)) <-- Doesn't work; Only set in glDraw

Subscribers

People subscribed via source and target branches