Page MenuHomePhabricator

Add a "Mark all as read" button at Special:Notifications for no-JS users
Closed, ResolvedPublic

Description

Add a "Mark all as read" button at Special:Notifications for mobile and no-JS users, e.g. https://www.mediawiki.org/wiki/Special:Notifications


See also: T71732: I want to clear the indicator for messages (For editors with hundreds of notifications)

Event Timeline

Quiddity raised the priority of this task from to Needs Triage.
Quiddity updated the task description. (Show Details)
Quiddity subscribed.
EBernhardson added a project: good first task.
EBernhardson set Security to None.

Presently, Special:Notifications mobile page has 'Mark group as read' for notifications grouped by day:

IMG_1042.PNG (1×640 px, 79 KB)

no-JS mode also displays 'Mark group as read'

Screen Shot 2016-06-07 at 2.24.44 PM.png (407×775 px, 51 KB)

Catrope renamed this task from Add a "Mark all as read" button at Special:Notifications for mobile and no-JS users to Add a "Mark all as read" button at Special:Notifications for no-JS users.Aug 8 2016, 9:55 PM

Changing title because this is done in mobile, just not in no-JS

This is a micro-task, and someone asked for advice on IRC. As MatmaRex said, start with https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker and the links in there. After installing Vagrant (see link in there), you can enable the 'echo' role as stated at https://www.mediawiki.org/wiki/MediaWiki-Vagrant#Using_roles .

I would recommend you start by reading https://phabricator.wikimedia.org/diffusion/ECHO/browse/master/includes/special/SpecialNotificationsMarkRead.php and skimming the class referenced from onSubmit.

When you start working on it, please claim the task.

Change 371614 had a related patch set uploaded (by MtDu; owner: MtDu):
[mediawiki/extensions/Echo@master] [WIP] Add 'Mark all as read' button

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

Change 371614 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Add 'Mark all as read' button

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