Last modified: 2013-06-12 23:13:05 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T49846, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47846 - Notification email digests: Copy tweaks
Notification email digests: Copy tweaks
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Echo (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Luke Welling
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-29 21:15 UTC by Fabrice Florin
Modified: 2013-06-12 23:13 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Fabrice Florin 2013-04-29 21:15:34 UTC
As discussed, I found a couple small copy issues related to the email digest on MediaWiki.org:


1. Subject line
It would be great if the subject line could include the site name, as follows:

'You have new notifications on Wikipedia today'

(instead of: 'You have new notifications today')


2. Add a salutation

To be consistent with the HTML email specs, I recommend we add a 'Hi' salutation at the top of the plain text emails, to help make them a bit friendlier:

'Hi Fabrice Florin'

(instead of: 'Fabrice Florin')


3. Add colons (:) to category headers

Given that we have no formatting on plain text emails, I think that adding a ':' colon after the category headers would help establish the relation between the header and the contents below:

'Page link:'

(instead of: 'Page link')


Here are the full feature requirements for this Email Digest feature:

http://www.mediawiki.org/wiki/Echo_(Notifications)/Feature_requirements#Email_Digests
Comment 1 Luke Welling 2013-04-29 21:44:17 UTC
https://gerrit.wikimedia.org/r/#/c/61477
Comment 2 Luke Welling 2013-04-29 23:03:08 UTC
Erik B. points out, and I agree, that wordings of the form "You have 42 new notifications on Wikipedia today" are a bit clunky sounding.


"You have new notifications on Wikipedia" would be cleaner, or if you want to emphasize the time period, past tense is probably more consistent.

"You received 42 new notifications today on Wikipedia"
Comment 3 Fabrice Florin 2013-04-30 00:10:57 UTC
Thanks, guys, excellent point!

I agree with you that it would be best to move 'today' before 'on Wikipedia', to emphasize the date, as you suggest:

'You have new notifications today on Wikipedia'

(instead of: 'You have new notifications on Wikipedia today')

Though I would recommend we stick with 'have' over 'received', because it seems simpler.

Also, for the record, I believe we gave up on giving numbers of notifications (e.g. '42'), because it's hard to accurately estimate the totals due to bundling. Correct me if I'm wrong on that point. :)
Comment 4 bsitu 2013-05-03 00:29:37 UTC
Can we remove the ':' from the category title, for example, 'Talk page post:' and 'Page links:'.  This message is used in the dismiss interface as well, it looks very awkward if it's re-enabled
Comment 5 Fabrice Florin 2013-05-03 01:05:03 UTC
Hi Benny, we have no plans to re-enable the Dismiss feature in its current form. The reason we disabled it is because we decided it was not appropriate at the category level, and needs a different design with more granularity.

So there is no longer any need to support that old feature. Instead, let's optimize for the current feature which is live. Once HTML email is deployed in a couple weeks, we can revisit that decision.

While we're discussing email copy, there are a few other issues I recommend we fix:

* Individual email notification titles should also have 'on Wikipedia' at the end, if we are on a wikipedia.org site. For example, we should say: 'You have a new talkpage message on Wikipedia'.

* The first sentence in individual email notifications about talk pages should have a ':' colon at the end of the sentence, so that it can refer to the payload text below. For example: 
'Fabrice Florin posted on your talk page:
/* Kudos on deploying Notifications! */ new section'

* The first sentence in individual email notifications about page links should have a '.' period at the end of the sentence, so that you know the sentence has ended. For example: 
'Tam Valley was linked from Golden Gate National Recreation Area.'

Please check that all sentences in other individual notifications have an appropriate punctuation mark at the end, so we send grammatically correct notifications to our users.

Thanks!
Comment 6 bsitu 2013-05-03 01:16:12 UTC
(In reply to comment #5)
> Hi Benny, we have no plans to re-enable the Dismiss feature in its current
> form. The reason we disabled it is because we decided it was not appropriate
> at
> the category level, and needs a different design with more granularity.
> 
> So there is no longer any need to support that old feature. Instead, let's
> optimize for the current feature which is live. Once HTML email is deployed
> in
> a couple weeks, we can revisit that decision.
> 
> While we're discussing email copy, there are a few other issues I recommend
> we
> fix:
> 
> * Individual email notification titles should also have 'on Wikipedia' at the
> end, if we are on a wikipedia.org site. For example, we should say: 'You
> have a
> new talkpage message on Wikipedia'.
> 
> * The first sentence in individual email notifications about talk pages
> should
> have a ':' colon at the end of the sentence, so that it can refer to the
> payload text below. For example: 
> 'Fabrice Florin posted on your talk page:
> /* Kudos on deploying Notifications! */ new section'
> 
> * The first sentence in individual email notifications about page links
> should
> have a '.' period at the end of the sentence, so that you know the sentence
> has
> ended. For example: 
> 'Tam Valley was linked from Golden Gate National Recreation Area.'
> 
> Please check that all sentences in other individual notifications have an
> appropriate punctuation mark at the end, so we send grammatically correct
> notifications to our users.
> 
> Thanks!

Hi Fabrice,

The category title is also used in preference page and probably some other places in the future.  In the case colon doesn't apply, we would need to create a new set of category messages.
Comment 7 Fabrice Florin 2013-05-03 01:34:55 UTC
Thanks, Benny. I'm fine with colons after the labels in preferences, but I see your point about future uses. How about adding the colons in the email code? Would that be possible?
Comment 8 Andre Klapper 2013-05-03 09:24:05 UTC
(In reply to comment #7)
> How about adding the colons in the email code? Would that be possible?

If that's a proposal to split strings (turn "foo:" into "foo" and ":") I'd file an I18N bug if I saw that. e.g. French needs "foo :" and there are RTL langs.
If that's not a proposal to split strings, please elaborate.
Comment 9 bsitu 2013-05-03 17:20:11 UTC
(In reply to comment #7)
> Thanks, Benny. I'm fine with colons after the labels in preferences, but I
> see
> your point about future uses. How about adding the colons in the email code?
> Would that be possible?

Fabrice, as Andre pointed out in the previous comment, we usually try to avoid concatenating message in the code.  In English, it is a colon following foo like foo:, but in some other languages it may be some other symbol than colon.
Comment 10 bsitu 2013-05-03 17:21:28 UTC
(In reply to comment #9)
> (In reply to comment #7)
> > Thanks, Benny. I'm fine with colons after the labels in preferences, but I
> > see
> > your point about future uses. How about adding the colons in the email code?
> > Would that be possible?
> 
> Fabrice, as Andre pointed out in the previous comment, we usually try to
> avoid
> concatenating message in the code.  In English, it is a colon following foo
> like foo:, but in some other languages it may be some other symbol than
> colon.

The position of the colon may be different as well
Comment 11 Ryan Kaldari 2013-05-03 18:36:55 UTC
There is a 'colon-separator' i18n message in core that we should be able to use for this.
Comment 12 Luke Welling 2013-05-03 20:17:38 UTC
Passing colon-separator in, and having a space for it as $1 should work then?  That way, we can optionally call it with blank, and a translator could move the $1 to the start of the line if that makes sense for a RTL language?

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links