emacs-devel
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: New custom faces for rmail.el

From: Glenn Morris
Subject: Re: New custom faces for rmail.el
Date: Tue, 12 Feb 2008 00:46:20 -0500
User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/) 
Bastien wrote:

> +(defface rmail-header-name-face
> +  '((t (:inherit font-lock-function-name-face)))
> +  "Face to use for highlighting the header names."
> +  :group 'rmail-headers
> +  :version "22.1")

"23.1"

Face names should not end in -face.

> +(defcustom rmail-header-name 'rmail-header-name-face "\
> +*Face to use for highlighting the header names."
> +  :type '(choice (const :tag "Default" nil)
> +              face)
> +  :group 'rmail-headers)

There's no need for a variable, just the face is enough.
reply via email to
[Prev in Thread] Current Thread [Next in Thread]