0

The two reviewers of https://stackoverflow.com/review/suggested-edits/22388699 agreed that

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

I'm a bit baffled. I replaced an explanation of a detail with the full code (which includes the detail) from the post author's comment, and removed some necessary text. I also added very relevant tags:

because the post is about Dyalog APL's interface to WinForms

because the question is about (unwanted) scaling of one GUI element in relation to another

because the user clearly wanted a behaviour that differs from the default behaviour.

As I'm fairly inexperienced on SO, please explain to me why my edit defaced the post with promotion of a product or service, or was deliberately destructive.

17
  • 15
    Significant Code changes are generally rejected. If you were adding from OP's comment, you could have mentioned it in the suggest edit comment
    – Suraj Rao
    Mar 6, 2019 at 15:49
  • 2
  • 12
    Well, "insert code [..]" is a rather unclear description for an edit when you wanted to say "add OPs code from a comment they wrote under the question".
    – Tom
    Mar 6, 2019 at 15:52
  • 7
    Also, by adding tags like scale and default-value you're at least risking me rejecting it for that. Those tags are overly broad, and not very descriptive of the question. They add nothing to the question in my opinion
    – Erik A
    Mar 6, 2019 at 15:59
  • 4
    I dont think either of those duplocates do it justice because the OP here did not clearly explain what they were doing in the edit comment, as opposed to both of the duplicates where such an explanation was used. The answer here is: ... Because you didn't usd the edit comment to properly explain your edit."
    – user4639281
    Mar 6, 2019 at 16:06
  • @TinyGiant Sounds like an answer to me.
    – Adám
    Mar 6, 2019 at 16:11
  • Note that while Tiny seems like a sane advice, nothing guarantees that reviewers would read a finely written edit comment, see meta.stackexchange.com/q/199584/213575 and meta.stackoverflow.com/a/315855/792066
    – Braiam
    Mar 6, 2019 at 16:21
  • @Braiam Right. I guess sub 2000 rep users just shouldn't attempt at editing then.
    – Adám
    Mar 6, 2019 at 16:27
  • Actually, I consider the opposite approach. Non-reviewers shouldn't review at all.
    – Braiam
    Mar 6, 2019 at 16:28
  • @Braiam That would require a change in behaviour of others. I'm attempting to change my own behaviour first.
    – Adám
    Mar 6, 2019 at 16:31
  • That's exactly what I don't want. Your behavior is good, and should be promoted and fostered. The reviewers behavior is bad and should be discouraged.
    – Braiam
    Mar 6, 2019 at 16:33
  • 4
    Reviewers can not see the comments so it is super important to use an appropriate edit summary. Mar 6, 2019 at 16:57
  • 3
    @Adám not really - how would the UI show a 30 comment long chain with limited screen real estate. And Edit Reviewers shouldn't need to read through a long comment chain to understand why an edit was needed. That is what the edit summary is for - to provide a the editor a way to communicate the reason/need/importance to the reviewers, and to serve also as a general note in the review history about why the edit was needed Mar 6, 2019 at 18:40
  • 8
    'F' ⎕WC 'Form' 'Drawing Lines' ('Size' 25 50) ('Coord' 'Pixel') ⋄ 'F.Line' ⎕WC 'Poly' ((10 90)(5 60)) looks almost exactly like gibberish code edits in an audit. If this is some kind of special dsl (with Unicode nonsense in it) I feel bad for folks who edit in the tag, because there's going to be a lot of false negative reviews.
    – user1228
    Mar 6, 2019 at 18:46
  • 6
    I have retroactively approved this edit, because it was a helpful edit and the rejection was invalid. The problems were two-fold: (1) your edit comment was inadequate, as it did not explain the origin of this code you were adding, and (2) as Will said, the code you added was indistinguishable from gibberish. That is, of course, a problem with Dyalog, but suggested edits are reviewed by the entire community, not just tag/subject matter experts.
    – Cody Gray Mod
    Mar 6, 2019 at 18:58

0