2957: Add a "I agree" ability to a review issue

s.sch******@gmai***** (Google Code) (Is this you? Claim this profile.)
Aug. 7, 2013
What version are you running?
-> 1.7.6

What operating system are you using? What browser?
-> Windows 7 64-bit, Firefox 20

What's the URL of the page this enhancement relates to, if any?
-> Reviews

Describe the enhancement and the motivation for it.
-> Allow for reviewers to easily show agreement with an existing open review issue. This interface would let the reviewers quickly indicate that they agree with another reviewer that this particular review issue should be addressed. 

Multiple reviewers may see the same problem/bug in a review. In this case the first reviewer (Alice) to identify the problem/bug will comment on the code and open an issue. Another reviewer (Bob) may agree that this issue needs to be resolved. Today Bob could leave a comment to say "Please fix the issue Alice pointed out," or restate what Alice already wrote. That's time consuming and redundant. It would be helpful to have a simple interface (perhaps a button) to show agreement with Alice's comment. The author would then see that multiple people feel that this issue needs to be addressed and can take that into consideration when addressing the review comments.



Please provide any additional information below.
I've seen several reviewers that mark a review as Ship It and include a message like "Please address comments pointed out by Alice," or the more subtle "No additional comments." These indicate or imply that the existing comments need to be addressed. 

I feel this is even more valuable now that we have the list of Open Issues on a review (a new feature in 1.7.6).
david
#1 david
I think the list of open issues actually makes this less important because it's easier to keep track of remaining work. If there's a disagreement about whether something should be changed, then sure, there should be discussion. If there's not, then I agree that it's redundant to re-state it, but adding extra UI elements to address that isn't as nice a solution as just not saying the redundant things.
  • +WontFix