27

I'm reviewing a suggested edit that wraps the code in a stack snippet and removes the JSFiddle link:

https://stackoverflow.com/review/suggested-edits/5797556

I have skipped the review since I'm not sure whether to approve or reject.

What should be done in case I found something similar?

2
  • 2
    Looks like the Stack Snippets UI isn't rendered in the output diff. That could be a problem.
    – BoltClock
    Sep 18, 2014 at 4:55
  • 1
    Since there is really no point to having a snippet containing a question's broken code example, I reverted the edit.
    – Cerbrus
    Sep 18, 2014 at 8:24

1 Answer 1

29

As these are clearly not being done by the OP I would reject them.

There is absolutely no point in mass editing posts to convert code to Stack Snippets as I've said both here, here and on Meta Stack Exchange.

If you find you have to edit one of your own posts for any reason then that's the time to update any code to be a snippet.

14
  • 19
    Perhaps the edit interface should temporarily display a warning with this info?
    – user247702
    Sep 18, 2014 at 8:24
  • @ChrisF, I have rejected two suggested edit that converted to stack snippet, here and here, but the majority approved the review, Was I wrong? Or should I keep rejecting those? Sep 19, 2014 at 10:43
  • 3
    @YuliamChandra - the fact that others don't do the right thing shouldn't stop you. Carry on rejecting, other's might eventually get the message.
    – ChrisF Mod
    Sep 19, 2014 at 10:49
  • 5
    If somebody happens to be reading a new post that's already on the front page, and it's clear that a stack snippet would make the question or answer obvious and easy to demonstrate, why not edit it if it only takes a moment? It does no harm.
    – nhinkle
    Sep 19, 2014 at 20:52
  • 6
    @ChrisF People don't get the message unless you mass-ban them.
    – bjb568
    Sep 19, 2014 at 21:16
  • @nhinkle that's not what I'm concerned about. If you're doing that sort of thing, then you're not doing mass edits.
    – ChrisF Mod
    Sep 19, 2014 at 21:39
  • 1
    the fact that others don't do the right thing shouldn't stop you. Carry on rejecting, other's might eventually get the message. And that, kids, is why StackOverflow's review process is flawed. Sep 21, 2014 at 3:54
  • Should we be encouraging people to treat posts originally authored by someone else different than posts originally authored by themselves? I would think that the right course of action is the right course of action regardless of original authorship.
    – corsiKa
    Sep 21, 2014 at 7:54
  • @corsika the issue is that old posts are being mass edited. If it was a new one, I'd be less concerned.
    – ChrisF Mod
    Sep 21, 2014 at 8:50
  • 1
    @ChrisF: So this suggestion I just came across is OK?
    – Bergi
    Sep 21, 2014 at 10:58
  • @Bergi - I would have said it was pointless. I ran the snippet and, as it counted down from 1 minute, I assumed it was working. If it was meant to show that the code was broken it didn't do that at all.
    – ChrisF Mod
    Sep 21, 2014 at 18:00
  • @ChrisF, should we rollback approved suggested edits that simply add in a stack snippet, especially if the snippet has no use. For instance, here the snippet does nothing, doesn't log anything to the console, has nothing for html, and does not demonstrate broken code Oct 3, 2014 at 20:44
  • @PatrickEvans - treat it like any other edit. If it's useless or makes the post worse roll it back.
    – ChrisF Mod
    Oct 3, 2014 at 20:47
  • 1
    It'd be nice if we could flag our own edits to our posts as low-impact or something so as not to affect question sorting.
    – canon
    Aug 12, 2015 at 17:39

You must log in to answer this question.

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