43

Suggested edit: https://stackoverflow.com/review/suggested-edits/22896266

Screenshot:

enter image description here

How do I tell the editor that "gerhdtcfjvygkbuhnj" is not a useful comment for an edit? Reject and edit the comment? Or is it not a problem and I should simply improve the edit?

17
  • 40
    By rejecting the edit with a custom rejection message.
    – Martijn Pieters Mod
    Commented May 1, 2019 at 11:25
  • 33
    @MartijnPieters: Should we really reject an edit that (even without an explanation) looks fine just due to the explanation? I mean, it's just a formatting and spelling correction edit. No need to come up with a detailed reasoning why it's a good edit.
    – BDL
    Commented May 1, 2019 at 12:05
  • 59
    @BDL: it is indicative of a lack of care, boredom and respect for the rest of the community reviewing these edits. In this specific case, there was a larger problem and the account is now gone.
    – Martijn Pieters Mod
    Commented May 1, 2019 at 12:23
  • 4
    @MartijnPieters and what was the problem?
    – sanyassh
    Commented May 1, 2019 at 12:25
  • 7
    @Sanyash: sorry, I can't go into details on that.
    – Martijn Pieters Mod
    Commented May 1, 2019 at 12:38
  • 28
    @BDL Some edit summaries are bad enough to warrant rejection. On a more general note, I feel the importance of edit summaries is sometimes overlooked. I have seen quite a few Meta controversies that could have been avoided by a clear summary indicating the point of an edit.
    – duplode
    Commented May 1, 2019 at 13:31
  • @BDL It doesn't need to be highly detailed but one or two words describing one of the improvements made would have been enough: "Added spacing" or even just "Formatting".
    – BSMP
    Commented May 1, 2019 at 21:21
  • @MartijnPieters or BDL: Please consider putting that as an answer. Rejection seems like it would be the correct answer, and so is getting more upvotes here in comments, than the only actual answer. Commented May 1, 2019 at 21:42
  • 1
    "Edited post content" much better.
    – user4639281
    Commented May 1, 2019 at 21:44
  • 50
    Moderators usually get custom flags with "mmmmmmmmmmmmmm", which are declined with the standard decline reason "nnnnnnnnnnnnnnnnnnn" (src)
    – Bhargav Rao Mod
    Commented May 2, 2019 at 0:19
  • @BhargavRao I wonder if it's an old meme or something real.
    – iBug
    Commented May 2, 2019 at 5:02
  • 1
    And, if the edit looks OK without comment, isn't it good to click Improve Edit, and edit the comment?
    – FZs
    Commented May 2, 2019 at 5:27
  • 1
    @FZs You can't edit the edit summary. Their edit will get accepted with that edit summary and any improvement you make goes up with its own edit summary. The only way to prevent a bad edit summary from being saved is to reject the entire edit.
    – BSMP
    Commented May 2, 2019 at 6:01
  • 1
    Does the editor not realise that the comment field is optional? Perhaps they felt the edit speaks for itself and put some junk in there just to satisfy a perceived requirement for a comment?
    – AJFaraday
    Commented May 2, 2019 at 6:10
  • 2
    @AJFaraday it's only optional if you have full edit privilege (2k rep on SO). Suggested edits requires manual input for edit comment.
    – Andrew T.
    Commented May 2, 2019 at 9:29

3 Answers 3

20

In a case like this it seems like it would be a good use of pinging the editor by leaving a comment on the post to explain to them the purpose of the comment fields on a post edit. Overall the edit looks good and it appears that they just don't fully understand the purpose of the comment.

8
  • 20
    Pinging the editor works in this case only becaue the edit was approved. Before the edit is approved, you can't ping the author of a suggested edit.
    – yivi
    Commented May 1, 2019 at 12:19
  • 1
    You understand me correctly. The body of edit is rather good, the comment is the only problem, that's why I didn't reject it and asked on meta.
    – sanyassh
    Commented May 1, 2019 at 12:23
  • 6
    @yivi personally I would be a bit confused if a good edit got rejected because of a bad edit comment.
    – Joe W
    Commented May 1, 2019 at 12:33
  • 3
    I'm not saying that. But the OP saw the comment during review. At that point, leaving a comment and pinging the editor was not an option.
    – yivi
    Commented May 1, 2019 at 12:34
  • You can't pin the author of this particular edit however - they're anonymous. Commented May 2, 2019 at 0:33
  • 2
    @JackBashford There were other problems with the account and it was deleted
    – BSMP
    Commented May 2, 2019 at 6:09
  • 2
    It looks more like they did a keyboard mash because they didn't feel like writing a description than they don't know.
    – jpmc26
    Commented May 2, 2019 at 12:34
  • AFAIK, the asdlkfjldsajf and a generic "improving formatting" would be the same. The edit is too self evident an improvement of the post.
    – Braiam
    Commented May 3, 2019 at 8:24
17

Edits should only be rejected when they are not improving the post or when it is unclear why an edit has been made. When it is totally clear why an edit is being made (fixing capitalization, adding basic formatting, ...) an explanation is not necessarily required.

Note, that most of these edits contain an explanation stating something along the line of "Improved formatting", "Fixed spelling" which don't add any additional value.

The quality of the edit description comes into play only when it is unclear why an edit was necessary. For example, when content from a comment gets added to the post, then it is important to state where this content came from.

I would reject an edit that adds something new to the question when there is no explanation for it. I would accept an edit that only changes formatting no matter what the description states.

11
  • What if I put a spam link as an edit summary on an otherwise valid "improved formatting" edit suggestion?
    – Andrew T.
    Commented May 2, 2019 at 9:35
  • 6
    @AndrewT. The comment becomes the edit summary in the page history. Luckily such a link would remain inert; the UI does not parse edit summaries in any way, so the spam link is not clickable, search engines will not find it and add to the target's page rank. Still, it would indicate a problem with the account that made the suggestion, so as a moderator, I'd very much would like to hear about such behaviour. (Moderators can change edit summaries too, so if it's something offensive then we can also help clean that up).
    – Martijn Pieters Mod
    Commented May 2, 2019 at 10:20
  • 2
    Agreed. As a special note, I always make sure to reject edits which mislead, by listing something benign like "Improved formatting" as the edit summary when they also did something more significant (like alter the title or tags). Often, the major edits don't make sense in those cases, anyway.
    – jpaugh
    Commented May 2, 2019 at 21:48
  • 1
    @jpaugh That's the kneejerk reaction that doesn't help. If I improved formatting and also fixed the capitalization of i in the title, would you reject my edit?
    – Braiam
    Commented May 3, 2019 at 8:25
  • @Braiam I'm specifically referring to cases where there are major edits, but only minor edits are mentioned in the subject; like if the title is rewritten, and it says, "grammar fixes." As an attempt to somehow "hide" or distract the reviewer from a more significant edit, it's tantamount to vandalism, even if the author was merely careless rather than intentionally vandalizing.
    – jpaugh
    Commented May 3, 2019 at 19:56
  • 1
    @jpaugh what you consider a major edit? What is a minor edit? Is Bill edit major? Is my edit a minor one? Trying to draw lines in the sand isn't useful. And edit should be considered by whenever or not it objectively improves the post. Any other metric is bound to produce undesired effects.
    – Braiam
    Commented May 3, 2019 at 20:11
  • @AndrewT. I think that's a bit of a special case. I'd think quite a few decisions would differ from usual in the case of outright spam rather than just useless or unhelpful information.
    – rydwolf
    Commented May 4, 2019 at 3:17
  • @Braiam I say a "major" edit is one that changes the content/meaning of the post, rephrases it, or changes the tags. It must be semantically checked for accuracy, and read very carefully. A "minor" one improves the existing content (spellcheck, formatting, spacing, etc). It's much easier to scan quickly, for the price that small semantic changes might be overlooked. Both linked edits are major, and the edit description should mention the most dramatic change (ideally, first). No description is actually better than one which claims to be minor, but isn't, because it must be read carefully.
    – jpaugh
    Commented May 6, 2019 at 14:56
  • @jpaugh any edit change the content of the post. That's not useful. Nor its useful the others metrics. Stack Exchange wants you to objectively look at the edit and tell if it improves the post or not. Your method is very arbitrary and wouldn't benefit the future reader at all.
    – Braiam
    Commented May 6, 2019 at 14:59
  • @Braiam You're right that this rule might be hard to communicate to the whole community of editors, or even reviewers; you might also point out that I could read every edit carefully, and review far fewer of them. But I assume that some (many?) reviewers will quickly scan any edits described as non content-altering, without thinking further about it. That's why I believe edit descriptions that don't mention major edits are much more harmful than missing descriptions. It's a habit that should be discouraged.
    – jpaugh
    Commented May 6, 2019 at 15:00
  • @Braiam Please substitute "meaning" for content in my comments. Not every edit changes the meaning. I agree that it is arbitrary where the line is drawn, but I think drawing a line is still more useful than not.
    – jpaugh
    Commented May 6, 2019 at 15:03
-9

Reject the edit with a custom message explaining that a gibberish comment is "indicative of a lack of care, boredom and respect for the rest of the community reviewing these edits", per Martijn Pieters.

5
  • 2
    By custom do you mean the cause harm option? Thats the only way I can see to enter a message. Is that message then passed back to the person suggesting the edit?
    – Dale K
    Commented May 2, 2019 at 4:55
  • 1
    @DaleBurrell - Yes, that's the causes harm option. The reason you enter is then visible on the edit review. I think the user gets pinged about their edit getting rejected as well.
    – BSMP
    Commented May 2, 2019 at 5:22
  • 3
    "Causes harm" is the custom rejection reason. Confusion over that led to this proposal, and also this one.
    – Cody Gray Mod
    Commented May 2, 2019 at 6:10
  • If the edit ends up being approved, despite one rejection with a custom reason, does the editor still see that custom reject reason? Commented May 2, 2019 at 7:31
  • 3
    The editor won't see the custom reason unless they go back at a later date to check the status of the review. There is no notification to the user if an edit they make gets rejected.
    – Joe W
    Commented May 2, 2019 at 19:25

You must log in to answer this question.

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