could you do a mockup of what you intend? Just to get the idea. I
don't quite grasp it.
Also, how would your proposed design scale to a very large review? Say
one consisting of 50+ files.
ReviewBoard do sorts headers and implementation files for C/Objective
C/C++ so that the header is right before the implementation,
but during pagination it doesn't take that into account (that I know
of, but I might be wrong).
It's an interesting idea, either way :)
On Fri, Aug 20, 2010 at 1:56 PM, Geoffrey Zheng <geoffreyzh...@yahoo.com> wrote:
> The current diff viewer does not scale at all for a large review with
> lots of changes in lots of files.
> Maybe I'm just spoiled by the incredibly fast and super convenient
> Beyond Compare. kdiff3 is also great.
> I know I'm asking too much for a web app to work like a native diff
> tool. But it could work much better by using tabs.
> You would put the current file list (which is great) on the left, and
> a tabbed pane on the right. Each tab contains one file.
> There are at least 2 immediate and immense improvements:
> 1. The diff page height only grows at one extra line per file. Now the
> page could be so long that I can't even see the scrollbar thumb.
> 2. You can jump back and forth in the files you're interested. This is
> really the killer especially for reviewing c/c++ code with changes in
> both .c and .h.
> Is there any tabbed pane widget available in RB's current setup? I
> could try it if this isn't totally crazy.
> Want to help the Review Board project? Donate today at
> Happy user? Let us know at http://www.reviewboard.org/users/
> To unsubscribe from this group, send email to
> For more options, visit this group at
Want to help the Review Board project? Donate today at
Happy user? Let us know at http://www.reviewboard.org/users/
To unsubscribe from this group, send email to
For more options, visit this group at