Page MenuHomePhabricator

C2. Implement patrolling other people's edits in Flow
Closed, ResolvedPublic3 Estimated Story Points

Description

Wiki contributors use edit patrolling to make sure that a trusted contributor has looked at a newbie's edit, and thinks it's a good contribution. When there's an unpatrolled edit, that item has a red exclamation point in recent changes and watchlist. The experienced contributor can review the edit on the diff page or the oldid page, and click a link to signal they've marked it as patrolled. See below for example screenshots.

For Flow posts and edits made by users who need to be patrolled:

On the oldid page, put a link that says [ Mark this page as patrolled ]

On the diff page, put a link that says [ Mark as patrolled ]

Those link to an &action=markpatrolled link.

When you click that, the [ Mark as patrolled ] link disappears, and there's a temporary modal (mw.notify): "The change to Pagename has been marked as patrolled."

Examples and screenshots:

Link to oldidpage from recent changes:

flow example - rc link to oldid.jpg (93×728 px, 84 KB)

Link to oldidpage from history:

flow example - link to oldid from history.jpg (211×873 px, 129 KB)

Wikitext oldid page, showing the mark as patrolled link:

wikitext example - oldid page.jpg (488×1 px, 358 KB)

Flow oldid page, where the link should be:

flow example - oldid page.jpg (505×1 px, 341 KB)

Wikitext diff page, showing the mark as patrolled link:

2_banjo_mark_as_patrolled_link.jpg (511×1 px, 263 KB)

Flow diff page, where the link should be:

flow example - diff page.jpg (435×1 px, 163 KB)

Example of the &action=markpatrolled link:

patrolled link.jpg (24×823 px, 41 KB)

The temporary modal that says the edit has been patrolled:

3_banjo_patrolled.jpg (498×1 px, 254 KB)

In grouped recent changes, the clickable timestamps should also be a link to the oldid page:

grouped rc oldid link.jpg (290×757 px, 274 KB)

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
DannyH updated the task description. (Show Details)
DannyH added a project: Design.

A question for the smart people who know things:

As far as I understand it, the [ Mark as patrolled ] link appears on the diff page.

But -- new pages don't have a diff page, and Flow treats posts as if they're each a new page. So where does the Mark as patrolled link go?

On wikitext pages, it shows in the footer on both the regular view and oldid view of the original revision (maybe later revisions too), if the oldest revision is unpatrolled (which it would be for a fresh new page).

Our analogy to the first oldid view is http://flow-tests.wmflabs.org/w/index.php?title=Topic:Sd4yv8rsdn6x53h0&topic_postId=sd7ejjb1nqh8j49i&topic_revId=sd7ejjb1nqh8j49i&action=single-view . We could show it in the footer there.

For the post itself in normal view, we could consider showing it in ... menu.

Patrol regular view.png (203×1 px, 13 KB)

Patrol oldid view.png (240×1 px, 22 KB)

Hmm, I can see how you get to the oldid view from the history page if you click on the time/datestamp link, but I don't see a way to get there from recent changes.

edit patrol new pages.jpg (96×1 px, 127 KB)

So if you want to edit patrol these new pages from RC, where would you click?

DannyH updated the task description. (Show Details)
DannyH added a subscriber: Pginer-WMF.

Per our discussion, the link also shows on normal page views (not just oldid). See first screenshot in my last post.

@Mattflaschen, how would you get to the corresponding page for a Flow post? They all get displayed in the same thread.

For Flow, instead of a link in the bottom right of the view page, we could use a Patrol entry in the ... menu.

I guess that would patrol the initial version of the post, and diffs could be patrolled separately.

I don't want to put an item in the menu -- it's a task that not many people do, and having it in the menu would confuse people.

I think having the link on the diff page and the oldid page is enough -- patrollers will be able to get there from RC and the watchlist.

I don't want to put an item in the menu -- it's a task that not many people do, and having it in the menu would confuse people.

Only people who have the 'patrol' right will see it, though, and only if it's not already patrolled.

Oh, that's a good point. Silly of me not to think of that. Yeah, we could do that. Do you think people would like it?

I think it's consistent with the other moderation actions we already have there (hide, delete, etc.). People will probably be fine with it.

When you say "in the menu", do you mean the drop down menu at the right side of the post? If so, would that mean that patrollers need to open the dropdown to see whaether a post is patrolled and take action?

When you say "in the menu", do you mean the drop down menu at the right side of the post? If so, would that mean that patrollers need to open the dropdown to see whaether a post is patrolled and take action?

Yes. But it's a hover menu, and won't people normally be finding stuff to patrol from Recent Changes or Watchlist (so they'll already know it needs to be patrolled)?

Mmmok, it's worth trying this in real use.

DannyH renamed this task from Implement patrolling other people's edits in Flow to C2. Implement patrolling other people's edits in Flow.Jun 3 2015, 7:13 PM
DannyH raised the priority of this task from Medium to High.

Change 216312 had a related patch set uploaded (by Sbisson):
[WIP] RC patrolling for Flow

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

Roan pointed out that Stephane's patch didn't have a way to get to the oldid page from grouped recent changes.

That's my bad, I forgot to call that out specifically. I updated the ticket, and added a screenshot:

grouped rc oldid link.jpg (290×757 px, 274 KB)

The timestamp is not even a link when there is only one entry (no grouping). The diff link is not working. And there is less information compared to the non-grouped RC page.
Should we also address those? It starts to look like a new story.

Screen Shot 2015-06-10 at 10.13.39.png (67×493 px, 17 KB)

Screen Shot 2015-06-10 at 10.17.30.png (71×627 px, 19 KB)

Yeah, good idea -- let's do grouped RC as a separate ticket. This one can go through review without the grouped RC links.

Change 216312 merged by jenkins-bot:
RC patrolling for Flow

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

Is it supposed to show

02:20, 11 June 2015 ... marked revision 0 of page Tema:lkjahsdlkjahdlkjahdlkjahsd

at
http://ca.wikipedia.beta.wmflabs.org/wiki/Especial:Registre/patrol?uselang=en
?

Thx!

Is it supposed to show
02:20, 11 June 2015 ... marked revision 0 of page Tema:lkjahsdlkjahdlkjahdlkjahsd
at
http://ca.wikipedia.beta.wmflabs.org/wiki/Especial:Registre/patrol?uselang=en
?

Filed as a bug https://phabricator.wikimedia.org/T102191

Couple of issues are still to be addressed

  • Link to oldidpage from history - edits are not marked as patrolled
  • In grouped recent changes, the clickable timestamps should also be a link to the oldid page

Note: Maybe [ Mark this post/comment/title/description/edit as patrolled ] should be used instead of
[ Mark this page as patrolled ] ?

Couple of issues are still to be addressed

  • Link to oldidpage from history - edits are not marked as patrolled
  • In grouped recent changes, the clickable timestamps should also be a link to the oldid page

The grouped RC thing was split off from this task last week as T102021: D4. Edit patrolling links in grouped RC (CA).

Done, looks great.

This is not working for me. The link which would give me access to a page with the patrol link is just plain text:

Flow-patrol.png (146×596 px, 56 KB)

For Flow posts, the link to click on is the timestamp. That takes you to the oldid page, where there's a Mark as patrolled link.

We didn't link "diff" to oldid, because it's not a diff page, and that would probably be even more confusing.

Does that work for you?

Not really, as I'll probably have to put a JS in place to scrap the page and move/copy the link to the right place (i.e. the same place were I find it on normal pages).

Hmm. Where do you click when it's a new page that should be patrolled?

I click in the page title. But notice

  • "Wikipédia:Contato/Fale com a Wikipédia" is not a new page, and a new comment to an existing page should not behave like the creation of a new page.
  • Even if it was, clicking on the page title which appears in the Watchlist (or even in the topic title, which in the screenshot is "Gravei Pagina por engano!") opens a page where there is no link to [Mark this page as patrolled].
  • The timestamp is not linked for normal pages (either on the watchlist, the [usual] recent changes, or the real time recent changes [https://github.com/Krinkle/mw-gadget-rtrc/]), so linking it on Flow comments makes the interface inconsitent

I click in the page title. But notice

  • "Wikipédia:Contato/Fale com a Wikipédia" is not a new page, and a new comment to an existing page should not behave like the creation of a new page.

A new post is closer to a new page than a diff (linking diff would just be wrong, since it doesn't go to a diff), but it's not quite the same as a new page either.

I'm open to moving that link around if it's causing a problem. Using the diff link would be odd, but we've done a lot of odd things in the name of progress. I'd like to know if other people are finding it hard... we should ask on Catalan or Hebrew.