Application Indicators

Merge lp://qastaging/~ted/indicator-application/hide-on-unref into lp://qastaging/indicator-application/0.4

  1. hide-on-unref
  2. Merge into trunk
Proposed by Ted Gould
Status: Merged
Merged at revision: not available
Proposed branch: lp://qastaging/~ted/indicator-application/hide-on-unref
Merge into: lp://qastaging/indicator-application/0.4
Diff against target: 11 lines (+1/-0)
1 file modified
src/libappindicator/app-indicator.c (+1/-0)
To merge this branch: bzr merge lp://qastaging/~ted/indicator-application/hide-on-unref
Reviewer Review Type Date Requested Status
Indicator Applet Developers Pending
Review via email: mp+20149@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Ted Gould (ted) wrote :

Hiding the status icon before unreffing it to make working with the
notification area cleaner.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/libappindicator/app-indicator.c'
2--- src/libappindicator/app-indicator.c 2010-02-23 14:34:38 +0000
3+++ src/libappindicator/app-indicator.c 2010-02-25 16:56:20 +0000
4@@ -885,6 +885,7 @@
5 {
6 g_signal_handlers_disconnect_by_func(G_OBJECT(self), status_icon_status_wrapper, status_icon);
7 g_signal_handlers_disconnect_by_func(G_OBJECT(self), status_icon_changes, status_icon);
8+ gtk_widget_hide(GTK_WIDGET(status_icon));
9 g_object_unref(G_OBJECT(status_icon));
10 return;
11 }

Subscribers

People subscribed via source and target branches