[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#61567: 30.0.50; Define display-time-time-and-date-indicator face
From: |
Robert Pluim |
Subject: |
bug#61567: 30.0.50; Define display-time-time-and-date-indicator face |
Date: |
Fri, 17 Feb 2023 11:58:25 +0100 |
>>>>> On Fri, 17 Feb 2023 12:40:22 +0200, Protesilaos Stavrou
>>>>> <info@protesilaos.com> said:
>> From: Robert Pluim <rpluim@gmail.com>
>> Date: Fri, 17 Feb 2023 11:31:08 +0100
>>
>>>>>>> On Fri, 17 Feb 2023 08:37:32 +0200, Eli Zaretskii <eliz@gnu.org>
said:
Eli> ** New face 'display-time-time-and-date-indicator'.
Eli> This is used for displaying the time and date components of
Eli> 'display-time-mode'.
>>
Eli> should be enough.
>>
>> >> +(defface display-time-time-and-date-indicator nil
>> >> + "Face of the `display-time-format'."
Eli> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Eli> "Face for `display-time-format'."
>>
>> Thatʼs a very repetitive name. How about just
>> `display-time-indicator-face' (although I have a vague memory of there
>> being a guideline to not end faces in -face)
Protesilaos> Indeed, the name is awkard. I also remember that use of
"-face" is
Protesilaos> discouraged, but could not find the resource for it.
I canʼt find anything in the elisp manual. But I do note that all the
font-lock faces end in '-face'
Protesilaos> My initial idea was to use "display-time-face", which would
also be
Protesilaos> symmetric with "display-time-mail-face". Note though that the
latter is
Protesilaos> defined as a 'defcustom', so I wasn't sure what the right
approach is.
`defface' defined faces are customizable, so I donʼt think it matters.
Robert
--
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Protesilaos Stavrou, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Eli Zaretskii, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Protesilaos Stavrou, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Robert Pluim, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Protesilaos Stavrou, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face,
Robert Pluim <=
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Protesilaos Stavrou, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Robert Pluim, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Eli Zaretskii, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Robert Pluim, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Protesilaos Stavrou, 2023/02/18
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Basil Contovounesios, 2023/02/17
- bug#61567: 30.0.50; Define display-time-time-and-date-indicator face, Drew Adams, 2023/02/17