7

I got this review, but I'm confused by the whole thing. The message says:

Proposing to remove topic, as it's literally entirely copied and pasted from its GitHub page.

OK, maybe it's a delete? Then below is the text:

deletes topic

The tense and the capitalisation seems a bit off. Perhaps it is a delete? However, the body text has a salmon-pink background and no strike-through of the text. The buttons say Approve Change and Reject Change, but it's supposed to be a deletion. Has someone added the text but another reviewer has asked for deletion, so should I reject too, or has someone come along later and voted to delete, so I should approve the deletion change?

I'm very confused!

6
  • 3
    Disclaimer: I submitted that deletion. That aside, you're not the only one who got confused with a deletion today. Earlier I had flagged an example in that topic before I investigated the topic further, and somebody went ahead and submitted it for deletion. While it got deleted, one of the reviewers rejected it but commented that they agreed the example looked like an advertisement. So I'm going to go ahead and say that deleting stuff in Documentation needs some tweaking.
    – cteski
    Feb 17, 2017 at 4:14
  • 1
    I see that other one has the comment/header "removed 1 example" - is this user text? Auto-generated? The flag and the "Handles improvement requests" text in the pink also suggests someone added the body text as a response to the improvement request, then the reviewer felt it wasn't up to scratch, so "Reject" is the correct answer.
    – Ken Y-N
    Feb 17, 2017 at 6:40
  • 2
    Well, it is not going to hurt usability if that button text would say "Approve deletion". Feb 17, 2017 at 9:09
  • I happened upon another case of a reviewer rejecting a deletion, but they provided what looks to be ways to improve the example as a rejection reason.
    – cteski
    Feb 17, 2017 at 13:29
  • 2
    I'm going to bring this up with the team. It does seem like the prompts and actions could be more clear. Maybe the automated prompt should read something like "This proposed change deletes the topic." And also make the change @HansPassant suggests. Feb 17, 2017 at 18:43
  • I removed the [status-review] tag from this post because Documentation no longer exists.
    – Slate StaffMod
    Jan 24 at 20:52

1 Answer 1

9

This is just a delete request. No more than that.
A deletion is also a "change", so "Reject/Approve Change" makes sense in that way.

The "deletes topic" is literally what this change does.

To be honest, I don't see how that request can be interpreted as anything other than a deletion.

Not the answer you're looking for? Browse other questions tagged .