Page MenuHomePhabricator

Make TopFiveReviewers configurable
Closed, ResolvedPublic

Description

It is possible to make the TopFiveReviewers on Special:ValidationStatistics configurable? When there is a config it is possible to increase or decrease the numbers auf users. Maybe allow a value less or equal to 0 to disable this list and do not show the relevant section on ValidationStatistics.

Thanks.


Version: unspecified
Severity: enhancement

Details

Reference
bz26268

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:18 PM
bzimport set Reference to bz26268.
bzimport added a subscriber: Unknown Object (MLST).

*** Bug 25775 has been marked as a duplicate of this bug. ***

Save for after the Pending Changes fork

(In reply to comment #2)

Save for after the Pending Changes fork

There's going to be a Pending Changes fork?

(In reply to comment #3)

(In reply to comment #2)

Save for after the Pending Changes fork

There's going to be a Pending Changes fork?

Yes, and it was mentioned back over the summer (I'm pretty sure I've talked about it on IRC before).

FlaggedRevs is really two extensions: traditional FlaggedRevs and Pending Changes. A good portion of the code is disjoint. From a code maintenance perspective, it makes the most sense in the long run to split off the Pending Changes feature into its own extension.

(In reply to comment #3)

There's going to be a Pending Changes fork?

Latest version of the roadmap talks about this some more:
http://www.mediawiki.org/wiki/Pending_Changes_enwiki_trial/Roadmap

The fork is contingent on Pending Changes being kept on enwiki. If English Wikipedia drops it, we'll probably drop development on the configuration and focus any dev activity on the traditional FlaggedRevs config.

(In reply to comment #0)

It is possible to make the TopFiveReviewers on Special:ValidationStatistics
configurable? When there is a config it is possible to increase or decrease the
numbers auf users. Maybe allow a value less or equal to 0 to disable this list
and do not show the relevant section on ValidationStatistics.

Thanks.

Is the motivation here to increase or decrease/disable it? Maybe it could be hard coded to 10 (or something) if it's the former (increase)?

Perhaps a CSS class could be used to allow for disabling it without needed more config cruft.

(In reply to comment #6)

Perhaps a CSS class could be used to allow for disabling it without needed more
config cruft.

CSS to "disable" stuff is bad, it can easily be reshown then, if they want it disabled, it should be able to be done.

I lean towards either keeping it fixed or getting rid of it. I'm tired of all these silly config vars running around.