|
From: | Kristoffer Grönlund |
Subject: | Re: [PATCH] * etc/themes/wombat-theme.el: Don't set foreground on region |
Date: | Sun, 28 Feb 2016 08:13:33 +0000 |
Sounds good to me too.
Mitchel Humpherys <address@hidden> writes:
> On Mon, Feb 22 2016 at 05:04:31 PM, Mitchel Humpherys
> <address@hidden> wrote:
>> Setting a foreground color on the `region' face is slightly annoying.
>> This is subjective, of course, but the consensus among other themes and
>> around the web is to *not* set a foreground color on the region face.
>
> Any objections to this? I should have mentioned that the reason it's
> slightly annoying is because setting a foreground color hides existing
> syntax highlighting within the region text.
The patch sounds OK to me.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
[Prev in Thread] | Current Thread | [Next in Thread] |