Comment #8 on issue 46 by chipx86: 80-column indicator in diff view

It would need to be done in such a way where it's 1) fast, 2) customizable without
regenerating the HTML, and 3) user-configurable.

Not everyone at a company will necessarily have the same standards, so the user would need to be able to change this. It'd have to dynamically update the HTML. This is the slow, complicated part. Due to syntax highlighting, the text is broken up into many text fragments surrounded by tags. So knowing the correct place is a bit tricky.

We'd be able to hack it by adding a vertical line or something, except then
word-wrapping is an issue.

I'm all for adding this feature, but someone needs to find a good way to do this and
implement it. It's not on my todo list though.

You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:

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

Reply via email to