Add filter set for 'Watchlisted pages' to RC Filters
Closed, ResolvedPublic8 Estimated Story Points
Actions

Description

Add a filter to the RC Filters set that enables (logged in) users to identify changes to pages on their Watchlist. There will be three options, as shown below (with proposed language) below:

Watchlisted pages

On Watchlist
Changes to pages on your Watchlist.

New Watchlist changes
Changes to Watchlisted pages you haven't visited since the changes occurred.

Not on Watchlist
Everything except changes to your Watchlisted pages.

Functionality

"New Watchlist changes" is a subset of "On Watchlist." If the user selects both filters, "New" is shown in the "no-effect" display state.
The "Not on Watchlist" option finds EVERYTHING that is not an "On Watchlist" change. I.e., together, "On Watchlist" and "Not on Watchlist" provide total coverage.
Position of group: add to the end of the existing menu.

Details

Subject	Repo	Branch	Lines +/-
RC Filters: watch filter highlight	mediawiki/core	master	+3 -1
RC Filters: watchlist	mediawiki/core	master	+108 -0
RC Filters: group.allSelected consider subsets	mediawiki/core	master	+20 -2

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Resolved	SBisson	T162675 'Integrated filters'—add all RC Page filters into the new UI along with selected new functions
Resolved	SBisson	T163964 Add filter set for 'Watchlisted pages' to RC Filters
Invalid	None	T164301 "New Watchlist changes" filter does not count users' own edits as visits to Watched pages
Resolved	Catrope	T164314 "Fatal exception of type "Wikimedia\Rdbms\DBQueryError" when url with Watchedlisted filters is used

Event Timeline

• jmatazzoni renamed this task from Add filter set for ' to Add filter set for 'Watchlisted pages' to RC Filters .Apr 26 2017, 11:50 PM

• jmatazzoni created this task.

@Pginer-WMF, the functionality re. the filters being displayed but grayed out was suggested by James. I don't think we have a design for that. Can you please provide one—unless you disagree with this functionality. It was suggested by James. It's not consistent with what we did for Patrolled, though I rather wish it were...

• jmatazzoni moved this task from Untriaged to Ready for Pickup on the Collaboration-Team-Triage (Collab-Team-Q4-Apr-Jun-2017) board.Apr 26 2017, 11:52 PM

• jmatazzoni updated the task description. (Show Details)Apr 27 2017, 12:14 AM

Pginer-WMF added a parent task: T162675: 'Integrated filters'—add all RC Page filters into the new UI along with selected new functions.Apr 27 2017, 7:39 AM

Pginer-WMF mentioned this in T162675: 'Integrated filters'—add all RC Page filters into the new UI along with selected new functions.

I suggest we keep things consistent, and that we either use the grayed out treatment for both this and the patrolled filter, or hide both. In other words, let's make the issue of whether unavailable filters are hidden or grayed out orthogonal to this task, and move it to a separate task.

Catrope updated the task description. (Show Details)Apr 28 2017, 10:01 PM

Catrope set the point value for this task to 8.

In T163964#3222528, @Catrope wrote:

I suggest we keep things consistent, and that we either use the grayed out treatment for both this and the patrolled filter, or hide both. In other words, let's make the issue of whether unavailable filters are hidden or grayed out orthogonal to this task, and move it to a separate task.

Broken out to T164122. Additionally, we don't really have to figure this out for this task just yet, because while RCFilters is in beta, users who are not logged in cannot possibly see the RCFilters UI.

SBisson claimed this task.May 1 2017, 7:26 PM

SBisson moved this task from Ready for Pickup to In Development on the Collaboration-Team-Triage (Collab-Team-Q4-Apr-Jun-2017) board.

Change 351193 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/core@master] RC Filters: watchlist

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

gerritbot added a project: Patch-For-Review.May 1 2017, 8:58 PM

SBisson moved this task from In Development to Needs Review on the Collaboration-Team-Triage (Collab-Team-Q4-Apr-Jun-2017) board.May 1 2017, 8:59 PM

Change 351309 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/core@master] RC Filters: group.allSelected consider subsets

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

Catrope moved this task from Needs Review to QA Review on the Collaboration-Team-Triage (Collab-Team-Q4-Apr-Jun-2017) board.May 2 2017, 5:33 PM

Change 351309 merged by jenkins-bot:
[mediawiki/core@master] RC Filters: group.allSelected consider subsets

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

Change 351193 merged by jenkins-bot:
[mediawiki/core@master] RC Filters: watchlist

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

ReleaseTaggerBot added a project: MW-1.30-release-notes.May 2 2017, 6:00 PM

@Etonkovidova showed me that this was broken somehow. IIRC it was that when both filtering and highlighting by "New watchlist changes", you'd expect all displayed changes to be highlighted, but they weren't.

In T163964#3229113, @Catrope wrote:

@Etonkovidova showed me that this was broken somehow. IIRC it was that when both filtering and highlighting by "New watchlist changes", you'd expect all displayed changes to be highlighted, but they weren't.

From what I can see it highlights too much. Patch coming.

Change 351380 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/core@master] RC Filters: watch filter highlight

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

Change 351380 merged by jenkins-bot:
[mediawiki/core@master] RC Filters: watch filter highlight

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

ReleaseTaggerBot edited projects, added MW-1.30-release-notes (WMF-deploy-2017-05-09_(1.30.0-wmf.1)); removed MW-1.30-release-notes.May 2 2017, 10:00 PM