Re: Revision selector

2010-08-20 Thread Geoffrey Zheng
e you are, I would really encourage you to post > > your code tohttp://reviews.reviewboard.org, so I can check it out and > > discuss this further. > > > Cheers, > > > Eduardo Felipe. > > > On Fri, Aug 20, 2010 at 1:24 PM, Geoffrey Zheng > > wrote: > &g

Re: Tabbed diff viewer

2010-08-20 Thread Geoffrey Zheng
Skip did a perfect job sketching it. Thanks! If you haven't used/seen Beyond Compare, please do yourself favor and head to http://www.scootersoftware.com/moreinfo.php?zz=moreinfo_compare The problem with too many tabs has been long solved by at least browsers. Just see how Firefox does it with a

Tabbed diff viewer

2010-08-20 Thread Geoffrey Zheng
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 w

Re: Revision selector

2010-08-20 Thread Geoffrey Zheng
Screenshot is uploaded as rb.png http://reviewboard.googlegroups.com/web/rb.png?gsc=iBqKOxYAAABgpEsrTenpw2OJ9ZY_yaTfM9KWFkvR1kwtm4g5T1-szg -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~-

Revision selector

2010-08-20 Thread Geoffrey Zheng
The revision selector has always been a bit confusing to me (and new RB users in my company) and inconvenient. It's not exactly clear what diff you're looking at, cumulative or incremental (interdiff). And it may take an extra click to get an interdiff, for example you have 3 revisions and you're

Re: Diff with baseline

2008-11-13 Thread Geoffrey Zheng
Sorry if the word "baseline" is confusing--now it seems like it's the wrong word to use. What I mean is that I'd like to see the cumulative effect of all diffs, because as I mentioned in the example workflow, it's necessary and important at the end of the iterations to ignore the intermediate dif