isearch-lazy-highlight-face

isearch-lazy-highlight-face

[Miles Bader]

I'd like to rename this face to just `isearch-lazy-highlight' before the
release (with a face alias for compability).  Any objections?

Thanks,

-Miles
-- 
Come now, if we were really planning to harm you, would we be waiting here,
 beside the path, in the very darkest part of the forest?

Re: isearch-lazy-highlight-face

[Richard Stallman]

    I'd like to rename this face to just `isearch-lazy-highlight' before the
    release (with a face alias for compability).  Any objections?

Ok.  Since this face was first created a few months ago, I see no need
for the alias.

Re: isearch-lazy-highlight-face

[Juri Linkov]

> I'd like to rename this face to just `isearch-lazy-highlight' before the
> release (with a face alias for compability).  Any objections?

`isearch-lazy-highlight-face' was already renamed to `lazy-highlight'
in January 2005.  `isearch-lazy-highlight-face' can remain as an alias
for indefinite time.

On a related topic, I'd like to propose a slight improvement for the
background color of `isearch' face.  Currently background and
foreground colors are both light which makes the highlighted search
text hardly readable.  I'm aware of the comment in the face definition
saying why the background color must not be dark.  Surely, "magenta4"
used as a foreground color under the cursor would be too dark when the
color of the cursor is black.  What I propose is to make the background
color only slightly darker by replacing "magenta2" with "magenta3".
The character under the cursor is still well visible with "magenta3"
used for its foreground color.

 (defface isearch
   '((((class color) (min-colors 88) (background light))
      ;; The background must not be too dark, for that means
      ;; the character is hard to see when the cursor is there.
-     (:background "magenta2" :foreground "lightskyblue1"))
+     (:background "magenta3" :foreground "lightskyblue1"))
     (((class color) (min-colors 88) (background dark))
      (:background "palevioletred2" :foreground "brown4"))
     (((class color) (min-colors 16))
      (:background "magenta4" :foreground "cyan1"))
     (((class color) (min-colors 8))
      (:background "magenta4" :foreground "cyan1"))
     (t (:inverse-video t)))
   "Face for highlighting Isearch matches."
   :group 'isearch)

-- 
Juri Linkov
http://www.jurta.org/emacs/

Re: isearch-lazy-highlight-face

[Juri Linkov]

>     I'd like to rename this face to just `isearch-lazy-highlight' before the
>     release (with a face alias for compability).  Any objections?
>
> Ok.  Since this face was first created a few months ago, I see no need
> for the alias.

ChangeLog says that it was created a few _years_ ago, so the alias
is needed for it.  It seems all is correct already with
isearch-lazy-highlight-face.  Isn't it?

As for removing -face suffix from face names, 

    C-u M-x list-faces-display RET -face$ RET

still displays too many unconventional face names waiting to be
renamed.

-- 
Juri Linkov
http://www.jurta.org/emacs/

Re: isearch-lazy-highlight-face

[Richard Stallman]

    ChangeLog says that it was created a few _years_ ago, so the alias
    is needed for it.  It seems all is correct already with
    isearch-lazy-highlight-face.  Isn't it?

Based on this info, I think yes.

    As for removing -face suffix from face names, 

	C-u M-x list-faces-display RET -face$ RET

    still displays too many unconventional face names waiting to be
    renamed.

If you want to rename some, and make aliases for them,
that can't hurt.

Re: isearch-lazy-highlight-face

[Richard Stallman]

      What I propose is to make the background
    color only slightly darker by replacing "magenta2" with "magenta3".
    The character under the cursor is still well visible with "magenta3"
    used for its foreground color.

I tried this, and I have no opinion about the change.