unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master daf008e82b: Note caveat about remapping basic faces
       [not found] ` <20220214103905.9E8C8C00891@vcs2.savannah.gnu.org>
@ 2022-02-14 10:56   ` Po Lu
  2022-02-14 11:07     ` Lars Ingebrigtsen
  2022-02-14 13:17     ` martin rudalics
  0 siblings, 2 replies; 7+ messages in thread
From: Po Lu @ 2022-02-14 10:56 UTC (permalink / raw)
  To: emacs-devel; +Cc: Lars Ingebrigtsen

Lars Ingebrigtsen <larsi@gnus.org> writes:

> +Note that face remapping does not work reliably parent faces of
> +@dnf{basic faces}.  (These are the faces that are used in mode lines,
> +header lines and the like.)  For instance, @code{mode-line-inactive}
> +inherits from @code{mode-line}, but remapping @code{mode-line} won't
> +normally have the desired effect on @code{mode-line-inactive}.
> +Instead you have to remap @code{mode-line-inactive} directly.

Do we have a list of all the basic faces somewhere else in the manual?



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 10:56   ` master daf008e82b: Note caveat about remapping basic faces Po Lu
@ 2022-02-14 11:07     ` Lars Ingebrigtsen
  2022-02-14 11:20       ` Po Lu
  2022-02-14 13:17     ` martin rudalics
  1 sibling, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2022-02-14 11:07 UTC (permalink / raw)
  To: Po Lu; +Cc: emacs-devel

Po Lu <luangruo@yahoo.com> writes:

> Do we have a list of all the basic faces somewhere else in the manual?

No, and I'm not sure we want to document it.  I think explaining that
there are problems here is sufficient.

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 11:07     ` Lars Ingebrigtsen
@ 2022-02-14 11:20       ` Po Lu
  2022-02-14 17:25         ` Eli Zaretskii
  0 siblings, 1 reply; 7+ messages in thread
From: Po Lu @ 2022-02-14 11:20 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen <larsi@gnus.org> writes:

> No, and I'm not sure we want to document it.  I think explaining that
> there are problems here is sufficient.

I think people will run into the problem (or potentially unrelated
problems) with faces other than `mode-line', so it would be good to
document which ones the problem can occur with.

Thanks.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 10:56   ` master daf008e82b: Note caveat about remapping basic faces Po Lu
  2022-02-14 11:07     ` Lars Ingebrigtsen
@ 2022-02-14 13:17     ` martin rudalics
  2022-02-14 14:16       ` Lars Ingebrigtsen
  1 sibling, 1 reply; 7+ messages in thread
From: martin rudalics @ 2022-02-14 13:17 UTC (permalink / raw)
  To: Po Lu, emacs-devel; +Cc: Lars Ingebrigtsen

>> +Note that face remapping does not work reliably parent faces of

Is the "for" missing in the original text too?

martin




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 13:17     ` martin rudalics
@ 2022-02-14 14:16       ` Lars Ingebrigtsen
  2022-02-14 18:12         ` martin rudalics
  0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2022-02-14 14:16 UTC (permalink / raw)
  To: martin rudalics; +Cc: Po Lu, emacs-devel

martin rudalics <rudalics@gmx.at> writes:

>>> +Note that face remapping does not work reliably parent faces of
>
> Is the "for" missing in the original text too?

Original text?

Anyway, fixed now.

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 11:20       ` Po Lu
@ 2022-02-14 17:25         ` Eli Zaretskii
  0 siblings, 0 replies; 7+ messages in thread
From: Eli Zaretskii @ 2022-02-14 17:25 UTC (permalink / raw)
  To: Po Lu; +Cc: larsi, emacs-devel

> From: Po Lu <luangruo@yahoo.com>
> Cc: emacs-devel@gnu.org
> Date: Mon, 14 Feb 2022 19:20:40 +0800
> 
> Lars Ingebrigtsen <larsi@gnus.org> writes:
> 
> > No, and I'm not sure we want to document it.  I think explaining that
> > there are problems here is sufficient.
> 
> I think people will run into the problem (or potentially unrelated
> problems) with faces other than `mode-line', so it would be good to
> document which ones the problem can occur with.

Actually, we do have a section that describes the basic faces, so I
added a cross-reference there from the new text added by Lars.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: master daf008e82b: Note caveat about remapping basic faces
  2022-02-14 14:16       ` Lars Ingebrigtsen
@ 2022-02-14 18:12         ` martin rudalics
  0 siblings, 0 replies; 7+ messages in thread
From: martin rudalics @ 2022-02-14 18:12 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: Po Lu, emacs-devel

 >>>> +Note that face remapping does not work reliably parent faces of
 >>
 >> Is the "for" missing in the original text too?
 >
 > Original text?

I only read Po Lu's quote of that text.  I didn't look in the sources.

 > Anyway, fixed now.

Fine, martin



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-02-14 18:12 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <164483514532.13188.6767538885997396285@vcs2.savannah.gnu.org>
     [not found] ` <20220214103905.9E8C8C00891@vcs2.savannah.gnu.org>
2022-02-14 10:56   ` master daf008e82b: Note caveat about remapping basic faces Po Lu
2022-02-14 11:07     ` Lars Ingebrigtsen
2022-02-14 11:20       ` Po Lu
2022-02-14 17:25         ` Eli Zaretskii
2022-02-14 13:17     ` martin rudalics
2022-02-14 14:16       ` Lars Ingebrigtsen
2022-02-14 18:12         ` martin rudalics

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).