Re: [PATCH] Give the hunk comment its own color
- From
Jeff King <peff@peff.net>
- Date
- Nov 18, 2009, 22:44 UTC
- Message-ID
- <20091118224448.GA4616@coredump.intra.peff.net>
- In-Reply-To
- <7vaayjebu5.fsf@alter.siamese.dyndns.org>
On Wed, Nov 18, 2009 at 01:56:34PM -0800, Junio C Hamano wrote:
> On black-on-white terminals, cyan tends to be less visible, and I think > that is the whole point of painting the hunk header @@ .. @@ in that > color--- make it less distracting).
Hmm. I find cyan-on-white gratingly ugly instead of "less distracting", but then again, I find black-on-white terminals to be eye-searing in general.
> I would suggest "normal" as the neutral default. After all, the purpose > of the funcname in the hunk header is to give context to people who read > patches.
I think that is sensible.
-Peff