| Issue 1745: | Diff viewer hidden 1 line produces no space saving | |
| 3 people starred this issue and may be notified of changes. | Back to list |
Affected Version: 2.4.2-2-ge9a1970 The patch diff viewer shows (skipping 1 common lines) even though there is no space savings. To keep visual distractions to minimum, I think diff should not hide less than 3, possibly 4 common lines. The blue line is a good indicator of a larger chunk of hidden code, but useless for tiny savings.
Sep 8, 2014
I just tested this on revision 6b6c8f72 (Mon Sep 8) and it correctly ignores common lines < 2. This is a smaller threshold than suggested by @yuriastrakhan but still indicates this is no longer a bug. Suggest closing. |
|
| ► Sign in to add a comment |
Owner: bklarson@gmail.com
Labels: -Priority-Minor Priority-Trivial