Re: [PATCH 4/4] ui-blame: Allow syntax highlighting

2017-10-24 Thread Jason A. Donenfeld
Just waiting on resubmission of this 4/4, then. I'll put 1-3 in a branch. Jason ___ CGit mailing list CGit@lists.zx2c4.com https://lists.zx2c4.com/mailman/listinfo/cgit

Re: [PATCH 4/4] ui-blame: Allow syntax highlighting

2017-10-21 Thread Jeffrey Smith
On Sat, Oct 21, 2017 at 9:43 AM, John Keeping wrote: > > This doesn't account for tabs, which is noticable in long lines (I > happened to test with cgit.c where line 615 is quite a bit longer than > average). > > It is fixed by using: > > const char *start = blame_nth_line(sb, line); >

Re: [PATCH 4/4] ui-blame: Allow syntax highlighting

2017-10-21 Thread John Keeping
On Tue, Oct 17, 2017 at 11:17:35PM -0500, Jeff Smith wrote: > Place file contents into a single block so that syntax highlighting can > be applied in the usual fashion. Place the alternating color bars > behind the file contents. Force the default syntax highlighting > background to transparent.

[PATCH 4/4] ui-blame: Allow syntax highlighting

2017-10-17 Thread Jeff Smith
Place file contents into a single block so that syntax highlighting can be applied in the usual fashion. Place the alternating color bars behind the file contents. Force the default syntax highlighting background to transparent. Signed-off-by: Jeff Smith --- cgit.css | 10