GNU bug report logs -
#73862
[PATCH] Add `header-line-active` and `header-line-inactive` faces.
Previous Next
Reported by: trevor.m.murphy <at> gmail.com
Date: Fri, 18 Oct 2024 12:58:02 UTC
Severity: wishlist
Tags: patch
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
Message #117 received at 73862 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
On Sun, Dec 08, 2024 at 8:29 AM, Eshel Yaron <me <at> eshelyaron.com> wrote:
> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> From: Eshel Yaron <me <at> eshelyaron.com>
> Cc: Aaron Jensen <aaronjensen <at> gmail.com>, monnier <at> iro.umontreal.ca,
> trevor.m.murphy <at> gmail.com, 73862 <at> debbugs.gnu.org
> Date: Sun, 08 Dec 2024 15:11:07 +0100
>
> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> We also haven't yet heard from Eshel.
>
> I don't have any concrete suggestions at this point. It's surely a
> complicated situation.
>
> Did you try the patch I posted, and if you did, did it resolve your
> original problems?
>
> I did now, and it doesn't entirely solve the problem: with this patch,
> after remapping header-line in one buffer, the remapping now works for that
> buffer as expected, but it still also "spreads" to other buffers, which is
> unexpected.
>
I see the leak now as well. When I tested it at first, I did not see it
perhaps because I did not switch back to the original window and trigger a
computation of the basic faces. That's what causes the leak.
Steps:
(setq header-line-format "Some header")
(face-remap-set-base 'header-line 'highlight)
(switch-to-buffer-other-window "new")
;; In new buffer/window:
(setq header-line-format "Some header")
(other-window)
;; In original buffer/window:
(set-face-attribute 'default nil :height (+ (face-attribute 'default
:height) 10))
Aaron
[Message part 2 (text/html, inline)]
This bug report was last modified 214 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.