* isearch-lazy-highlight-face
@ 2005-06-07 7:13 Miles Bader
2005-06-07 14:28 ` isearch-lazy-highlight-face Richard Stallman
2005-06-07 14:45 ` isearch-lazy-highlight-face Juri Linkov
0 siblings, 2 replies; 6+ messages in thread
From: Miles Bader @ 2005-06-07 7:13 UTC (permalink / raw)
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?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: isearch-lazy-highlight-face
2005-06-07 7:13 isearch-lazy-highlight-face Miles Bader
@ 2005-06-07 14:28 ` Richard Stallman
2005-06-08 0:59 ` isearch-lazy-highlight-face Juri Linkov
2005-06-07 14:45 ` isearch-lazy-highlight-face Juri Linkov
1 sibling, 1 reply; 6+ messages in thread
From: Richard Stallman @ 2005-06-07 14:28 UTC (permalink / raw)
Cc: emacs-devel
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: isearch-lazy-highlight-face
2005-06-07 14:28 ` isearch-lazy-highlight-face Richard Stallman
@ 2005-06-08 0:59 ` Juri Linkov
2005-06-09 8:28 ` isearch-lazy-highlight-face Richard Stallman
0 siblings, 1 reply; 6+ messages in thread
From: Juri Linkov @ 2005-06-08 0:59 UTC (permalink / raw)
Cc: emacs-devel, miles
> 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/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: isearch-lazy-highlight-face
2005-06-08 0:59 ` isearch-lazy-highlight-face Juri Linkov
@ 2005-06-09 8:28 ` Richard Stallman
0 siblings, 0 replies; 6+ messages in thread
From: Richard Stallman @ 2005-06-09 8:28 UTC (permalink / raw)
Cc: miles, emacs-devel
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: isearch-lazy-highlight-face
2005-06-07 7:13 isearch-lazy-highlight-face Miles Bader
2005-06-07 14:28 ` isearch-lazy-highlight-face Richard Stallman
@ 2005-06-07 14:45 ` Juri Linkov
2005-06-11 23:17 ` isearch-lazy-highlight-face Richard Stallman
1 sibling, 1 reply; 6+ messages in thread
From: Juri Linkov @ 2005-06-07 14:45 UTC (permalink / raw)
Cc: emacs-devel
> 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/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2005-06-11 23:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-06-07 7:13 isearch-lazy-highlight-face Miles Bader
2005-06-07 14:28 ` isearch-lazy-highlight-face Richard Stallman
2005-06-08 0:59 ` isearch-lazy-highlight-face Juri Linkov
2005-06-09 8:28 ` isearch-lazy-highlight-face Richard Stallman
2005-06-07 14:45 ` isearch-lazy-highlight-face Juri Linkov
2005-06-11 23:17 ` isearch-lazy-highlight-face Richard Stallman
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.