Last modified: 2012-10-30 16:54:56 UTC
Created attachment 11262 [details] Screenshot – Opera This happens for me on Opera 12.02 and Firefox 15 on Windows XP. gerrit appears to try to mitigate this issue by inserting <br>s into long lines, but it inserts them slightly too late, when the text is already off-screen. I was able to work-around this for myself using two lines of CSS: .GJEA35ODFF{ white-space: pre-wrap } .GJEA35ODFF br{ display: none }
Created attachment 11263 [details] Screenshot – Firefox
This is actually based on a per-user setting, and shouldn't unconditionally wrap. I spoke with upstream, and this was the consensus: "If you wrapped all the time, you would not be able to make your screen just big enought to see either the left of the right pane only / since it would just wrap instead / not to mention that on tablets it would likely always wrap then" Under Differences -> Preferences when viewing a patch, you can set the default column width which is where Gerrit will wrap for you. If the default is too wide, you can set it to a lower number. Marking WFM, since this is working as designed.
Hm, this sounds like a pretty useless feature to me, but at least it has an explanation. Thanks. A single checkbox for wrapping the lines to fit on screen or not would make more sense to me. I didn't expect an option like this, which is probably why I missed it. I set column width to 999 and kept the first line of my workaround. Thanks again.