Merge lp:~townsend/unity/fix-force-show-window into lp:unity

Proposed by Christopher Townsend
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 3995
Proposed branch: lp:~townsend/unity/fix-force-show-window
Merge into: lp:unity
Diff against target: 22 lines (+4/-0)
1 file modified
unity-shared/PluginAdapter.cpp (+4/-0)
To merge this branch: bzr merge lp:~townsend/unity/fix-force-show-window
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Marco Trevisan (Treviño) Approve
Review via email: mp+260378@code.launchpad.net

Commit message

Also use the Compiz show() method when forcing an unmapped window to be visible when clicking on it's active Launcher icon.

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 :

Well, this covers the case we're focusing.

There's still a problem when unity crashes, so that it's impossible those windows propertly, but still an edge case.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'unity-shared/PluginAdapter.cpp'
2--- unity-shared/PluginAdapter.cpp 2015-04-09 14:42:07 +0000
3+++ unity-shared/PluginAdapter.cpp 2015-05-29 12:31:22 +0000
4@@ -781,7 +781,10 @@
5 {
6 CompWindow* window = m_Screen->findWindow(window_id);
7 if (window && (window->actions() & CompWindowActionMinimizeMask))
8+ {
9 window->unminimize();
10+ window->show();
11+ }
12 }
13
14 void PluginAdapter::Shade(Window window_id)
15@@ -962,6 +965,7 @@
16 if (forced_unminimize)
17 {
18 top_window->unminimize();
19+ top_window->show();
20 }
21
22 top_window->raise();