We don't do anything about this. Realistically, in order to fix it, we'd have to do text processing on each outputted line and insert extra tags to tell the browser it can break there. This would greatly increase the size of the page and reduce the diff generation and load times. So we don't do it. It's annoying, but really, it's something best left to the browsers to handle. We do what we can by telling the browser to be aggressive with the word wrapping, but we can't control how aggressive they choose to be.
Christian -- Christian Hammond - [email protected] Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Jul 23, 2010 at 1:05 AM, Lan Cheng <[email protected]> wrote: > Thanks David. You're right. I checked reviewboard diff behaviour in > the Firefox and the Firefox breaks lines according to spaces and > comma. > But how do you deal with the situation when a line of code is long and > no spaces and commas in it? We have some long attribut names in our > code and usually make the line hard to break. You know moving scroll > bar back and forth is not convenient when reviewing code. > > On Jul 23, 3:33 pm, David Trowbridge <[email protected]> wrote: > > Typically this happens when lines in the source file contain long > > portions of text which have no "breakable" characters (such as spaces > > or certain punctuation). Different browsers have different characters > > that they'll wrap on, so that's why they're different. > > > > -David > > > > > > > > On Fri, Jul 23, 2010 at 12:28 AM, Lan Cheng <[email protected]> wrote: > > > Hi guys, > > > Do you come into this kind of issues when using Reviewboard? When I > > > view diffs in the IE browser, the vertical bar between old version and > > > new version is not in the middle of my screen and I have to move the > > > horizontal scroll bar of the browser to see the whole lines in the new > > > versions. > > > > > Firefox handles this issue a little better, but it also has the same > > > problem sometime. > > > > > Why this issue happens? How can we avoid this problem? > > > > > Any thoughts? > > > > > Thanks. > > > > > -- > > > Want to help the Review Board project? Donate today athttp:// > www.reviewboard.org/donate/ > > > Happy user? Let us know athttp://www.reviewboard.org/users/ > > > -~----------~----~----~----~------~----~------~--~--- > > > To unsubscribe from this group, send email to > [email protected]<reviewboard%[email protected]> > > > For more options, visit this group athttp:// > groups.google.com/group/reviewboard?hl=en- Hide quoted text - > > > > - Show quoted text - > > -- > 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/ > -~----------~----~----~----~------~----~------~--~--- > To unsubscribe from this group, send email to > [email protected]<reviewboard%[email protected]> > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en > -- 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/ -~----------~----~----~----~------~----~------~--~--- To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
