GNU bug report logs - #16969
Isearch: lazy-highlight face sometimes has foreground and background the same colour.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 16969 in the body.
You can then email your comments to 16969 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: bug-gnu-emacs <at> gnu.org
Subject: Isearch: lazy-highlight face sometimes has foreground and background
 the same colour.
Date: Sat, 8 Mar 2014 18:35:50 +0000

Hi, Emacs.

In isearch.el, the face `lazy-highlight' is defined with only a
:background colour, e.g. turquoise3.  On a Linux tty, this is cyan.

When the text which has a lazy match itself has a cyan foreground
colour, the lazy highlighting leaves the cyan text on a cyan background
invisible.  This happens, for example, on the Linux tty when searching
through an Info menu.  This is surely a bug.

An effective solution would be to give the `lazy-highlight' face a
foreground colour, e.g. black.

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #8 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Alan Mackenzie <acm <at> muc.de>
Cc: 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sat, 08 Mar 2014 23:55:45 +0200

> An effective solution would be to give the `lazy-highlight' face a
> foreground colour, e.g. black.

It's useful to keep the foreground color to help seeing the context
especially when a match is highlighted on a font-look foreground.

But the problem needs to be fixed anyway.  So maybe we could add the
face attribute :distant-foreground to the `lazy-highlight' face
that could be applied only when the background color is near to the
foreground color like a cyan foreground on a cyan background.

Message #11 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: Juri Linkov <juri <at> jurta.org>
Cc: 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sat, 8 Mar 2014 23:01:18 +0000

Hi, Juri.

On Sat, Mar 08, 2014 at 11:55:45PM +0200, Juri Linkov wrote:
> > An effective solution would be to give the `lazy-highlight' face a
> > foreground colour, e.g. black.

> It's useful to keep the foreground color to help seeing the context
> especially when a match is highlighted on a font-look foreground.

Yes.

> But the problem needs to be fixed anyway.  So maybe we could add the
> face attribute :distant-foreground to the `lazy-highlight' face
> that could be applied only when the background color is near to the
> foreground color like a cyan foreground on a cyan background.

I don't know what ":distant-foreground" means.  It doesn't seem to be in
the Elisp manual.

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #14 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Daniel Colascione <dancol <at> dancol.org>
To: Juri Linkov <juri <at> jurta.org>, Alan Mackenzie <acm <at> muc.de>
Cc: 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sat, 08 Mar 2014 16:45:07 -0800

[Message part 1 (text/plain, inline)]

On 03/08/2014 01:55 PM, Juri Linkov wrote:
>> An effective solution would be to give the `lazy-highlight' face a
>> foreground colour, e.g. black.
> 
> It's useful to keep the foreground color to help seeing the context
> especially when a match is highlighted on a font-look foreground.
> 
> But the problem needs to be fixed anyway.  So maybe we could add the
> face attribute :distant-foreground to the `lazy-highlight' face
> that could be applied only when the background color is near to the
> foreground color like a cyan foreground on a cyan background

And what? Are we going to apply distant-foreground to everything and
hope that the color we choose is legible in whatever theme the user has?
We really need automatic contrast adjustment, not one-off fixes.

[signature.asc (application/pgp-signature, attachment)]

Message #17 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Jan Djärv <jan.h.d <at> swipnet.se>
To: Alan Mackenzie <acm <at> muc.de>
Cc: Juri Linkov <juri <at> jurta.org>, 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sun, 09 Mar 2014 10:37:48 +0100

Hi.


Alan Mackenzie skrev 2014-03-09 00:01:
> Hi, Juri.
>
> On Sat, Mar 08, 2014 at 11:55:45PM +0200, Juri Linkov wrote:
>>> An effective solution would be to give the `lazy-highlight' face a
>>> foreground colour, e.g. black.
>
>> It's useful to keep the foreground color to help seeing the context
>> especially when a match is highlighted on a font-look foreground.
>
> Yes.
>
>> But the problem needs to be fixed anyway.  So maybe we could add the
>> face attribute :distant-foreground to the `lazy-highlight' face
>> that could be applied only when the background color is near to the
>> foreground color like a cyan foreground on a cyan background.
>
> I don't know what ":distant-foreground" means.  It doesn't seem to be in
> the Elisp manual.

You can't have looked very hard.

Faces => Face attributes:

`:distant-foreground'
     Alternative foreground color, a string.  This is like `:foreground'
     but the color is only used as a foreground when the background
     color is near to the foreground that would have been used.  This
     is useful for example when marking text (i.e. the region face).
     If the text has a foreground that is visible with the region face,
     that foreground is used.  If the foreground is near the region
     face background, `:distant-foreground' is used instead so the text
     is readable.


	Jan D.

Message #20 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: Jan Djärv <jan.h.d <at> swipnet.se>
Cc: Juri Linkov <juri <at> jurta.org>, 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sun, 9 Mar 2014 13:25:44 +0000

On Sun, Mar 09, 2014 at 10:37:48AM +0100, Jan Djärv wrote:
> Hi.


> Alan Mackenzie skrev 2014-03-09 00:01:

> > I don't know what ":distant-foreground" means.  It doesn't seem to be in
> > the Elisp manual.

> You can't have looked very hard.

:-).  It was late at night.  I had the release version of the Elisp
manual open, not the trunk version.

> Faces => Face attributes:

> `:distant-foreground'
>       Alternative foreground color, a string.  This is like `:foreground'
>       but the color is only used as a foreground when the background
>       color is near to the foreground that would have been used.  This
>       is useful for example when marking text (i.e. the region face).
>       If the text has a foreground that is visible with the region face,
>       that foreground is used.  If the foreground is near the region
>       face background, `:distant-foreground' is used instead so the text
>       is readable.

Thanks!

> 	Jan D.

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #23 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Daniel Colascione <dancol <at> dancol.org>
Cc: Alan Mackenzie <acm <at> muc.de>, 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sun, 09 Mar 2014 23:43:52 +0200

>> But the problem needs to be fixed anyway.  So maybe we could add the
>> face attribute :distant-foreground to the `lazy-highlight' face
>> that could be applied only when the background color is near to the
>> foreground color like a cyan foreground on a cyan background
>
> And what? Are we going to apply distant-foreground to everything and
> hope that the color we choose is legible in whatever theme the user has?

We need to guarantee that default faces don't produce illegible
combinations of foreground and background colors.  This can be achieved
by using :distant-foreground on the default face definition.
A user redefining the face can also manually adjust :distant-foreground
to use another color if the user doesn't like the default color
used when the distance is more than the threshold.

> We really need automatic contrast adjustment, not one-off fixes.

Automatic contrast adjustment is useful too when the user has no preference
for a color to use instead of an illegible color and want to adjust all
such colors automatically.  There is a request in bug#16974 for this feature.

Message #26 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Drew Adams <drew.adams <at> oracle.com>
To: Juri Linkov <juri <at> jurta.org>, Daniel Colascione <dancol <at> dancol.org>
Cc: Alan Mackenzie <acm <at> muc.de>, 16969 <at> debbugs.gnu.org
Subject: RE: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sun, 9 Mar 2014 15:20:48 -0700 (PDT)

> We need to guarantee that default faces don't produce
> illegible combinations of foreground and background colors.

Yes, that is the _only_ *need* here.  It is the need raised
by the bug.

And the simplest and most foolproof way to fulfill that need and
fix the bug is to simply give face `lazy-highlight', by default,
a black/white foreground for a light/dark background mode.

Just as we do already for face `isearch', and just as we have
always done.  That should have been done for `lazy-highlight'
long ago.  End of story.

Any user who wants to fiddle with the new "feature" that lets
other highlighting show through and tries to adjust colors
automatically can always customize these search-hit faces to get
that new effect.  There is no reason to impose this gimmick on
Emacs users now by _default_.

Today's flashy new feature is too often tomorrow's out-of-fashion
annoyance.  Just stick with what is simple and has always worked
well.  If, after a few years, we find that _most users choose_ to
customize to get the new effect, we can then turn it on by default.
There is no reason to jump to that now.

It amazes me that it took so long and was such a bloody battle to
get something like `transient-mark-mode' turned on by default,
even though users commonly customized Emacs for decades to turn it
on, and yet you are ready to willy nilly change longstanding
default behavior such as this before the new behavior has even
been offered as a possibility for users to _choose_.  It hasn't
even been released yet, and you already want to make it the default.

First things first: turn on `delete-selection-mode' by default,
why dontcha?  And offer this new highlighting style as an
_option_ for now: opt in, not opt out, for the new.

Consider making it the default behavior only later, after you have
had a chance to see how many users actually choose it.  Don't get
carried away by your enthusiasm for your shiny new object.

Message #29 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Daniel Colascione <dancol <at> dancol.org>
Cc: Juri Linkov <juri <at> jurta.org>, Alan Mackenzie <acm <at> muc.de>,
 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has foreground
 and background the same colour.
Date: Sun, 09 Mar 2014 23:01:16 -0400

> And what? Are we going to apply distant-foreground to everything and
> hope that the color we choose is legible in whatever theme the user has?

For now, yes.

> We really need automatic contrast adjustment, not one-off fixes.

Not sure about "really", but I agree it's a desirable feature.  But it
won't be in 24.4.


        Stefan

Message #32 received at 16969 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Juri Linkov <juri <at> jurta.org>
Cc: Alan Mackenzie <acm <at> muc.de>, 16969 <at> debbugs.gnu.org
Subject: Re: bug#16969: Isearch: lazy-highlight face sometimes has
 foreground and background the same colour.
Date: Sat, 05 Feb 2022 23:46:18 +0100

Juri Linkov <juri <at> jurta.org> writes:

> But the problem needs to be fixed anyway.  So maybe we could add the
> face attribute :distant-foreground to the `lazy-highlight' face
> that could be applied only when the background color is near to the
> foreground color like a cyan foreground on a cyan background.

I've now done this in Emacs 29.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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