What is meta? ×
Meta Stack Exchange is where users like you discuss bugs, features, and support issues that affect the software powering all 127 Stack Exchange communities.

Issue 1:
I find it a bit irritating that when I click the Improve button and further edit it, even if I mark it as helpful, only Community takes the credit for the approval. I know the topic itself will contain the link to the suggested edit, but my activity history only says that I made an edit, and not approved a suggested edit. The same holds true for marking one as unhelpful. The activity history should say I rejected an edit, and then made an additional edit. I know it's not often, but sometimes I do try to go back to a suggested edit I've rejected, but when I improved and rejected, it's not traceable for me as it doesn't show in my history or in the question's revision history. So if I don't remember what user suggested it, it's lost to me.

This issue also plagues the Proofreader and Reviewer badges which are awarded for approving and rejecting suggested edits, which essentially you have done, you're just not receiving credit for it. I suggest that when I mark an edit as helpful or not helpful, both me and Community be listed as the approvers or rejecters, unless another user is already listed in the appropriate place.

Update: Here is an excellent example of the confusion this causes.

Update 2: Also, when improving a post and marking it as "not helpful", it would be nice to be able to add a rejection reason along with it.

Issue 2:
Expanding off the idea that users need more notice for suggested edits that get rejected, the activity history page is much too useless when it comes to listing the suggested edits a user has made. Literally all it does is list them. A user still has to click through to each one in order to find out if it was approved or rejected, or even if it's still pending. Take, for example, this screen shot of the suggested edit history:

suggested edit history

Is it really necessary to say I suggested a suggested edit? Really? Of course I suggested it, it's a suggested edit. This isn't tautology club!

A good way to fix this would be to change the second "suggested" word into one of: pending, approved, or rejected. This would be a quick way to tell which edits have been approved or rejected and which ones are still pending. Users would be able to target out rejected edits and review them for the reasons. Something like this:

revised suggested edit history

share|improve this question
I think the "Rejected" one could profit from showing the reasons why the edit suggestion was rejected. I put effort into those... –  sarnold Mar 22 '12 at 2:53
add comment

1 Answer

I like where this is heading. Issue 1 has now been resolved with the update to the new review system, where an Edit action is now recorded under your profile and shows on the suggested edit itself, so credit is given to you and Community, like so:

Edit by animuson, Approved by Community

However, it seems that we're still not shown an Approved/Rejected mark next to suggested edits when viewing our list of suggestions in our profile. This is peculiar because the people reviewing have this information listed in their list of reviews, like so:

Reviews by me, one Approved, one Edited

Could this same layout not be applied to the suggestions list, showing Approved, Rejected, or Pending next to each item?

share|improve this answer
Along with Approved, Rejected, and Pending, I think that they could also use Improved and Rewritten. Improved would mean that a helpful edit was edited further. A Rewritten would mean that a non-helpful edit was edited rather than rejected. All of these headings should have a (Why?) link next to them that shows any feedback from the reviewers. –  Brythan Aug 3 '13 at 21:38
And yet another situation to cover: rollback. If a user's edit is accepted but later rolled back, the Accept should change to a Rolled Back. –  Brythan Aug 7 '13 at 4:48
@Brythan That's getting carried away. The status is meant to be an action from the review. An external rollback is not even related to the review panel whatsoever. As far as Improved or Rewritten- if they go that way I'd prefer they both just be combined as Edit to match the existing actions. But really I'd rather them see an Approved or Rejected there; the fact that it got Edited doesn't seem all that relevant when browsing my list of reviews. –  animuson Aug 7 '13 at 4:51
I would argue that the purpose of the message is to give feedback to the editor. The review is irrelevant to the editor except in that it can give feedback. A rollback also offers a chance for feedback and should be marked. In regards to your other point, I agree that people want to know if their edits were Accepted or Rejected. I believe that Improved and Rewritten do that. They also communicate that the edit was found wanting but that the issue was addressed. Should reviewers reject mostly good edits so as to leave rejection feedback? Improved/Rewritten feedback is better there IMO. –  Brythan Aug 7 '13 at 5:12
add comment

You must log in to answer this question.

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