google-code-export / rietveld

Automatically exported from code.google.com/p/rietveld
Apache License 2.0
1 stars 0 forks source link

Show number of comments made by me beside total number of comments. #391

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Right now the total number of comments are shown as 
"X comments"
It would be much better to handle large reviews if it also showed how many of 
them are by me (to quick inspect where I've missed responding to comments by 
reviewers):
"X comments (Y by me)" or just simply
"X comments (Y)"

At what URL are you accessing Rietveld?  (e.g. codereview.appspot.com)
Please note if you are using the Google Apps Labs version (e.g.
codereview.<yourdomain>).

https://chromiumcodereview.appspot.com/

*** If you are a Google employee please say so or mail me (Guido van
Rossum) directly. ***
Yes.

Please provide any additional information below.
I am interested in working for the patch, if this seems useful feature for 
others, thanks. 

Original issue reported on code.google.com by lazy...@google.com on 30 Aug 2012 at 9:40

GoogleCodeExporter commented 9 years ago
I like the "(Y by me)" addition and am looking forward to your patch.  If Y is 
0, should it say (0 by me) or nothing?  If you're not logged in, it should 
definitely be suppressed (not say (0 by me)).

Original comment by gvanrossum@gmail.com on 30 Aug 2012 at 9:56

GoogleCodeExporter commented 9 years ago
I agree, (0 by me) is superfluous imo, even when I'm logged in.
Patch on the way.

Original comment by lazy...@google.com on 31 Aug 2012 at 5:12