« first day (40 days earlier)      last day (292 days later) » 

03:45
what's up? I got an invite from somebody
03:57
@RetoKoradi 5709377 is invalid, the technical content of an answer shouldn't be changed by a suggested edit. 5709521 is too minor, there is no need to edit a post that should be deleted. Same with 5702600.
@RetoKoradi 5709388 is too minor since it doesn't fix other major problems in the post. 5702818 changes the technical content of the answer.
04:22
well, I disagree on most of those. that's why we have multiple people voting. you can't expect everybody to agree with all of your votes
04:53
@RetoKoradi It would help if you point out why you disagree as well. Remember that the reviewers here are to approve good content that makes the place better. If you share with us why you think the five reviews were alright it would give room for discussion and a chance for both of us (you and us, the people who try to educate reviewers) to learn a new stance.
05:29
stackoverflow.com/review/suggested-edits/5709377: Looks like a legitimate edit to me. I know this is partly controversial, but I'm in the camp that believes fixing minor problems in answers is perfectly fine. Which is also supported by the official guidelines.
It is invalid because it changes the naming of the code without a decent reason. Usually to request a change in code, users critique for such edits in comments and have the poster change it themselves.
Changing code is not welcomed because they are most of the times up to own preferences (unless it's formatting to par semantics).
The reason is given in the edit comment. As I said, I know some people share your point of view on this. Others don't.
I'll clarify.
Would you accept an edit that changes the following code blocks from 1 to 2?
Number number = getNumber();
setNumber(number);

----

setNumber(getNumber());
stackoverflow.com/review/suggested-edits/5709521: Looks like a very significant formatting improvement. Point taken about the post deserving to be deleted. If I had read it more closely, I probably would have cast a close vote instead. It did get 3 Approves and 1 Reject, so I wasn't alone with my vote.
On that example? No, of course not. It does not fix a problem in the code.
It's too minor as it indents code, but puts the comment lines: "I want to navigate back to the page with the field I enter the cell data into .Navigate "web page"" into the code blocks. Without fixing those problems, approving this edit would mean another edit would have to be commenced to fix that.
The general law to "Too Minor" rejects is that if I can improve your edits in a way way better (aka you're not fixing every significant problem), then the editor is not doing it right. There is no point to waste another three reviewers' time.
As said in that text, "suggested edits should be substantive improvements addressing multiple issues in the post.". Only if nothing else can you fix, code formatting improvements should deserve the two rep points.
05:39
Yeah, that poster used comments in the code to explain what is going on. Note that it uses valid VBA comment syntax. So while maybe unconventional, it's perfectly valid to have that text inside the code block.
I'll let you score there. For this edit, look at the title - "How do I" can efficiently be converted to "How to". "it's fun learning though" is noise - the editor should of removed it.
The editor has 1251 rep - he's not new to the site. There's no reasoning / excuse to leave out other significant problems. The edit is too minor.
Anyway, moving on. stackoverflow.com/review/suggested-edits/5702600 is a similar case. It's a bad question. Should have close voted in hindsight. I normally focus on the merit of the edit when I'm in the edit review queue, and on closing when I'm in the close review queue. Could certainly have done that better.
Also, as for "It did get 3 Approves and 1 Reject, so I wasn't alone with my vote." (Emphasis mine), that shows the problem of our reviewers approving minor edits. That's why this room exists. Sorry for your time if this has gone on for too long.
The edit you linked is painting the titanic while it sinks. The tag doesn't improve the question at all.
There's still mistakes left unfixed, like "u" -> "you".
I don't disagree with that. stackoverflow.com/review/suggested-edits/5702600... yes, that vote could have gone either way. Definitely on the minor side. I did not spot any other major issues in the post, so I was generous this time.
Another problem to fix is to change "And since" to just "Since".
There's nothing to connect in these lines.
05:47
Sorry, copied wrong URL...
You should only approve edits that makes an improvement, or you're giving out free reputation to editors who farms this thing.
Frankly, the whole rep aspect is not a big consideration for me. Let them have their unicorn points. My goal is to improve the quality of the site.
I'm with you on improving the quality but by improving minor edits, you're encouraging them to make more minor edits.
@RetoKoradi It was an anonymous user anyway, so both Improving or Reject & Edit-in was a decent option. That one could have gone either way.
05:51
I really wish there was a better way to handle this whole editing process. It looks like Shog9 is working on some improvements. In an ideal world, any improvement to a post would be a good thing. But I'm aware of the caveats (like undesired bumping of posts). Complicated topic that has filled many pages...
Ironically, some people on MSE are blasting me for a post where I promoted rejection of (what I consider) minor edits. Others tell me that I approve too many minor edits. You really can't please everybody here.
I agree, and I'm glad you're aware. Currently all we can do is by education. There has been signals of "Too minor editors" for a while now, arises on Meta and some chat rooms. That's why this room exists so we can talk to our reviewers.
That's also a problem since editors were not _taught_ how to review - the interface says:
Approve edits you know are correct
Improve Edit the edit by making additional corrections
Reject and Edit the original post and provide a completely new edit
Reject those you know are wrong
Skip if you are not sure and want to go to the next suggested edit
Reject those you know are wrong <- wth is that supposed to mean?!
I have done a few 1000 reviews, so I'm aware of how it works. I reject a significant number of edits that end up being approved anyway. My approval rate in the edit queue is 55%. I know some are stricter, but I'm certainly not one of the people who approve everything.
Good to hear. Thanks for the awareness.
Jul 27 at 19:56, by bjb568
I have caught the users which I invited to this room approving minor, often invalid edits with the edit summary "Added new tags for better reach of questions and for proper audience!". The proper review for these was to reject as too minor, invalid edit, or vandalism. Approving these edits rewards this type of bulk low-quality editing, which is not the signal we want to send.
I certainly appreciate the effort of working on improving things. There's a lot that could be better. But please realize that just because somebody disagrees with you, that doesn't automatically mean that they are wrong.
You can improve those. Improved edits gets insta-approved anyway.
06:03
Locking out others while somebody works on a review/improvement is a very recent feature. Improving edits was a lost cause before. See my question on MSE, which was never really answered: meta.stackoverflow.com/questions/260969/….
BTW, the current edits review system "checks out" a ticket to you. If you receive the review task, it will not be distributed to anyone else from the review page (unless they have a link or uses the edit (1) button in main post). You can now write thorough reject analysis as a reject reason, or improve to your hearts' desire.
Yes, but that was only introduced a few days ago.
That is true, just thought I'd mention.
 
6 hours later…
12:13
This is meant for dreamyourcareer. Please don't make minor edits. Up to two thirds of the edits that I looked through were too minor. See 5702997, 5702992, 5703419 and 5703369.
1. Don't add a Thank you to the posts you edit. You should remove it. 2. When you edit I've to I have and the like this is really not necessary and is dependent on self-preference. 3. You shouldn't edit technology names as code blocks. php is a name and should not be formatted as code.
@ArtjomB. @Mention editors in the relevant posts as a comment, they will be notified.
This room exists to poke reviewers not to approve edits that shouldn't be approved otherwise, and you can invite users into a chat room by visiting their chat.stackoverflow.com profile -> Invite this user to..
12:43
@Unihedron I already posted a comment to one of the posts of the user and also invited him to this room, but he obviously ignored me. I flagged one of the edited posts for moderator attention now with a link the first chat message here.
Oh man, I completely forgot that this room is about reviewers and not editors.
... :P
Well, I'm going back to hunting for good questions, see you around some other time. :)
k, bye

« first day (40 days earlier)      last day (292 days later) »