Kallithea issues archive

Issue #80: diff view: suggestion: indication of 80-chars limit

Reported by: Thomas De Schampheleire
State: new
Created on: 2015-01-23 12:37
Updated on: 2015-01-23 18:11


In diff views, it would be nice to be able to see where the 80 / 120 / ... character limit is, mainly useful for code review in projects where there is a limit on line length.

Some example implementations: - a kind of ruler at the top of each diff section (for each file) where column 10, 20, ... is indicated. - similar to vim's colorcolumn, a permanent different background color of column 80, 120, ... - an indication on the line itself, only visible when hovering the line

The columns to highlight could be fixed, a configuration option (per project, or per user), or dynamically changeable at the top of the diff view (with a default value).



Comment by Thomas De Schampheleire, on 2015-01-23 13:02

Comment by Mads Kiilerich, on 2015-01-23 18:11

This and other "coding style compliance" visualization is something that I think it is better to handle with some kind of plugin. In my case, a patch on top of the upstream version works just fine.