Last modified: 2012-05-02 00:49:57 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T38101, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 36101 - "mark as reviewed"
"mark as reviewed"
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
PageCuration (Other open bugs)
unspecified
All All
: Normal minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 36110
  Show dependency treegraph
 
Reported: 2012-04-19 16:48 UTC by Oliver Keyes
Modified: 2012-05-02 00:49 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Oliver Keyes 2012-04-19 16:48:04 UTC
The "mark as reviewed" button appears at the *top* of articles rather than the bottom. The language is also confusing; it should be either "mark as triaged" (to keep in with the other language we're using) or "mark as patrolled" (to remain in keeping with the language editors are used to).
Comment 1 Fabrice Florin 2012-04-19 23:08:34 UTC
I agree with all these points.

We are going to have to make a decision pretty soon as to whether this feature should be called 'reviewed', 'triaged' or 'patrolled'.

I favor "reviewed" (a friendlier term, IMHO) -- but could also go back to "patrolled" (more hostile, but may be our best bet, if the community insists on it). 

I don't favor "triaged" because it doesn't describe the action that's being taken very effectively -- and the noun "triager" is not as appropriate as"reviewer" or "patroller" for the user of that feature.
Comment 2 Krinkle 2012-04-19 23:35:56 UTC
Just my 2c here, I'd recommend using "patrol" (and derived phrases like patrolling,  patrolled, patroller, etc.) because the core feature that this extension enhances is both internally and in the interface referred to as patrolling:
* user right "patrol"
* database fields *_patrolled
* API actions, filters and properties
* SpecialPage filters hide/show patrolled
and the documentation all over the wikis for "edit patrol" RCPatrol and "new page patrol" NPPatrol.

Since this extension didn't introduce the feature, but introduces an (awesome) new interface to that existing feature, it would be confusing to call it something else.


I agree with Fabrice about the ambiguity in the word "triage". Also, as a foreigner, I have to say that until I got deeper into Wikimedia, I had never heard the word "triage" before. So from a global point of view I think "patroller" is more integrated and known.


And although "review" is at least as descriptive and known as "patrol", the word "review" somewhat conflicts with the terminology for FlaggedRevs (which, contrary to mediawiki patrolling, does not check individual changes but checks the resulting page as a whole) so best not to confuse with those "reviewers".
Comment 3 Ryan Kaldari 2012-05-02 00:49:47 UTC
After endless hours of discussion, we finally settled on retaining 'review' and changing Special:PageTriage to Special:NewPagesFeed.

This has been implemented in https://gerrit.wikimedia.org/r/#change,6361.

I'm not sure there's actually any conflict between the new page 'reviewing' and revision 'reviewing' (in the sense of FlaggedRevs). They are similar processes and will probably eventually have similar interfaces to deal with them. I would support eventually merging 'autopatrol' with 'autoreview' and merging 'patrol' with 'review' in the userrights if the community would support it. There's really no reason to have these separated.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links