Merge lp:~sindriava+launchpad/pantheon-notify/default-notification-icon-blueprint into lp:~tombeckmann/pantheon-notify/trunk

Proposed by Jiří Šebele
Status: Merged
Merge reported by: Tom Beckmann
Merged at revision: not available
Proposed branch: lp:~sindriava+launchpad/pantheon-notify/default-notification-icon-blueprint
Merge into: lp:~tombeckmann/pantheon-notify/trunk
Diff against target: 14 lines (+3/-1)
1 file modified
src/pantheon-notify.vala (+3/-1)
To merge this branch: bzr merge lp:~sindriava+launchpad/pantheon-notify/default-notification-icon-blueprint
Reviewer Review Type Date Requested Status
Tom Beckmann Pending
Review via email: mp+102892@code.launchpad.net

Description of the change

I added a default notification icon for notifications that doesn't provide its own and for which icon cannot be determined. The icon used is "dialog-information"

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/pantheon-notify.vala'
2--- src/pantheon-notify.vala 2012-04-20 15:38:41 +0000
3+++ src/pantheon-notify.vala 2012-04-20 16:04:19 +0000
4@@ -73,7 +73,9 @@
5 } else {
6 try { //use app icon, if available
7 tex.set_from_icon_name (new Gtk.Image (), app.down (), Gtk.IconSize.DND);
8- } catch (Error e) {}
9+ } catch (Error e) {
10+ tex.set_from_icon_name (new Gtk.Image (), "dialog-information", Gtk.IconSize.DND);
11+ }
12 }
13 var text = new Clutter.Text.with_text ("", "<span font='10' weight='bold'>"+name+"</span>\n"+body);
14 text.use_markup = true;

Subscribers

People subscribed via source and target branches

to all changes: