33

A particular user has been suggesting unnecessary edits for the past few days. All that user does is removing the code that was present in the question, and then pasting it back. There is absolutely no change otherwise. And for the description of the edit, the person mentions "Improved Formatting".

The code is already very well formatted, and the user makes no additional formatting improvements at all. As far as approving/rejecting the edits is concerned, I have done whatever seems fit by looking at whether there are any additional improvements.

Should I report this user's behavior? If yes, how?

13
  • 4
    If he gets enough edits refused (hopefully if the edits are as you say, that's just a matter of time) he'll be banned from editing altogether
    – Patrice
    Commented Sep 28, 2015 at 15:20
  • 2
    I don't think you need to report him, he'll eventually get banned like @Patrice said, btw hello P, good day
    – Just Do It
    Commented Sep 28, 2015 at 15:26
  • 1
    @ryanyuyu: Ok.. The edit is not really a spam attempt then. I will look at markdowns too. Recently I've been looking only at the rendered output of the edit and that made edits seem unreasonable. I'll start considering both. Thanks. Commented Sep 28, 2015 at 15:28
  • 2
    Why is this question getting down-voted though? Commented Sep 28, 2015 at 15:56
  • 1
    That's an answer we will never be able to obtain @displayName
    – Just Do It
    Commented Sep 28, 2015 at 15:57
  • @edrodriguez: Why so? The answer is there and it's mentioned down below. Commented Sep 28, 2015 at 15:58
  • 1
    "Why is this question getting down-voted though?" I was replying to that comment @displayName
    – Just Do It
    Commented Sep 28, 2015 at 16:00
  • 3
    @displayName Edrodriquez meant we'll never be able to obtain an answer to the question "Why is this question getting down-voted though?" Unless the down-voters comment and explain why they downvoted, we have zero way of knowing. They weren't giving you the reason the question was downvoted.
    – Kendra
    Commented Sep 28, 2015 at 16:06
  • @Kendra: Oh!... :D Silly me.. :D Commented Sep 28, 2015 at 16:06
  • 1
    What I meant is, we will never be able to know the reasoning behind a down-vote. Since really no explanation is needed. It could be because someone doesn't think its a good question or just cause it's Monday. That's why I said we won't know the answer as to why it's down-voted. Since -for me- it truly seems like a legitimate question here on meta @displayName
    – Just Do It
    Commented Sep 28, 2015 at 16:07
  • 6
    Obligatory downvote reason: Tim lost his keys
    – ryanyuyu
    Commented Sep 28, 2015 at 16:42
  • @pnuts: That which has happened to you in that question of yours, is really bad. Already 34 down-votes and then 12 more... It's like kicking a man when he's down. :D :) Commented Sep 29, 2015 at 14:37
  • 2
    @pnuts - I got kicked for asking the same question about a question of mine. People then seemed to purposely go to back to the original question and add more downvotes. Then whoever was posting on meta started beating me up for having asked, saying I should have just moused over the arrows to find out. It really turned me off. Not just here, but gave me a poor feeling about people... Oh well. At least I got some good programming help. And my edits of other questions were deemed useful (I worked hard on them to figure out the cryptic questions, so I was hoping they would.)
    – curls
    Commented Sep 30, 2015 at 5:46

3 Answers 3

33

You mean like this suggestion? If you click in the markdown diff, you'll notice the suggestions add an HTML comment for syntax highlighting. That by itself is not the most useful thing in the world, but it is potentially a minor improvement. I personally think it's not enough of an improvement; I think it fits the "No improvement/superfluous" rejection reason.

However it's not malicious, and it seems to be a (lazy) honest attempt at improving stuff. As other comments have noted, if that user keeps making these poor suggestions, (s)he'll get edit banned in their own. Well, as long as reviewers are doing their jobs. Hopefully, after a few rejections that user will figure out what edits are appropriate on their own.


Addendum in case the user does not improve the quality of their edits or they are not banned. If all of the following are true:

  • You continue to see these suggestions because reviewers keep approving them
  • You are absolutely sure that the user is spamming the same kind of suggestions (serial minor edits)
  • Those suggestions should all have been rejected

you can follow bluefeet's advice and flag one of the posts with an approved suggested edit (emphasis mine):

When you flag for moderator attention, just include some details about the issue. I'd suggest including a few links to some of the suggested edits and explain that the user suggested many incorrect edits and reviewers approved them. Moderators can ban users from suggesting edits, and they can ban reviewers who approved them.

Since you mention that the user has done this "for the past few days" it might be appropriate to flag their behavior. In general when you first encounter these kinds of edits, you should hesitate to report the user's behavior.

28
  • 9
    I am not sure how well the reviewers are doing their jobs. The user has +126 rep so far today just adding language tags to code. Commented Sep 28, 2015 at 15:33
  • 6
    @displayName it means they have had 63 of these same edits approved so far today. every time you get an approved suggest edit you get +2 reputation. Commented Sep 28, 2015 at 15:38
  • 1
    @displayName My issue is that if all of the edits they are doing is just adding the language tag to the code then they are just rep farming and should be stopped. IMHO just added language formatting is not enough of an edit. Commented Sep 28, 2015 at 15:42
  • @NathanOliver: I agree with you about that. Commented Sep 28, 2015 at 15:46
  • 9
    Just an addition: I am usually generous for approving edits when someone makes some useful contribution. In my opinion, we shouldn't worry whether the user is rep-farming or not. We should treat each edit at its merit. Even if they are rep-farming but providing "useful" edits, they deserve that +2. Commented Sep 28, 2015 at 15:48
  • 5
    Do note that mods now can ban a user from suggesting edits. I'll try to find the link to that on MSE, but that's been in the past couple months here. Edit: Found it!
    – Kendra
    Commented Sep 28, 2015 at 16:03
  • 1
    @Kendra nice to know. We'll have to get bluefeet to update her answer in the linked post then.
    – ryanyuyu
    Commented Sep 28, 2015 at 16:04
  • 8
    Please note that adding language syntax coloring is not how you should edit posts. If the question is properly tagged with the specific language, the site will automatically pick correct syntax coloring. So the correct way to edit is to add the missing language tag to the question.
    – Lundin
    Commented Sep 29, 2015 at 6:38
  • 5
    This. Look at the Markdown before complaining about an edit. Commented Sep 29, 2015 at 12:35
  • 1
    I am unaware of any evidence that mods want to get flagged because of serial minor edits. The changes to the edit review queue have all been towards "allow even the most ridiculous of small improvements to be approved" despite general meta consensus the other way. Commented Sep 29, 2015 at 17:58
  • @blujay What are you responding to? Commented Sep 29, 2015 at 23:37
  • Is there a way to communicate directly to the user? After all, they may not realize and an honest and clear and kind comment to them, may solve the problem. (A hurtful or rude one won't fix anything of course.)
    – curls
    Commented Sep 30, 2015 at 5:49
  • 1
    @curls: @ notifications on the question will reach the user. They aren't in the suggested list, but if you actually type their username, they'll get notified. Editors are on the notification list for the posts they edit. Commented Sep 30, 2015 at 6:22
  • TJCrowder - can you explain that more clearly? I've noticed I'm doing something not right. Namely that I use @ so and so and the name and @ disappear. Yet the comments box says to use @ and reference the person you are commenting to. Also, does the user's name have to be typed exactly as they typed it? I don't know what you mean by well, by any of your phrases. I'm too new. What's not on the suggested list? Where's that list? What editors? The person gets notification if they've edited a question but the edit was rejected but a comment is added to the question to tell them why?
    – curls
    Commented Sep 30, 2015 at 6:32
  • 1
    @curls: (Your second one pinged me) When you see the @ and name disappear, you're running into this "feature": Atwood decided that he knows better than you whether it's appropriate social behavior to include that when you're addressing the OP (and Shog9 has chosen to keep it). The rules for whether it gets removed are arbitrary and relate to whether there are comments from others in-between. When you're not addressing the OP, it will remain. Commented Sep 30, 2015 at 16:11
25

For what it is worth, has had a bit of a problem with regard to syntax highlighting on Stack Overflow. See my post about it here: What's happening to syntax highlighting for [css] questions? So for anyone wondering if this specific issue could be solved by changing the default language hint to lang-css, the answer is no because it would break syntax highlighting for a completely different set of questions.

This is not the first time a user has made a series of suggested edits solely to add the lang-css hint, and that is the reason why it happens. While I normally wouldn't mind these edits, it is pretty disappointing that this particular user couldn't be bothered to write a meaningful description, even to copy and paste — they just stuck with the "improved formatting" placeholder. I feel like I would reject these edits on grounds that the editor simply isn't putting effort into their edits for this reason.

But in general, as ryanyuyu has said, if there are more areas that can be improved that were clearly ignored by the editor then the edits should be summarily rejected as not fixing enough outstanding issues, because that too signals a lack of effort on the editor's part.

6
  • 4
    That is right. If the editor would leave one good hint in the description, I wouldn't be asking this question at all. I always read the description when I don't understand the purpose of an edit. Commented Sep 28, 2015 at 16:11
  • 1
    One problem I noticed with things like CSS is that there is only one "default highlight language" per post. So if you have some HTML and some CSS in one post, both code blocks will be highlighted using the same rules.
    – Mr Lister
    Commented Sep 28, 2015 at 16:37
  • @MrLister If you have multiple languages, don't use default highlight. Instead, set the language manually for each code block.
    – Oriol
    Commented Sep 28, 2015 at 16:44
  • 2
    @Mr Lister: "default" is more like "auto" - the syntax highlighter tries to figure out what language a specific code block is in and highlights it accordingly. The real problem is that the highlighter can't distinguish a CSS snippet. And that is why CSS requires a language hint, be it associated with the tag, or with an explicit comment in the Markdown source.
    – BoltClock Mod
    Commented Sep 28, 2015 at 16:47
  • @Oriol That's why I like stack-snippets so much! Even though they insert the code into the post the wrong way round. But my comment was more a reflection on existing posts, where if the OP doesn't know about the <!-- language --> trick, al but one of the code blocks will come out wrong. That's maybe what triggered this specific person's editing spree.
    – Mr Lister
    Commented Sep 28, 2015 at 16:55
  • 2
    @Mr Lister: You mean the way they insert CSS before HTML? It makes sense if you think about it - stylesheets go into the head element, which precedes body, and the HTML pane represents the contents of the body element only.
    – BoltClock Mod
    Commented Sep 28, 2015 at 16:56
4

Where syntax highlighting is absent but would be an improvement then adding an appropriate HTML comment is good, so should be encouraged.

I have quite often edited posts for that reason and no other. I have seen others have done so too. What I have not seen – as far as I can recall, so far – is complaints about that. Which is significant because someone at least on SO has managed to complain about practically everything else I have done here.

Editing is not the issue, rather suggesting edits - because these come to the attention of reviewers. If You continue to see these suggestions because reviewers keep approving them there is no issue. The Suggested Edits review queue is often down to 0 - the system is clearly handling the volume adequately – and the improvements (HTML comments) have been made. No cause to flag.

If other problems with the post are not corrected at the same time, that also is no issue. At least the post is better than it was. There is (almost) always the option of further edits/edit suggestions if further tidying is appropriate.

If there are no other problems with the post, that also is no issue. Job done.

If, as a reviewer, you don’t want to see incomplete edit suggestions that I understand. But it is quite simple really, hit Skip. And if that is too onerous, don’t review – there are plenty of others who will. And unless others do, skipping has the same effect as rejecting anyway.

Better of course is to click on Improve Edit - win-win. Suggester has saved you adding the syntax highlighting element and all improvements are made. Job done. This is consistent with flagging, ie to bring to the notice of others what needs attention but cannot be unilaterally fixed by the user. Flagging, used properly is also good.

If you however can’t be bothered to see things through to completion you should not reject an improvement just because the person suggesting an edit also was not comprehensive. That leaves the post in a worse state than it might have become and sends the wrong message about what does and does not count as an improvement.

But your first question is “Should I report this user's behavior?”, the answer to which is NO (so your second question irrelevant).

2
  • 2
    I asked this question not because the user was doing incomplete edits or because it was onerous task for me. It seemed like the user was copy-pasting the code which I now understand was actually the language hint to help with code-color (It isn't obvious in the Rendered Output view). Moreover the user provided no hint in the description about his/her edit. It really seemed like an attempt to make undue rep points and I condoned this behavior for 3-4 days. Commented Oct 1, 2015 at 2:55
  • Ensuring that code has the correct syntax highlighting is a good thing, but IMO it's too trivial to require the time of 4 people, so users with under 2k rep should be discouraged from making such edits, unless they are also improving the post in other ways. So choosing "Improve Edit" is a great idea - it means that no more reviewers will waste time on it and (AFAIK) the original editor won't get the 2 points, discouraging them from making further trivial edits. But I guess it may be a good idea to ping the original editor with a brief comment saying why such edits are discouraged.
    – PM 2Ring
    Commented Oct 6, 2015 at 11:25

You must log in to answer this question.

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