I was just going through some Suggested Edit reviews and reviewed an edit that made a substantial change to how the code works. I decided to reject the edit, and noticed that 2 other users had also flagged it as an "Invalid Edit". However, once I clicked to reject it I got the notification saying that the edit has already been approved.
Still convinced that this was an invalid edit, I opened the answer to check if perhaps any comments had been made that justified that change to the code. I couldn't see any so I decided to rollback the answer.
Was I right to rollback the edit, or should I have left it as it was?
Here is the answer I am referring to in this particular instance.
EDIT: I hadn't originally noticed, but the user that made the suggested edit was in fact the user that asked the question. So consider this when providing an answer, although I personally feel that this still doesn't justify the edit being approved.
Update: The OP of the question, Steve, has since added a new answer which shows the modified code from the original suggested edit. Fair play to Steve for reacting to the situation, and contributing a new answer.
To be more precise about what I am asking:
- Are there acceptable cases where rolling back an approved suggested edit is considered acceptable?
- If so, what are the guidelines for judging this?
- Are there any factors (such as the edit being suggested by the OP) that bypass these "rules"?
- What alternative actions can be taken other than the rollback? (e.g. should a flag be raised for moderator attention?)
Side note: I forgot to leave a reason for my rollback originally, and I do not seem to be able to edit in a reason - it lets my type and save, but it doesn't persist to the revisions page.