GNU bug report logs - #33567
Syntactic fontification of diff hunks

Previous Next

Package: emacs;

Reported by: Juri Linkov <juri <at> linkov.net>

Date: Sat, 1 Dec 2018 22:13:02 UTC

Severity: wishlist

Tags: patch

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: 33567 <at> debbugs.gnu.org, juri <at> linkov.net
Subject: bug#33567: Syntactic fontification of diff hunks
Date: Tue, 18 Dec 2018 17:55:15 +0200
On 18.12.2018 1:11, Juri Linkov wrote:
>>> Also with more highlighting in diff hunks now diff indicators need more
>>> distinctive colors that I also tested.
>>
>> Thanks, this LGTM.
> 
> Pushed to master and closed.

Juri, thank you. Looks like an helpful feature.

Have you considered bringing over the "default" face foregrounds as 
well, though, not just the special highlighting?

My brain sees syntax highlighting in some places and keeps wanting to 
interpret the green and red text and something syntactically meaningful 
as well.

Maybe we could add extra ancestors for diff-added and diff-removed 
faces, with default foregrounds, and use them for the base highlighting 
of diff chunks, aside from +'s and -'s.

Another thing that messes up the display a bit, is that comments and 
docstrings also have customized backgrounds in my theme, but that's less 
essential and seems harder to fix.




This bug report was last modified 6 years and 146 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.