Page MenuHomePhabricator

Watchlist preferences should link to the watchlist e-mail notification option
Closed, DeclinedPublic

Description

Author: philip

Description:
Patch to add e-mail notification reference to the Watchlist preference tab

The Preferences->Watchlist tab does not refer to the following option:

  • E-mail me when a page on my watchlist is changed

Because that option is set (and listed) under the E-Mail section in Preferences->User Profile.

This patch attempts add this link so users know that this Watchlist E-mail notification option exists.

Note: I'm not sure if this patch is correct, as I cannot test it at this time. For example, it sets default to ''.


Version: 1.21.x
Severity: normal
See Also: T68965: Have watchlist email setting in "Notifications" tab in Special:Preferences

attachment watchlist-user-prefs-email-notification.txt ignored as obsolete

Details

Reference
bz42743

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:49 AM
bzimport set Reference to bz42743.
bzimport added a subscriber: Unknown Object (MLST).

philip wrote:

Patch to add e-mail notification reference to the Watchlist preference tab

Updated patch to also modify MessagesQqq.php.

Attached:

Hi! Thanks for your patch!

You are welcome to use Developer access

https://www.mediawiki.org/wiki/Developer_access

to submit this as a Git branch directly into Gerrit:

https://www.mediawiki.org/wiki/Git/Tutorial

Putting your branch in Git makes it easier for us to review it quickly.
Thanks again! We appreciate your contribution.

sumanah wrote:

Related to bug 24632 ? Asking Jared.

If a link is added i'd prefer that it be more of a "symbolic link" so that it can be toggled on or off from either location, and the values are synced.

I do have some concern over preference creep, where we add more and more thing to the user prefs, including things that are duplicated in multiple screens.

We (design) would love to do an audit of preferences based on the new analytics we're tracking on them to reduce the amount of options and reorganize things into fewer, but more logical buckets.

In this specific case my recommendation would be to have a link associated with the watchlist preferences that says something like "Notification options" that goes to the section where the controls exist now.

If a link is added i'd prefer that it be more of a "symbolic link" so that it can be toggled on or off from either location, and the values are synced.

I do have some concern over preference creep, where we add more and more thing to the user prefs, including things that are duplicated in multiple screens.

We (design) would love to do an audit of preferences based on the new analytics we're tracking on them to reduce the amount of options and reorganize things into fewer, but more logical buckets.

In this specific case my recommendation would be to have a link associated with the watchlist preferences that says something like "Notification options" that goes to the section where the controls exist now.

(In reply to comment #5)

We (design) would love to do an audit of preferences based on the new
analytics we're tracking on them to reduce the amount of options and
reorganize things into fewer, but more logical buckets.

I'm not sure what new analytics you're referring to, but this is bug 52807 (more specifically https://bugzilla.wikimedia.org/showdependencytree.cgi?id=52807&hide_resolved=1).

Change 90002 had a related patch set uploaded by Gerrit Patch Uploader:
Patch to add e-mail notification reference to the Watchlist preference tab

https://gerrit.wikimedia.org/r/90002

thiemowmde subscribed.

This looks like an auto-generated ticket for a patch that does nothing but duplicating a preference.