[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: About the :distant-foreground face attribute
From: |
David Engster |
Subject: |
Re: About the :distant-foreground face attribute |
Date: |
Sun, 12 Jan 2014 14:07:23 +0100 |
User-agent: |
Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.3 (gnu/linux) |
Jan Djärv writes:
> Hello.
>
> 12 jan 2014 kl. 13:21 skrev David Engster <address@hidden>:
>
>> Jan Djärv writes:
>>> 12 jan 2014 kl. 12:14 skrev David Engster <address@hidden>:
>>>
>>>> David Engster writes:
>>>>> I'm wondering: We already can set different face attributes depending on
>>>>> DISPLAY's 'background' property, which can be 'light' or 'dark'. Say the
>>>>> user is working with a 'dark' background by default, but we now detect
>>>>> that one of the font-lock faces has not enough contrast when highlighted
>>>>> by the region: why not simply switch to the face that is defined for
>>>>> 'light' background instead?
>>>>
>>>> Hey look, a tumbleweed!
>>>>
>>>> I'm assuming everybody's simply stunned by my ingenious proposal?
>>>>
>>>
>>> A theme does not have to suppy colors for properties light or dark.
>>
>> So? It also does not have to supply the distant-foreground attribute.
>
> Right. But distant-foreground is implemented,
It can be removed again. That is what this thread is about.
> you proposal is not and does not add anything except moving colors to
> some other place in the defface definition.
Exactly. I want to fix the original bug by Darren without the need to
introduce another face attribute.
> Also, for a theme that does have a dark and light
> version, there is no guarantee that applying the dark version on the
> light version (or vice versa) is consistent with the theme look.
A consistent color theme will choose the region's background color in a
way that the font-lock colors are always visible in the first place. The
problem is with the default colors. We can easily add a 'dark'/'light'
foreground attribute to our default font-lock colors that work well with
the default region background from GTK/NS.
-David
- Re: About the :distant-foreground face attribute, (continued)
- Re: About the :distant-foreground face attribute, Stefan Monnier, 2014/01/09
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/09
- Re: About the :distant-foreground face attribute, Eli Zaretskii, 2014/01/09
- Re: About the :distant-foreground face attribute, Chong Yidong, 2014/01/09
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/09
- Re: About the :distant-foreground face attribute, David Engster, 2014/01/09
- Re: About the :distant-foreground face attribute, David Engster, 2014/01/12
- Re: About the :distant-foreground face attribute, Jan Djärv, 2014/01/12
- Re: About the :distant-foreground face attribute, David Engster, 2014/01/12
- Re: About the :distant-foreground face attribute, Jan Djärv, 2014/01/12
- Re: About the :distant-foreground face attribute,
David Engster <=
- Re: About the :distant-foreground face attribute, Jan Djärv, 2014/01/12
- Re: About the :distant-foreground face attribute, Chong Yidong, 2014/01/12
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/12
- Re: About the :distant-foreground face attribute, Jan Djärv, 2014/01/12
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/12
- Re: About the :distant-foreground face attribute, Stefan Monnier, 2014/01/12
- Re: About the :distant-foreground face attribute, chad, 2014/01/12
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/09
- Re: About the :distant-foreground face attribute, Eli Zaretskii, 2014/01/09
- RE: About the :distant-foreground face attribute, Drew Adams, 2014/01/09