Merge lp:~unity-team/unity/fix-empty-systray into lp:unity

Proposed by Neil J. Patel
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: no longer in the source branch.
Merged at revision: 1161
Proposed branch: lp:~unity-team/unity/fix-empty-systray
Merge into: lp:unity
Diff against target: 23 lines (+5/-2)
1 file modified
src/PanelTray.cpp (+5/-2)
To merge this branch: bzr merge lp:~unity-team/unity/fix-empty-systray
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+58306@code.launchpad.net

Description of the change

Bug attached, we shouldn't show the tray if there are no icons.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/PanelTray.cpp'
2--- src/PanelTray.cpp 2011-04-13 10:33:02 +0000
3+++ src/PanelTray.cpp 2011-04-19 13:57:30 +0000
4@@ -57,8 +57,6 @@
5
6 gtk_container_add (GTK_CONTAINER (_window), GTK_WIDGET (_tray));
7 gtk_widget_show (GTK_WIDGET (_tray));
8-
9- gtk_widget_show_all (_window);
10 }
11 }
12
13@@ -109,6 +107,11 @@
14 SetMinMaxSize ((_n_children * 24) + (PADDING * 2), 24);
15 QueueRelayout ();
16 QueueDraw ();
17+
18+ if (_n_children)
19+ gtk_widget_show (_window);
20+ else
21+ gtk_widget_hide (_window);
22 }
23 }
24