[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#51605: Fwd: [PATCH] Update themes to support mode-line
From: |
Stefan Kangas |
Subject: |
bug#51605: Fwd: [PATCH] Update themes to support mode-line |
Date: |
Sun, 5 Dec 2021 19:05:36 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Stefan Kangas <stefan@marxist.se> writes:
> I'm forwarding this patch to the bug tracker, that AFAICT was never
> installed or followed up on. The first patch saw some discussion here:
>
> https://lists.gnu.org/archive/html/emacs-devel/2020-12/msg00993.html
>
> -------------------- Start of forwarded message --------------------
[snip]
> a second attempt at including minimal line-number support in the
> default Emacs themes.
>
> This has made me think that maybe the best place to make the
> line-number-face inherit from fringe would be in the line-number
> code... This might mean that we need an agreement that the fringe and
> the line-number are things that are semantically similar or, at least,
> different enough from the buffer.
AFAICT, the point of this patch is essentially changing the line-number
face to inherit the fringe face and line-number-current to have the
default foreground (to make the current line more visible).
I think this makes sense and looks good. I checked, and this seems to
be what VSCode is doing too, so I don't think it's a controversial
design choice.
So I'd suggest installing the below diff, and then an adapted version of
the proposed patch from Pedro Andres Aranda Gutierrez. Comments?
diff --git a/lisp/faces.el b/lisp/faces.el
index 9caba3a11b..16125950d1 100644
--- a/lisp/faces.el
+++ b/lisp/faces.el
@@ -2494,7 +2494,7 @@ trailing-whitespace
;; Definition stolen from linum.el.
(defface line-number
- '((t :inherit (shadow default)))
+ '((t :inherit (fringe shadow default)))
"Face for displaying line numbers.
This face is used when `display-line-numbers' is non-nil.
@@ -2507,7 +2507,7 @@ line-number
:group 'display-line-numbers)
(defface line-number-current-line
- '((t :inherit line-number))
+ '((t :foreground "black" :inherit line-number))
"Face for displaying the current line number.
This face is used when `display-line-numbers' is non-nil.
- bug#51605: Fwd: [PATCH] Update themes to support mode-line,
Stefan Kangas <=
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Eli Zaretskii, 2021/12/05
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Stefan Kangas, 2021/12/05
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Eli Zaretskii, 2021/12/06
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Stefan Kangas, 2021/12/06
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Eli Zaretskii, 2021/12/06
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Stefan Kangas, 2021/12/06
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Eli Zaretskii, 2021/12/06
- bug#51605: Fwd: [PATCH] Update themes to support mode-line, Juri Linkov, 2021/12/06