Page MenuHomePhabricator

Provide a list of predefined reasons in the protection form
Closed, ResolvedPublic

Description

Author: achates

Description:
Please implement a dropdown box with predefined reasons for deleting and protecting pages similar to the Dropdown-Box with predefined Block-Reasons. Admins sometimes give somewhat sloppy Deletion-Reasons, which may upset the Author of the deleted Page.


Version: unspecified
Severity: enhancement

Details

Reference
bz10799

Event Timeline

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

robchur wrote:

Refactoring to handle just the protection issue; bug 9633 handles predefined deletion reasons. I'm not sure we need such a list for protection, since a lot of protection issues will require a more thorough explanation, so I'm less inclined to think it a good idea to support a bunch of boilerplates.

I suspect a lot of cases are going to be fairly generic -- 'protecting for X hours in response to edit war' etc.

It might be suitable to allow a combination of both a drop-down boilerplate and a custom comment to clarify. Or that might be crazy talk, dunno. :D

mike.lifeguard+bugs wrote:

Having both a combo box and text box is exactly what we have currently for deletion and blocking; why should this be any different?

I think the messages for "other expiry" and "other/additional reason" should be split into two messages for the labels and two messages for the options. for consistency with other labels, the labels should have colons and be capitalised. While the options should not have colons, and at least the other expiry time option should not be capitalised.

(In reply to comment #5)

I think the messages for "other expiry" and "other/additional reason" should be
split into two messages for the labels and two messages for the options. for
consistency with other labels, the labels should have colons and be
capitalised. While the options should not have colons, and at least the other
expiry time option should not be capitalised.

Didn't even notice. Done in r40561.