SteamedPears / CodeReviewClientMaster

Other
4 stars 3 forks source link

comment summaries #46

Open spratt opened 12 years ago

spratt commented 12 years ago

In the comment area on the right of the page, we should display a summary of the comments on that line. The summary should be something like a button or a box that can be clicked on to expand the full comments.

spratt commented 12 years ago

We should display the first few users who have commented on a line. The more information we can put into these summaries, the better.

Gankra commented 12 years ago

We need to provide support for just loading the comment counts first. If we do that, I'll assume we're also loading with each comment number something called "preview" which previews some info. E.G.

[{"line":"3","count":"4","preview":"John: I don't know..."},{"line":"17","count":"2","preview":"Steve: I do know..."}]

Since we'd just be getting do/commentInfo, the preview part can be dynamically filled in with whatever (though caching this would obviously be faster).

spratt commented 12 years ago

Looks good, I'll implement this on the back end and let you know.