Mail

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

  1. no-other-unread
  2. Merge into 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
1=== modified file 'src/client/notification/libnotify.vala'
2--- src/client/notification/libnotify.vala 2015-11-23 01:31:56 +0000
3+++ src/client/notification/libnotify.vala 2016-01-19 20:01:45 +0000
4@@ -97,14 +97,7 @@
5 return;
6 }
7
8- string body;
9- int count = monitor.get_new_message_count(folder);
10- if (count <= 1) {
11- body = EmailUtil.strip_subject_prefixes(email);
12- } else {
13- body = ngettext("%s\n(%d other new message for %s)", "%s\n(%d other new messages for %s)", count - 1).printf(
14- EmailUtil.strip_subject_prefixes(email), count - 1, folder.account.information.email);
15- }
16+ string body = EmailUtil.strip_subject_prefixes(email);
17
18 // get the avatar
19 Gdk.Pixbuf? avatar = null;

Subscribers

People subscribed via source and target branches