Page MenuHomePhabricator

Remove Reading Lists editing functionality
Closed, ResolvedPublic2 Estimated Story Points

Description

Background

The way the Editing functionality is implemented causes a number of issues. The current way is

  1. featuring an HTML standards compliance issue (div.reading-lists-selectable is nested as span child, which can cause render side-effects)
  2. causing issues with the line-clamping of the card titles as it's sometimes available, sometimes not leading to unnecessary duplication of code.

Goal

  • Decide if full functionality or only UI is removed: As it only affects one API call, it seems better to remove it entirely
  • Remove accordingly

Acceptance criteria

  • Remove editing functionality

Requirement

Scope: ReadingLists extension (desktop + mobile).

  • Remove all Reading List editing functionality and associated code from the UI and backend.
  • This includes all components, buttons, and logic for editing or removing list entries.
  • Remove or clean up the following:
    • Editing mode toggle and toolbar.
    • <edit-button> and <remove-button> in Entries.vue.
    • EditButton and RemoveButton component files.
    • Unused i18n messages related to editing or removing entries.
    • Associated styles from styles.less.
    • API endpoint deleteEntries in ext.readinglists.api.
  • Verify that the Reading List page loads normally without errors when $wgReadingListsEnableSpecialPageToolbar = true.
  • The Special:ReadingLists page must display only viewing functionality (no editing or removal UI).

BDD

Feature: Remove Reading List editing functionality

  Scenario: Viewing Reading Lists after removal
    Given I open Special:ReadingLists with `$wgReadingListsEnableSpecialPageToolbar = true`
    When the page loads
    Then there are no visible edit or remove buttons on Reading List cards
    And no editing mode toggle is available
    And the page displays normally without console or rendering errors

Test Steps

Test Case 1: Verify editing UI removal

  1. Open Special:ReadingLists on a wiki with $wgReadingListsEnableSpecialPageToolbar = true.
  2. Inspect the UI for list entries.
  3. AC1: No edit or remove buttons are visible on any Reading List cards.
  4. AC2: No “edit mode” toggle or toolbar is displayed.
  5. AC3: Layout and styling remain consistent with non-editable Reading List views.

https://phabricator.wikimedia.org/T407282#11321876

QA Results - Prod

ACStatusDetails
1T407282#11321876
2T407282#11321876
3T407282#11321876

Event Timeline

Volker_E renamed this task from Remove Reading lists Editing functionality to Remove Reading Lists editing functionality.Oct 14 2025, 6:34 PM
Volker_E updated the task description. (Show Details)
Sneha set the point value for this task to 2.Oct 15 2025, 5:20 PM

Editing mode is currently available on the special page if you have $wgReadingListsEnableSpecialPageToolbar = true; in your LocalSettings.php

For the UI:

  • Remove editing from EntryItem.vue
  • Remove <edit-button> and <remove-button> on Entries.vue
  • Remove EditButton and RemoveButton components
  • Remove styles that are no longer needed in styles.less

Other cleanup:

  • Remove unused i18n messages
  • Update any tests that pertain to the editing functionality
  • Remove deleteEntries in ext.readinglists.api (as part of this task or could be done as a follow-up task)

Change #1196785 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/ReadingLists@master] app: Remove editing user interface

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

Change #1196786 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/ReadingLists@master] i18n: Remove all messages related to editing, selecting and removing

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

Change #1196786 abandoned by VolkerE:

[mediawiki/extensions/ReadingLists@master] i18n: Remove all messages related to editing, selecting and removing

Reason:

Merged in first in sequence I9d1daf22a2bd1955

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

Change #1196785 merged by jenkins-bot:

[mediawiki/extensions/ReadingLists@master] app: Remove editing user interface functionality and API call

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

HFan-WMF claimed this task.
HFan-WMF subscribed.

Verified that the editing functionality can no longer be used/seen in Beta!

Edtadros subscribed.

Test Result - Prod

Status: ✅ PASS
Environment: testwiki
OS: macOS Tahoe 26.x
Browser: Chrome Canary (latest as of test date)
Device: MS
Emulated Device: NA

Test Case 1: Verify editing UI removal

  1. Open Special:ReadingLists
  2. Inspect the UI for list entries.
  3. AC1: No edit or remove buttons are visible on any Reading List cards.
  4. AC2: No “edit mode” toggle or toolbar is displayed.
  5. AC3: Layout and styling remain consistent with non-editable Reading List views.
screenshot 321.png (1,519×1,143 px, 311 KB)
screenshot 320.png (1,519×1,143 px, 125 KB)