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
=== modified file 'src/pantheon-notify.vala'
--- src/pantheon-notify.vala 2012-04-20 15:38:41 +0000
+++ src/pantheon-notify.vala 2012-04-20 16:04:19 +0000
@@ -73,7 +73,9 @@
73 } else {73 } else {
74 try { //use app icon, if available74 try { //use app icon, if available
75 tex.set_from_icon_name (new Gtk.Image (), app.down (), Gtk.IconSize.DND);75 tex.set_from_icon_name (new Gtk.Image (), app.down (), Gtk.IconSize.DND);
76 } catch (Error e) {}76 } catch (Error e) {
77 tex.set_from_icon_name (new Gtk.Image (), "dialog-information", Gtk.IconSize.DND);
78 }
77 }79 }
78 var text = new Clutter.Text.with_text ("", "<span font='10' weight='bold'>"+name+"</span>\n"+body);80 var text = new Clutter.Text.with_text ("", "<span font='10' weight='bold'>"+name+"</span>\n"+body);
79 text.use_markup = true;81 text.use_markup = true;

Subscribers

People subscribed via source and target branches

to all changes: