all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Juri Linkov <juri@jurta.org>, Daniel Colascione <dancol@dancol.org>
Cc: Alan Mackenzie <acm@muc.de>, 16969@debbugs.gnu.org
Subject: bug#16969: Isearch: lazy-highlight face sometimes has foreground and background the same colour.
Date: Sun, 9 Mar 2014 15:20:48 -0700 (PDT)	[thread overview]
Message-ID: <2fe64fe3-c241-46a9-87d3-2c80b8bc1d9e@default> (raw)
In-Reply-To: <87siqroxnr.fsf@mail.jurta.org>

> 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.





  reply	other threads:[~2014-03-09 22:20 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-08 18:35 bug#16969: Isearch: lazy-highlight face sometimes has foreground and background the same colour Alan Mackenzie
2014-03-08 21:55 ` Juri Linkov
2014-03-08 23:01   ` Alan Mackenzie
2014-03-09  9:37     ` Jan Djärv
2014-03-09 13:25       ` Alan Mackenzie
2014-03-09  0:45   ` Daniel Colascione
2014-03-09 21:43     ` Juri Linkov
2014-03-09 22:20       ` Drew Adams [this message]
2014-03-10  3:01     ` Stefan Monnier
2022-02-05 22:46   ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2fe64fe3-c241-46a9-87d3-2c80b8bc1d9e@default \
    --to=drew.adams@oracle.com \
    --cc=16969@debbugs.gnu.org \
    --cc=acm@muc.de \
    --cc=dancol@dancol.org \
    --cc=juri@jurta.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.