Last modified: 2014-05-17 17:47:50 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 T44743, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42743 - Watchlist preferences should link to the watchlist e-mail notification option
Watchlist preferences should link to the watchlist e-mail notification option
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
User preferences (Other open bugs)
1.21.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 1932 51942
  Show dependency treegraph
 
Reported: 2012-12-05 22:18 UTC by philip olson
Modified: 2014-05-17 17:47 UTC (History)
8 users (show)

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


Attachments
Patch to add e-mail notification reference to the Watchlist preference tab (1.37 KB, text/plain)
2012-12-05 22:18 UTC, philip olson
Details
Patch to add e-mail notification reference to the Watchlist preference tab (2.25 KB, patch)
2012-12-05 22:42 UTC, philip olson
Details

Description philip olson 2012-12-05 22:18:26 UTC
Created attachment 11472 [details]
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 ''.
Comment 1 philip olson 2012-12-05 22:42:13 UTC
Created attachment 11473 [details]
Patch to add e-mail notification reference to the Watchlist preference tab

Updated patch to also modify MessagesQqq.php.
Comment 2 Andre Klapper 2012-12-06 12:03:24 UTC
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.
Comment 3 Sumana Harihareswara 2013-08-29 16:24:30 UTC
Related to bug 24632 ?  Asking Jared.
Comment 4 Jared Zimmerman (WMF) 2013-08-29 17:50:54 UTC
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.
Comment 5 Jared Zimmerman (WMF) 2013-08-29 17:51:20 UTC
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.
Comment 6 MZMcBride 2013-09-01 03:59:44 UTC
(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>).
Comment 7 Gerrit Notification Bot 2013-10-15 20:35:35 UTC
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

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


Navigation
Links