emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] * etc/themes/wombat-theme.el: Don't set foreground on region


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.


On Sun, 28 Feb 2016 07:21 Lars Ingebrigtsen, <address@hidden> wrote:
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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]