Page MenuHomePhabricator

Reading lists should be sortable
Closed, ResolvedPublic

Description

Background

The reading list API supports sorting my name or the time it was added to the list. We should allow users to sort the list from the Special:ReadingList page.

User story

As a user I want to sort my list

Requirements

TODO

BDD

  • For QA engineer to fill out

Test Steps

  • For QA engineer to fill out

Design

  • Add mockups and design requirements

Acceptance criteria

  • Add acceptance criteria

Communication criteria - does this need an announcement or discussion?

  • Add communication criteria

Rollback plan

  • What is the rollback plan in production for this task if something goes wrong?

This task was created by Version 1.2.0 of the Web team task template using phabulous

Event Timeline

I asked Sarah to include this feature during our 1:1 meeting yesterday. 👍🏽

Jdlrobson-WMF triaged this task as Low priority.EditedMar 24 2025, 10:18 PM

@Dillon I don't think this needs design input (right now). I think what would be useful is just documenting the technical limitations with the existing API and how difficult it would be to change that.

Change #1152355 had a related patch set uploaded (by Dillon; author: Dillon):

[mediawiki/extensions/ReadingLists@master] special: Implement the redesigned pages

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

Test wiki created on Patch demo by DHardy-WMF using patch(es) linked to this task:
https://patchdemo.wmcloud.org/wikis/90698141ee/w/

Change #1152355 merged by jenkins-bot:

[mediawiki/extensions/ReadingLists@master] special: Implement the redesigned pages

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

Jdlrobson-WMF claimed this task.

This came up in the design review today as something that probably won't be in scope for the first version as we'll want to get some data on whether users want it but it seems to be working as expected.

Test wiki on Patch demo by DHardy-WMF using patch(es) linked to this task was deleted:

https://patchdemo.wmcloud.org/wikis/90698141ee/w/