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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Manuel Giraud <manuel <at> ledu-giraud.fr>
Subject: bug#73734: closed (Re: bug#73734: [PATCH] Fix tmm menu layout)
Date: Sat, 19 Oct 2024 08:37:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#73734: [PATCH] Fix tmm menu layout

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 73734 <at> debbugs.gnu.org.

-- 
73734: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=73734
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Eli Zaretskii <eliz <at> gnu.org>
To: Manuel Giraud <manuel <at> ledu-giraud.fr>
Cc: rudalics <at> gmx.at, 73734-done <at> debbugs.gnu.org
Subject: Re: bug#73734: [PATCH] Fix tmm menu layout
Date: Sat, 19 Oct 2024 11:35:43 +0300

> From: Manuel Giraud <manuel <at> ledu-giraud.fr>
> Cc: rudalics <at> gmx.at,  73734 <at> debbugs.gnu.org
> Date: Tue, 15 Oct 2024 10:37:08 +0200
> 
> Hi,
> 
> I don't think there is fix that will work for everyone on this issue.  I
> thought that this was due to the fixed 30 value for colwidth but even
> when always using computed value (using frame-width for instance), there
> is always some frame-width that will exhibit the « confusion ».
> 
> I have tested with set-frame-width and decrementing it one by one.
> 
> My conclusion is that the last patch could still go in but does not
> really solve the original issue.

OK, I've now installed that patch on master, and I'm closing this bug.

Thanks.

[Message part 3 (message/rfc822, inline)]

From: Manuel Giraud <manuel <at> ledu-giraud.fr>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Fix tmm menu layout
Date: Thu, 10 Oct 2024 15:53:12 +0200

[Message part 4 (text/plain, inline)]

Tags: patch

Hi,

Depending on the frame and font size, some tmm menus could be somewhat
hard to read.  Here is what I get with the "Mark" menu from a Dired
buffer:

[2024-10-10T15:25+0200.png (image/png, inline)]

[Message part 6 (text/plain, inline)]

The selection character could be mistaken for part of another
keybinding.

This patch tries to solves this issue.  I tested it a bit with different
frame sizes and different fonts and get good enough results.

Here, how I justify the modification of `colwidth':

      - I don't think we need the "(min 30)" part since, if the frame is
        wide enough, we always get a colwidth of 30.
        
      - I don't think "(window-width)" is what we need since, by
        default, the *Completions* buffer will use the full frame width.

      - I tried to set the layout on three columns because "why not?"

While here, I had some clean up.

In GNU Emacs 31.0.50 (build 18, x86_64-unknown-openbsd7.6, X toolkit) of
 2024-10-09 built on computer
Repository revision: 9ed82c26a36caf9a9a85779cbb3a58b7ccd3dffb
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101013
System Description: OpenBSD computer 7.6 GENERIC.MP#344 amd64

Configured using:
 'configure CC=egcc CPPFLAGS=-I/usr/local/include
 LDFLAGS=-L/usr/local/lib MAKEINFO=gmakeinfo --prefix=/home/manuel/emacs
 --bindir=/home/manuel/bin --with-x-toolkit=lucid
 --with-toolkit-scroll-bars=no --without-cairo
 --without-compress-install'

[0001-Fix-tmm-menu-layout.patch (text/patch, attachment)]

[Message part 8 (text/plain, inline)]

-- 
Manuel Giraud

Previous Next

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