My favorites | Sign in
Logo
          
New issue | Search
for
| Advanced search | Search tips
Issue 46: 80-column indicator in diff view
12 people starred this issue and may be notified of changes. Back to list
Status:  Confirmed
Owner:  ----
Type-Enhancement
Priority-Low
Component-DiffViewer


Sign in to add a comment
 
Reported by trowbrds, May 15, 2007
I would be so totally in love in the diff viewer if it could show a thin
grey line at 80 columns, to make it easy to spot wide lines.
Comment 1 by chipx86, May 28, 2007
(No comment was entered for this change.)
Labels: Component-DiffViewer
Comment 2 by krbyrd, Jun 05, 2007
I would be even MORE in love if that '80' columns was settable by either the review
author or by reviewer. In the latter case, I guess it would be just a local setting.

Comment 3 by Wandering.Womble, Aug 27, 2007
Would also be very nice if there was a default and per-user 'force wrap' setting at
which point the diff-viewer would wrap the line- ie I'd prefer to have an inserted
forced wrap rather than having to scroll the browser window.

Comment 4 by trowbrds, Sep 28, 2008
(No comment was entered for this change.)
Status: Confirmed
Comment 5 by m...@roboss.com, Jul 14, 2009
Another idea is to cut off lines longer than some length (say 90) with an ellipsis. 
Then show a simple tooltip for the rest of the line when the user hovers over the '...'.

Looking at wrapped code can be a pain too.
Comment 6 by gic...@gmail.com, Sep 17, 2009
Maybe the background color could be different for characters after the boundary.
Example:
<div class="line"><span class="first-80">first 80 chars...</span>trailing chars...</div>
<div class="line"><span class="first-80">only 50 chars here...(30 chars
spaces)</span></div>

div.line {background-color: blue}
div.line .first-80 {background-color: white}
Sign in to add a comment

Hosted by Google Code