41

I'm hoping to get rid of the "too minor" edit rejection reason, in favor of a more direct way of indicating edits that fail to significantly improve a post. We're also fixing to warn editors when their edits are rejected. Between the two of these changes, I'm thinking the other rejection reasons are gonna get a whole lot more scrutiny in the near future...

So it seems like this would be a good time to revisit the guidance they provide, both to reviewers and editors.

Here are the existing rejection reasons in order from most-used to least-used, and my commentary on them:

  • too minor - This edit is too minor; suggested edits should be substantive improvements addressing multiple issues in the post.

As previously mentioned, this should be dropped as an explicit rejection choice, in favor of being implicitly chosen by the system in response to the submission of an alternate edit.

  • invalid edit - This edit is incorrect or an attempt to reply to or comment on the existing post.

This is a great reason hobbled by a terrible name - it is trivially easy to find suggested edits rejected for this reason that are neither incorrect nor an attempt to reply or comment. In practice, folks appear to use this for edits that add information they don't understand; while it is possible that some of these should be rejected for other reasons, the guidance provided by this reason to the editor is then useless. This reason should be restricted to attempts to reply to the post's author, and named accordingly.

  • radical change - This edit changes too much in the original post; the original meaning or intent of the post would be lost.

Changing the original meaning or intent is a great reason to reject an edit; changing "too much" is terrible guidance - as subjective as "too minor" if not more so. Usage reflects this, with the edits rejected for this reason ranging from those that completely replace the question/answer with one entirely different, to those that make comprehensive grammar corrections. Changing the meaning/intent is the cardinal sin here - the name and guidance should focus on this.

  • vandalism - This edit introduces spam, defaces the post in some way, or is otherwise inappropriate.

Another excellent reason ruined by making it too broad. Drop the ending "or is otherwise inappropriate".

  • excerpt not helpful - This edit does not follow any of our tag wiki guidelines and is unlikely to help instruct future visitors in the appropriate use of the tag.

The only problem with this reason (which only appears for tag wiki excerpt edits) is that you have to read a blog post to figure out what sort of "helpfulness" it's talking about. Robert Cartaino suggested an alternate wording that addresses this - we should use it.

  • wiki not helpful - This edit does not follow any of our tag wiki guidelines and is unlikely to help instruct future visitors in the appropriate use of the tag.

Again, if I have to read a blog post to remember what sort of stuff I'm looking for in a wiki edit then I'm probably going to get it wrong. Needs some short examples of what's good/bad in a tag wiki.

  • copied content - This edit plagiarizes content from an external source without proper attribution.

This is the least-used rejection reason, which is a shame because it should be getting used all the time on tag wiki edits! In fact, this is probably more important for tag wiki edits than the one directly above. Robert has some good suggestions for improvement here too - we should use them.

I'm not going to suggest my own changes to these just yet (apart from including Robert's); first, I'd like to hear your ideas - and your opinions on whether or not my criticisms merit changes at all. So, thoughts?

###Related: What are the rejection reasons for suggested edits?

13
  • 9
    "copied content" gets used less because by the time you check and choose it it's already been approved
    – random
    Aug 21, 2014 at 0:05
  • 2
    @random ... by people that don't even check.
    – Braiam
    Aug 21, 2014 at 0:16
  • 1
    You mention “in favor of being implicitly chosen by the system”; does the system currently have a set of criteria for this (other than too short), or would you be adding a new facility for it to do so, or would you just be enhancing the auto-detector in some fashion?
    – tchrist
    Aug 21, 2014 at 0:18
  • 1
    @random The submitter and approvers of plagiarism should have a reputation penalty, after mixed plagiarism/approve go to a mod queue to confirm.
    – bjb568
    Aug 21, 2014 at 0:23
  • @bjb568 Approvers of plagiarism should get a penalty? Can’t they plead ignorance of the law?
    – tchrist
    Aug 21, 2014 at 3:34
  • 5
    @tchtist Ignorance is the problem we're trying to solve here...
    – bjb568
    Aug 21, 2014 at 4:17
  • 1
    I don't agree with the way you're talking about replacing "too minor". Sometimes I consider the original post good enough, the only problem is minor spelling or grammar problems. I don't see the need for an alternate edit, I think the question can be left as is. That's why I currently reject the edit as too minor. Have I been on the Internet too long, so I've become accustomed to crappy writing?
    – Barmar
    Aug 21, 2014 at 19:42
  • 1
    I suspect most of us have, @Barmar.
    – Shog9
    Aug 21, 2014 at 19:43
  • This is live, right?
    – Braiam
    Sep 8, 2014 at 22:42
  • 1
    Elsewhere you made a comment about "Under X characters, where X is between 6 and 600?" --- It is still necessary for non-2k users to make edits of a 'substantial' nature (see for example, fixing a typo of 'now' to 'know' and 'it's' to 'its' programmers.stackexchange.com/review/suggested-edits/76440 ). It is disappointing to need to approve such other rewordings to actually see the improvement in the post.
    – user213963
    Sep 9, 2014 at 3:30
  • 1
    @tchrist Ignorance of the law is not an excuse.
    – TylerH
    Sep 9, 2014 at 13:22
  • 2
    I read this post and I still don't see a reason for removing the "Too Minor" rejection option, other than "I want to remove it", which is no reason at all. I also disagree with all the other suggested changes to the appropriate use of each of the remaining options, without an array of alternatives already put forward to replace the lost "choices" that were used more than any of the choices that have remained. I do not like such unanticipated, unilateral action.
    – TylerH
    Sep 9, 2014 at 13:25
  • 1
    The rationale is given here, @TylerH - I'm not interested in debating it here, as it is part of a larger problem. The rest of the rejection reasons have some issues as well, but I do believe they can be salvaged / augmented / replaced.
    – Shog9
    Sep 9, 2014 at 14:31

6 Answers 6

28

Big thanks to everyone who offered feedback here! I've reviewed it along with a heapin' helpin' of custom rejection reasons, and the following reasons are now live on all sites:

  • spam or vandalism (all post types)

    This edit defaces the post in order to promote a product or service, or is deliberately destructive.

  • no improvement whatsoever (all post types)

    This edit fails to make the post even a little bit easier to read, easier to find, more accurate or more accessible. Changes are either completely superfluous or actively harm readability.

  • irrelevant tags (questions only)

    This edit introduces tags that do not help to define the topic of the question. Tags should help to describe what the question is about, not just what it contains.

  • clearly conflicts with author's intent (Q/A only)

    This edit deviates from the original intent of the post. Even edits that must make drastic changes should strive to preserve the goals of the post's owner.

  • attempt to reply (Q/A only)

This edit was intended to address the author of the post and makes no sense as an edit. It should have been written as a comment or an answer.

  • copied content (Tag wikis / excerpts only)

    This edit copies a significant amount of content from an external source. Generic descriptions such as encyclopedia articles and ad copy do not provide useful guidance; try creating something useful to this community specifically, and be sure to attribute the original author. See: How to reference material written by others.

  • lacks usage guidance (tag excerpts only)

    Simply defining what a [tag] is rarely helps those using it unless the tag's name itself is ambiguous. Excerpts should describe why and when a tag should be used.

  • circular tag definition (tag excerpts only)

    Tag excerpts amounting to, "[tag] is for questions about [tag]" are pointless and usually rejected. Excerpts should describe why and when a tag would be used.

  • causes harm (all post types, replaces "custom" - prompts for free-text input)
    Describe how this edit would make the post worse.

What this will look like in most cases:

###Notes:

We're walking a fine line here between documenting what is expected of edits and trying to enumerate badness. I've attempted to focus the new reasons on the worst problems observed in suggested edits, while re-positioning "custom" as a more obvious choice in situations not listed. I originally included another reason, "polishes excrement" - but dropped it after realizing that it asked reviewers to determine the value not of the edit but of the post being edited. While I do believe these edits should be rejected in many cases, the reason for declining them resists a generic description - therefore if it does not fall into one of the other categories listed, my hope is that folks utilize the "causes harm" field to indicate why such an edit would be counter-productive.

Special thanks to Gilles, bjb568 and Care Bear for their suggestions - while I've used none of them verbatim, I've incorporated ideas from all of them.

16
  • No love for tag wiki/excerpt? :(
    – Braiam
    Sep 24, 2014 at 23:01
  • 1
    The last three reasons listed are tag wiki / excerpt reasons. I just didn't fake a screenshot of those, since they're not shown in most cases.
    – Shog9
    Sep 24, 2014 at 23:02
  • I don't understand what “disrespects the author” is supposed to include. Is that vandalism like inserting “the author is a poopyhead”, and does it train the spam filter (I'd prefer that lumped with spam like it is now)? Is it supposed to include all edits that introduce a factual error, but would be fine (not too major) if they were factually correct? I think this option can be removed: broaden “spam” back to “vandalism”, put factually incorrect under “causes harm”. “Circular tag definition” is a subset of “does not improve the post in any way”. Sep 24, 2014 at 23:09
  • Under “attempt to reply”, for answers, another common case is edits that should have been an edit to the question. Sep 24, 2014 at 23:11
  • "circular tag definition" tends to come up more on initial edits to tag excerpts, @Gilles - I guess in a sense you're not even improving on nothing, but that's kind of a leap.
    – Shog9
    Sep 24, 2014 at 23:18
  • 2
    Like @Gilles, I'm uncertain about the "disrespects the author". I think it means things like meta-sarcasm and repurposing a question in a way that asks something besides what the author wanted to know. I think the radical change descriptions Gilles suggested capture the problem better. Sep 24, 2014 at 23:31
  • 3
    Open to suggestions on the name, but absolutely not using "radical change" for anything - it sends entirely the wrong idea, unfortunately. "Radical" - some might prefer the term "heroic" - edits are sometimes not just ok, but desperately needed; what's not ok is disrespectful edits.
    – Shog9
    Sep 24, 2014 at 23:33
  • 2
    Based on a discussion we just had internally, I updated the "disrespects the author" reason to "clearly conflicts with the author's intent". Curious if @Gilles and others agree with the change.
    – Laura
    Sep 25, 2014 at 16:36
  • @Laura I find it a lot clearer. I'm still unsure how to classify incorrect edits (that introduce factual errors or bad formatting), which is my biggest gripe with the current options. Sep 25, 2014 at 17:00
  • "Does not improve the post in any way" and "clearly conflicts with author's intent" - I like both of these reasons; I think they would be improved by dropping "in any way" and "clearly" from the names. I guess the intent is for names to be descriptive of the close reasons but surely that's the job of the description. Limiting the name to what will clearly identify the item and no more communicates more strongly that the description is important and necessary for understanding.
    – Air
    Sep 25, 2014 at 18:51
  • I know it seems nitpicky but I bring it up because of the number of questions I've seen on various site metas about moderation items that are quickly answered if only they had carefully read the item's description. Apparently they think the name itself exhaustively describes the item; this should be discouraged. @Laura (whoops, screwed up the notifications)
    – Air
    Sep 25, 2014 at 18:54
  • 1
    @Gilles I would say that a factual error "causes harm" to a post and incorrect formatting at the very least "does not improve" it. No?
    – Air
    Sep 25, 2014 at 19:04
  • Shortened the former, @Air - that said, folks will cheerfully misinterpret even the most terse names rather than read descriptions, so I don't think we're really buying anything by making the names ambiguous - those not otherwise inclined to read the description will just misapply them in even more scenarios.
    – Shog9
    Sep 25, 2014 at 21:12
  • @Shog9 Of course you would say that, as one of a number of shapeless horrors.
    – Air
    Sep 25, 2014 at 21:24
  • 1
    Not necessarily, @Jan. Generally, you'd want some sort of excerpt for the top tags on a site - if you can't do better than a definition, so be it. But all too often, folks start just filling in definitions for tags without bothering to even consider the usage - this is particularly a problem on new sites, which is where this guidance comes from.
    – Shog9
    Oct 20, 2014 at 18:01
14

This post is of very low quality and should be deleted. Please only edit posts that can be salvaged.

4
  • 4
    I'd prefer "salvaged" rather than "saved". Less overloaded meaning. Sep 9, 2014 at 21:06
  • @CodyGray Fair enough.
    – bjb568
    Sep 9, 2014 at 23:44
  • Should be called "Don't polish turds"
    – apaul
    Sep 25, 2014 at 23:05
  • Any chance of turning this answer into a feature request? It's still a very good idea.
    – BSMP
    May 19, 2017 at 14:41
13

Yay, at last!

Questions and answers on the one hand, and tag wikis (bodies and excerpts) on the other hand, need different sets of reasons.

The guidance shown to reviewers and the guidance shown to the author of the suggestion needn't be the same. I'll propose wording for both.

For questions and answers

  • attempt to reply — This edit an attempt to reply to or comment on the existing post.

[question] To request some clarification or improvement from the author of the question, leave a comment under the question. If you want to provide a solution, use the answer box at the bottom of the page.

[answer] To request some clarification from the author or provide constructive criticism, leave a comment under the answer. If you want to provide an alternative solution, use the answer box at the bottom of the page.

Drop the “is incorrect” bit, which conflated two completely separate cases (attempts to reply, and modifying the post in a way that is factually incorrect). The guidance should be adjusted if the author doesn't have the comment privileges. I'm not very happy with my name proposal, I can't think of a better one right now.

  • incorrect edit — This edit introduces a factual error or invalid formatting.

This suggested edit introduced a factual error or invalid formatting.

In all these years I have sorely missed a way to clearly say “your heart is in the right place but your head isn't”.

It's more informative if the reviewer leaves custom feedback explaining how the added material is incorrect, but there are many cases where the feedback would be simply “the post was correct, read it and its references”. Leaving custom feedback should be just as optional as leaving a comment when downvoting an answer.

I'm open to ideas of something that encompasses factual errors, invalid formatting (common on Stack Overflow), and also rarer cases such as edits that introduce a grammatical error under the guise of correcting one.

  • radical change — This edit changes the core meaning or intent of the post, or denatures its style.

[question] This suggested edit changed too much in the original post. It added or removed too much material, or did not respect the original author. When editing questions, make sure not to accidentally remove the problem that the question seeks to solve.

[answer] This suggested edit changed too much in the original post. It added or removed too much material, or did not respect the original author. If you think an answer is too far from being correct, vote it down and, if possible, vote up correct answers or write an answer of your own.

“Changing too much” is reasonable guidance to give to the suggestor, but I agree with you that it isn't good guidance for the reviewers. What matters is whether the core meaning changes, not how many characters have changed. I'm not dead set on suggesting downvoting here, maybe suggesting a comment would be better? or both?

  • vandalism — This edit introduces spam, defaces the post in some way.

I'm not sure if we should expand on the guidance here. I've seen folks misuse the vandalism reason in the past; hopefully having a clearly labeled reason for incorrect edits will help in this respect.

We're getting a better mechanism for too minor, good riddance.

“Copied content” makes no sense on questions and answers. If it ever happens (which as far as I remember is 0 times in over 10,000 reviews), write a custom reason.

Tag wikis

  • incorrect edit; vandalism

These two are just like questions and answers.

  • markup in excerpt — Tag wiki excerpts do not support any form of markup.

For excerpts only. I want a predefined reason for that, to alert reviewers that they are reviewing an excerpt and provide this information to many an uninformed editor.

  • copied content — This edit copies content from an external source. It is not helpful or lacks attribution.

All content copied from an external source must be clearly marked as a citation and properly referenced. Generic descriptions such as encyclopedia articles and ad copy do not provide useful guidance regarding the tag as used in our community.

I'm not sure about this one: should attributed Wikipedia copypasta be lumped with unattributed copypasta or with “not helpful”?

The proposed text does not provide useful guidance regarding the tag as used in our community.

No, that's not a good name; I can't think of a better one at the moment. I dislike both the current name “not helpful” and the “RTFMBP” guidance.

I don't like Robert's proposed wording “lacks usage guidance”, because that's far from covering all cases. Some tags are about a precise concept (e.g. a specific product) and require no guidance. Some edits lack usage guidance because the appropriate guidance is already present. Conversely, an edit that says “the tag frobnication is for questions about frobnication” does provide usage guidance, but that doesn't make it helpful.

Do give this reason the same name for excerpts and bodies. I find it very annoying that they're currently in a different location because of the alphabetical sorting.

The blog posts currently used for wikis and excerpts isn't the best reference for this and the copied-content-with-attribution reason. It mixes two aspects: guidance on writing a good tag wiki, and explaining what the new excerpt feature is about. The blog post link should be replaced by a link to a new help center page that focuses on giving guidance in writing tag wikis and excerpts (a single page should cover both, as there is redundancy between the two).

I don't think we need anything like “radical change” for tag wikis. Removing useful material happens occasionally, but isn't very common. Completely rewording or reorganizing a tag wiki is not intrinsically bad.

10
  • 3
    Rarely has a Wikipedia dump been helpful for any tag wiki ever
    – random
    Aug 21, 2014 at 1:17
  • 2
  • Why external? What is content copied from a non-external source then? Similarly, what is attributed copied content then? Is the problem that it was not brand new text, or that it was unattributed, or both?
    – tchrist
    Aug 21, 2014 at 3:30
  • I like “invalid formatting”, although I know that this will be a bone of contention from people who like posts to look like ransom notes cut up from a dozen flashing neon signs.
    – tchrist
    Aug 21, 2014 at 3:33
  • @tchrist There are two problems, maybe even three; I'm still undecided as to whether they should be lumped together. Some suggested edits introduce plagiarism (unattributed content). Some provide improper attribution (e.g. they say “from Wikipedia” but don't clearly indicate what was copied). Some provide proper attribution and are still not useful. An advantage of lumping them together is to ease the job of the reviewer, so that they don't have to research whether the source allows copying and under what attribution conditions if the content isn't useful anyway. Aug 21, 2014 at 9:24
  • 1
    I agree that "copied content" should probably be left off on questions and answers. There's very little incentive to introduce plagiarism in as suggested edit (compared to submitting a plagiarized answer or duplicating a question). And besides, it's not something we can reasonably expect reviews to find. Finally, one of the other reasons ("radical change") is likely to apply. (Great suggestions, by the way.) Aug 21, 2014 at 17:21
  • What sorts of attempted edits to excerpts have you seen that try to introduce markdown formatting to them? Are you seeing attempts to render into bold or italic or monospace or su{b,per}scripts, or is that they are trying to create higher level structures like <h1> stuff or lists or hot URL links? BTW, I still would really like a way to pull up a list of “recent” Wiki edits. I would prefer if it were done like Review Queue histories, so it doesn’t expire, but even the 10k tools’ “last 50” approach would be better what we have — which is to say, better than nothing.
    – tchrist
    Sep 9, 2014 at 4:36
  • @tchrist Mostly links, sometimes bold and emphasis, occasionally things like lists. The data explorer would give you precise data. Sep 9, 2014 at 6:29
  • I like the idea of separate text for reviewers and editors, but such a change is beyond the scope of this project for the time being. I'll keep this in mind for the future though; until then: meta.stackexchange.com/questions/238333/…
    – Shog9
    Sep 24, 2014 at 22:57
  • @Shog9 I've split off the difference guidance text idea into a standalone feature request. Sep 24, 2014 at 23:54
4

How about renaming "invalid edit" to "not an edit", similar to how "not an answer" is defined?

The only other concern I have right now is that I believe "copied content" is underused because it's hard to know when that's the case. Renaming/clarifying it isn't gonna make a huge difference since the core problem is that you basically have to go to another site (Google!) and search for a phrase to see that the excerpt is plagiarized (y halo thar wikipedia!).

We should put more thought into that... Considering how exact the copies we get are, maybe we can do something to just check if Wikipedia has an article containing this exact text. Then again, maybe not. Either way, while it'd be nice to have a clearer description of the rejection reason that doubles as a call to action here, I don't think that's going to do much for solving the underlying issue.

2
  • 6
    The wording “not an edit” doesn't make much sense. Of course it's an edit! It should not have been one. (I don't like the wording “not an answer” either.) Aug 21, 2014 at 1:14
  • 7
    "should not be an edit" also works. :)
    – Adam Lear StaffMod
    Aug 21, 2014 at 1:24
0

Here's one I would love to see specifically addressed:

Adds excessive or unnecessary formatting.

It seems a little too common to see suggested edits that look like this:

how do I foo the bar in baz while

  • standing on one foot

    1. on friday afternoon

                                           in
      

iceland?

Inspired by: Reject an already-approved suggested edit when rolling it back

2
  • too specific. Already covered by "incorrect edit" and most reviewers will be glad to call your example "vandalism". Sep 26, 2014 at 0:16
  • @JanDvorak The example is intentionally exaggerated, and while it would fit in "incorrect edit" I think having a specific reason for it would help to curb the behavior.
    – apaul
    Sep 26, 2014 at 0:20
0

I think either the too minor reason should be introduced back or the language of no improvement whatsoever reason should be softened (or the title of the custom box may be changed). Because currently, users who made sincere suggestions with some (but not significant) improvement get a very hostile return. Note that there are no other option to fit this situation -- i.e., the edit causes no harm to the post and improves it a little.

This matter has been raised in here with a list of other examples and voting results support the above claim and the explanations given for the rejection of the request are not satisfactory.

You must log in to answer this question.

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