Page MenuHomePhabricator

Denote whether a change has already been opened by this user in the PersonalDashboard Review Changes feed
Open, In Progress, HighPublic1 Estimated Story Points

Description

As users browse the Review Changes feed in the PersonalDashboard, it would be helpful to denote which edits the user has already viewed, so that they can more easily visually scan edits they haven't looked at yet.

Since we're pulling edits from a variety of sources, and not doing so chronologically, it is likely that on a page reload we will display an entirely new set of edits to the user. As such, I think at this stage it is OK for this record to only persist for the duration of a single visit to the feed.

Designs

These designs assume that T421901 has been completed.

Figma here.

Frame 427318684.png (1×2 px, 280 KB)

Acceptance criteria

  • When a user clicks an entry in the Review Changes or Active Discussions modules, the card changes visual style to indicate it has been selected already.
  • This state does not need to persist beyond the active user session/page load.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Samwalton9-WMF updated the task description. (Show Details)
Samwalton9-WMF changed the task status from Open to Stalled.Tue, Apr 14, 3:15 PM
Samwalton9-WMF moved this task from Estimated to Kanban on the Moderator-Tools-Team board.

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

[mediawiki/extensions/PersonalDashboard@master] Improve UI of Review Changes and Active Discussions cards

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

Dillon changed the task status from Stalled to In Progress.Fri, Apr 24, 2:40 PM
Dillon subscribed.

The acceptance criteria for this task explicitly states no persistence is needed, therefore we can unblock it from T422607: Define and add Pinia storage for PersonalDashboard Review Changes module.

Resolved in the Gerrit patch for T421901: Improve UI of Review Changes and Active Discussions cards, as it was trivial to implement using the :visited CSS selector on the <a> tag in ListCard.vue.

Test wiki created on Patch demo by DHardy-WMF using patch(es) linked to this task:
https://374f2a3f9e.catalyst.wmcloud.org/w/

Change #1276815 merged by jenkins-bot:

[mediawiki/extensions/PersonalDashboard@master] Improve UI of Review Changes and Active Discussions cards

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

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

https://374f2a3f9e.catalyst.wmcloud.org/w/