27

I receive notifications of ‘suggested edits’ to my writing with cosmetic changes of unknown purpose, such as this one:

Screenshot of suggested edit

A common pattern is that an edit looks like it tries to make my answer or question conform to some formatting guidelines I’m not aware of. Invariably I can’t reject it because by the time I open it it’s already been unanimously approved.

What’s going on with these edits? Are there any formatting guidelines that I should follow but am not?

13
  • 28
    This particular edit was completely pointless and should not have been suggested, let alone accepted. There was zero point in italicizing those spots. Looks to me like someone just editing to edit.
    – Kendra
    Aug 13, 2015 at 18:07
  • 5
    And the review stats of those reviewers says it all.
    – Bart
    Aug 13, 2015 at 18:10
  • 6
    Looks like they improved your post. Move on.
    – GEOCHET
    Aug 13, 2015 at 18:22
  • 10
    The random italics is really bad. The line breaks in the css code are meh (but superfluous). That's a "no improvement whatsoever" edit.
    – ryanyuyu
    Aug 13, 2015 at 18:24
  • 6
    Are you unable to roll back edits once they've been approved?
    – BSMP
    Aug 13, 2015 at 18:28
  • 2
    @BSMP, thanks for the reminder. Forgot about that functionality for some reason. Aug 13, 2015 at 18:34
  • 7
    The edit made your CSS easier to read (for me, anyway). I suspect that was the inspiration for the edit rather than some guideline (that doesn't exist, as far as I know).
    – Cypher
    Aug 13, 2015 at 18:55
  • 2
    @Cypher - I get the motive for the line breaks in the CSS, but what's with the italics? It's not a research paper; there's no reason to use ALA or MLA style citation for just someone's name.
    – BSMP
    Aug 13, 2015 at 19:08
  • 1
    @BSMP I agree. The italics do not add anything. But that's our opinion, and you know what they say about opinions. :)
    – Cypher
    Aug 13, 2015 at 19:11
  • 15
    @Cypher you shouldn't force them on other people's posts while farming rep? :)
    – resueman
    Aug 13, 2015 at 19:34
  • 2
    Yes, the line breaks make a cleaner read. The italics? Maybe a minimal impact way of reaching the minimum number of changes to allow the edit. Necessary edit? Not really.
    – OhBeWise
    Aug 13, 2015 at 22:15
  • 2
    I absolutely expected that a whitespace-only edit will not be accepted as longer than 6 chars. But it was, when I just tried, by suggesting an edit indenting the css code again. (Maybe he expected the same, and made up some irrelevant extra changes to comply with the form? ) Aug 14, 2015 at 3:57
  • Of interest to read: meta.stackoverflow.com/a/302450/424903
    – Gimby
    Aug 14, 2015 at 9:36

2 Answers 2

26

I personally don't see anything wrong with the format of your post. I think your post is the victim of two separate but connected actions.

First you get a user who is doing suggested edits to get reputation and/or badges. Then you have people in the review queue that are rubber stamping everything (robo reviews) and unfortunately they are processing these edits.

Looking at the user who made the edit they seem to be getting a lot of reputation from edits and when I took a casual glance at some of them I would have rejected them. If you look at the review history of the first two users they have no rejects which is suspect to me as there should be an occasional edit rejection.

I think it might be appropriate for a mod to take a look and a timeout might be in order for all.

6
  • 10
    I would caution that the reviewer's stats on the review page might be really inaccurate. But in this case, just from browsing their profile's activity tab, I think you're right. For example, Reviewer 2
    – ryanyuyu
    Aug 13, 2015 at 18:22
  • 1
    Yes I reviewed their review history from their profile. I know not to trust the stats served up in the queue. Aug 13, 2015 at 18:23
  • 1
    I figured you did, but maybe others viewing this post don't know that. I didn't really want to edit this into your answer, but I though it was an important footnote.
    – ryanyuyu
    Aug 13, 2015 at 18:25
  • 28
    I've "communicated" with a couple of the reviewers.
    – Brad Larson Mod
    Aug 13, 2015 at 19:10
  • 4
    @BradLarson I am truly curious about what sinister action your scary quotation marks imply. Did you communicate by means of a horse's head in each of their beds?
    – Mark Amery
    Aug 15, 2015 at 17:49
  • 1
    You are correct ! Aug 16, 2015 at 6:38
4

About the edit in whole

It would be okay if a 2k user improved formatting, even if it's a very minor improvement. But thousands of 2k users have read the text and no one ever bothered about formatting. So there was no urgent need for such edit. For a reviewed edit this is just not enough.

About the details

  • @username in italic — NO, usernames should stay in regular text.
  • SoftwareName in italic — NO, just the same, no need to emphasize
  • HTML/CSS properties1 one on each line — YES, it's much easier to read this way.

(1) Don't know the right name, correct me if it's wrong.

2
  • 4
    Normally I'd agree that CSS properties should be on their own lines, but in this case, each line had related properties, so the readability hit really wasn't that bad. Making even that change was pretty dubious, and given that the rest of the post showed reasonable quality, not really justifiable. Aug 14, 2015 at 7:43
  • 1
    @NathanTuggy: I totally agree that it's not justifiable, especially for a reviewed edit. Added a note about it in my answer. Also, now I see that properties are grouped in lines, but it wasn't obvious to me because I don't specialize in CSS. Aug 14, 2015 at 9:25

You must log in to answer this question.

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