emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Customizing Buffer-menu-buffer-face


From: Richard Stallman
Subject: Re: Customizing Buffer-menu-buffer-face
Date: Sat, 14 May 2005 20:21:18 -0400

    Is there any reason why `Buffer-menu-buffer-face' is customized in the
    `font-lock-highlighting-faces' group?  The current choice (i) breaks the
    convention that all faces in this group are prefixed with "font-lock-"
    and (ii) falsely implies that this face is used by font-lock.

With the existing code, there is no reason.  However, that's due
to a bug.  Fixing the bug, with the patch below, it becomes
the case that M-x font-lock-mode controls whether this face
is actually used.  And then it makes sense to put this in the
group font-lock-highlighting-faces.

However, it should also be in the group Buffer-menu.


*** buff-menu.el        09 Apr 2005 19:50:08 -0400      1.80
--- buff-menu.el        14 May 2005 14:06:54 -0400      
***************
*** 77,82 ****
--- 77,83 ----
  (defface Buffer-menu-buffer-face
    '((t (:weight bold)))
    "Face used to highlight buffer name."
+   :group 'Buffer-menu
    :group 'font-lock-highlighting-faces)
  
  (defcustom Buffer-menu-buffer+size-width 26
***************
*** 754,760 ****
                                         (int-to-string (nth 3 buffer))
                                         `(buffer-name ,(nth 2 buffer)
                                           buffer ,(car buffer)
!                                          face Buffer-menu-buffer-face
                                           mouse-face highlight
                                           help-echo "mouse-2: select this 
buffer"))
                "  "
--- 755,761 ----
                                         (int-to-string (nth 3 buffer))
                                         `(buffer-name ,(nth 2 buffer)
                                           buffer ,(car buffer)
!                                          font-lock-face 
Buffer-menu-buffer-face
                                           mouse-face highlight
                                           help-echo "mouse-2: select this 
buffer"))
                "  "




reply via email to

[Prev in Thread] Current Thread [Next in Thread]