GNU bug report logs - #52293
29.0.50; [PATCH] Prevent further cases of duplicated separators in context menus

Previous Next

Package: emacs;

Reported by: Jim Porter <jporterbugs <at> gmail.com>

Date: Sun, 5 Dec 2021 05:59:01 UTC

Severity: normal

Tags: patch

Fixed in version 29.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


Message #65 received at 52293 <at> debbugs.gnu.org (full text, mbox):

From: Jim Porter <jporterbugs <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 52293 <at> debbugs.gnu.org, juri <at> linkov.net
Subject: Re: bug#52293: 29.0.50; [PATCH v3] Prevent further cases of
 duplicated separators in context menus
Date: Sun, 12 Dec 2021 13:59:16 -0800
On 12/12/2021 12:43 PM, Eli Zaretskii wrote:
>> Cc: 52293 <at> debbugs.gnu.org, juri <at> linkov.net
>> From: Jim Porter <jporterbugs <at> gmail.com>
>> Date: Sun, 12 Dec 2021 12:27:34 -0800
>>
>> On 12/11/2021 11:02 PM, Eli Zaretskii wrote:
>>> Are you saying that we will be recommending a convention for separator
>>> names only for menus popped up in context-menu-mode?  Does it make
>>> sense to have such a specialized convention?  Isn't it possible to
>>> show context menus outside of the context-menu-mode?
>>
>> No, I think this convention should be recommended for separators used
>> anywhere in Emacs going forward.
> 
> Ok, so then the effect of this change is wide, as I envisioned, and
> compatibility considerations still apply.

I'm also ok with applying this naming convention *only* to context menus 
and not treating this as a general guideline; that would still make it 
easier to remember the names of each context menu separator without 
having to double-check the source code.

I don't even have a problem with closing that bug (bug#52286) as 
wontfix, since it's just a minor issue anyway. It's really not a big 
deal if the naming is inconsistent, so long as the names are still unique.

I think this patch, which adds `top-separator' to help the 
de-duplication logic work better, would be useful to apply regardless of 
the decision on how the separators should be named.




This bug report was last modified 3 years and 136 days ago.

Previous Next


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