Page MenuHomePhabricator

Implement Curation Toolbar container in Vue.js
Closed, ResolvedPublic3 Estimated Story Points

Description

As a first step towards migrating the Curation Toolbar to Vue.js, we need to build the container, which the other features will live inside.

The Curation Toolbar should be able to:

  • Minimize
  • Open after being minimized
  • Navigate to the next article in the queue

We can use the pagetriage_tb parameter to access the new version (=new)

Details

Event Timeline

Samwalton9-WMF triaged this task as Low priority.
Samwalton9-WMF set the point value for this task to 3.

Change 938956 had a related patch set uploaded (by Jsn.sherman; author: Jsn.sherman):

[mediawiki/extensions/PageTriage@master] WIP Vue.js Curation Toolbar container

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

jsn.sherman changed the task status from Open to In Progress.Aug 3 2023, 3:56 AM

Contining some discussion from T343332#9120049:
Note that we're not planning on renaming the feature flag until after all testing is complete.

[...]

Rough draft idea:

  • &pagetriage_ui unset - Backbone everything

Makes sense, and is the current behavior.

  • &pagetriage_ui=1 - Special:NewPagesFeed vue rewrite
  • &pagetriage_ui=2 - Special:NewPagesFeed vue rewrite, AND page curation toolbar vue rewrite, AND links in Special:NewPagesFeed get appended with &ui_version=2

The current patch for the new toolbar can't work this way. vue feed can only share state with vue toolbar, and backbone feed only can share state with backbone toolbar.

I actually have a followup patch in T342434: Special:NewPagesFeed and Page Curation toolbar must share and persist state in progress to make the new feed and toolbar use the same state as the old one in order to maintain per-account persistence of settings. The original vue local storage implementation only kept settings per device. I would still recommend sticking with a single flag to toggle between vue and backbone everywhere, but this would be an option after the followup patch.

Test wiki created on Patch demo by SCardenas (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/bda3f73aef/w

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

https://patchdemo.wmflabs.org/wikis/bda3f73aef/w/

Change 938956 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] Vue.js Curation Toolbar container

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

Samwalton9-WMF updated the task description. (Show Details)
Samwalton9-WMF updated the task description. (Show Details)