Hi David.

There's a bug open for this at:


We should have this fixed for 1.0. It's not fixed yet, and it's been around
for a while, but it shouldn't be hard to fix. We'll probably get it in the


Christian Hammond - chip...@chipx86.com
VMware, Inc.

On Sat, Jan 31, 2009 at 8:08 AM, David Allouche <david.allou...@gmail.com>wrote:

> Steps to reproduce: Create a review request with a diff that includes
> the creation of a file. Update the diff with an additional change to
> the new file. Go to the diff view, view changes between r1 and r2.
> Observed behavior: The side-by-side diff table for the new file has
> class="sidebyside newfile", but there is actual data in the left
> column (the r1 of the new file). The CSS in diffviewer.css lines 19
> (table.sidebyside.newfile col.left) and 32 (table.sidebyside.newfile
> col.left) causes the left column to be narrow and the text inside it
> to wrap (inside firefox 3 at list, it seems IE7 does not honor
> white-space: pre-wrap).
> Expected behaviour: Since there is content in both columns, this
> side-by-side diff should not have the "newfile" class. Removing this
> class using Firebug leads to the desired layout.
> ReviewBoard-0.9.dev-20090116. Okay, that's not the lastest version,
> and I have not tested with the alpha prerelease, lacking time to do
> the upgrade.
> Is that a known issue? Have it been fixed in the alpha release?
> >

You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to 
For more options, visit this group at 

Reply via email to