ooyala / barkeep

The friendly code review system.
getbarkeep.org
1.42k stars 145 forks source link

One file per page #348

Open mbrochh opened 12 years ago

mbrochh commented 12 years ago

On big commits, and a slow internet connection or a slow PC barkeep loads increddibly long. Maybe it would be good to show an index of all changed files (like you do already) and show the number of comments behind the filename. Then just show one file per page and let us jump to the next file after each file.

The "global" comment should be entered in the header below the file list, not at the very bottom. This is usually the most important comment that sums things up or already contains LGTMs from other colleagues. It sucks that we have to scroll all the way down to see if there is a global comment, then scroll all the way up again and start the actual codereview.

Is this issuetracker the right place to suggest features?

philc commented 12 years ago

Yep, this is the correct place for feature requests.

cespare commented 11 years ago

@mbrochh by the way, the keyboard shortcut shift+c (C) takes you to the bottom of the page to add a new comment. It's a quick way of getting to the bottom to see all the commit-level comments.

mbrochh commented 11 years ago

@cespare that's awesome! Thanks a lot!