Merge lp:~elementary-apps/pantheon-mail/no-other-unread into lp:~elementary-apps/pantheon-mail/trunk

Proposed by Danielle Foré
Status: Merged
Approved by: Danielle Foré
Approved revision: 1942
Merged at revision: 1945
Proposed branch: lp:~elementary-apps/pantheon-mail/no-other-unread
Merge into: lp:~elementary-apps/pantheon-mail/trunk
Diff against target: 19 lines (+1/-8)
1 file modified
src/client/notification/libnotify.vala (+1/-8)
To merge this branch: bzr merge lp:~elementary-apps/pantheon-mail/no-other-unread
Reviewer Review Type Date Requested Status
Ralph Plawetzki (community) Approve
elementary Apps team Pending
Review via email: mp+283210@code.launchpad.net

Commit message

Remove notification text about "other unread messages"

To post a comment you must log in.
Revision history for this message
Ralph Plawetzki (purejava) wrote :

Looks good and works as expected.
Can be merged.

Revision history for this message
Ralph Plawetzki (purejava) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/client/notification/libnotify.vala'
--- src/client/notification/libnotify.vala 2015-11-23 01:31:56 +0000
+++ src/client/notification/libnotify.vala 2016-01-19 20:01:45 +0000
@@ -97,14 +97,7 @@
97 return;97 return;
98 }98 }
99 99
100 string body;100 string body = EmailUtil.strip_subject_prefixes(email);
101 int count = monitor.get_new_message_count(folder);
102 if (count <= 1) {
103 body = EmailUtil.strip_subject_prefixes(email);
104 } else {
105 body = ngettext("%s\n(%d other new message for %s)", "%s\n(%d other new messages for %s)", count - 1).printf(
106 EmailUtil.strip_subject_prefixes(email), count - 1, folder.account.information.email);
107 }
108 101
109 // get the avatar102 // get the avatar
110 Gdk.Pixbuf? avatar = null;103 Gdk.Pixbuf? avatar = null;

Subscribers

People subscribed via source and target branches