Page MenuHomePhabricator

Separate mediawiki.page.patrol.ajax to be useable in mobile
Closed, ResolvedPublic

Description

As commented in https://gerrit.wikimedia.org/r/#/c/213202/

Maybe a seperation of the module into a backend and frontend module is possible to be useable and smart enough for mobile :)

Event Timeline

Florian claimed this task.
Florian raised the priority of this task from to Low.
Florian updated the task description. (Show Details)
Florian added subscribers: gerritbot, Maryana, Andyrom75 and 9 others.

Change 213202 had a related patch set uploaded (by Florianschmidtwelzow):
Make mediawiki.page.patrol.ajax.js and the patrol link more flexible

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

Change 216128 had a related patch set uploaded (by Florianschmidtwelzow):
Some design to the matrol link and use ajax to patrol

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

Was this solved? Or does it actually not block T55684: Add support for mobile change patrolling?

It's something special i think: Patrolling is already available, but it's reload the page and doesn't use the ajax that we know from desktop. It's not blocking it in fact, but it's related :) (Btw.: The blocking was added by the "Create subtask" workflow).

Jdlrobson changed the task status from Open to Stalled.Sep 23 2015, 7:45 PM
Nemo_bis renamed this task from Seperate mediawiki.page.patrol.ajax to be useable in mobile to Separate mediawiki.page.patrol.ajax to be useable in mobile.Sep 25 2015, 6:40 AM

Change 213202 merged by jenkins-bot:
diff: Make DifferenceEngine patrol link creation more flexible

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

Change 216128 merged by jenkins-bot:
Some design to the patrol link and use ajax to patrol

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

Above patches were merged. This should be working now (?). Please create a new ticket if there is something more specific that needs doing.