GNU bug report logs - #73734
[PATCH] Fix tmm menu layout

Previous Next

Package: emacs;

Reported by: Manuel Giraud <manuel <at> ledu-giraud.fr>

Date: Thu, 10 Oct 2024 13:54:01 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Manuel Giraud <manuel <at> ledu-giraud.fr>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rudalics <at> gmx.at, 73734 <at> debbugs.gnu.org
Subject: bug#73734: [PATCH] Fix tmm menu layout
Date: Sat, 12 Oct 2024 16:10:10 +0200

Eli Zaretskii <eliz <at> gnu.org> writes:

[...]

> This doesn't align bindings if a variable-pitch font is used, but
> it still produces a better display than the current code does when
> variable-pitch font is the default font.

Thanks, yes this is better to remove the confusion (but sometimes misses
bindings alignment).  To finalize this patch, I still have two questions
left:

        - What is the purpose of the "(min 30)"?  Because if the window
          is large enough this will always return 30.

        - Should we use frame-width instead of window-width?  It seems
          that when this function is called (get-buffer-window
          "*Completions*") always returns nil.
-- 
Manuel Giraud
This bug report was last modified 270 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.