Merge lp:~azzar1/unity/fix-lp-1310200 into lp:unity

Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 3822
Proposed branch: lp:~azzar1/unity/fix-lp-1310200
Merge into: lp:unity
Diff against target: 17 lines (+7/-0)
1 file modified
panel/PanelIndicatorEntryView.cpp (+7/-0)
To merge this branch: bzr merge lp:~azzar1/unity/fix-lp-1310200
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+221486@code.launchpad.net

Commit message

Fix skype indicator blinking.

Description of the change

== Problem ==
Indicators disappear at the panel (clementine, dropbox, skype etc)

== Fix ==
Try to rescan the icon theme in case we don't find the 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 :

Yeah!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'panel/PanelIndicatorEntryView.cpp'
2--- panel/PanelIndicatorEntryView.cpp 2014-03-31 21:08:10 +0000
3+++ panel/PanelIndicatorEntryView.cpp 2014-05-30 07:23:55 +0000
4@@ -229,6 +229,13 @@
5 {
6 glib::Object<GIcon> icon(g_icon_new_for_string(proxy_->image_data().c_str(), nullptr));
7 info = gtk_icon_theme_lookup_by_gicon(theme, icon, size, flags);
8+
9+ if (!info)
10+ {
11+ // Maybe the icon was just added to the theme, see if a rescan helps.
12+ gtk_icon_theme_rescan_if_needed(theme);
13+ info = gtk_icon_theme_lookup_by_gicon(theme, icon, size, flags);
14+ }
15 }
16 else
17 {