bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#45502: [PATCH] Prettier key bindings in NS menu entries


From: Mattias Engdegård
Subject: bug#45502: [PATCH] Prettier key bindings in NS menu entries
Date: Tue, 29 Dec 2020 13:02:21 +0100

28 dec. 2020 kl. 19.36 skrev Alan Third <alan@idiocy.org>:

> I'm not sure either, but I guess the tabstop thing would look
> something like: [...]

Actually seems to work! (In your place I would feign a complete lack of 
surprise.)
Resulting patch attached.

28 dec. 2020 kl. 23.46 skrev Daniel Martín <mardani29@yahoo.es>:

> Thanks for the patch! It crashed Emacs when I tried to open the Gnus
> menu bar (the Gnus menu bar is an extreme case with lots of bindings).

Confirmed, but that is unrelated to my patch. Alan, will you have a look?

> I'm not sure if left-alignment or right-alignment would be better.  To
> improve visuals, Apple seems to align with respect to the ⌘ symbol, but
> that doesn't fit Emacs well because there's no single modifier that is
> used in almost every keybinding (some use Control, some Meta).  Also,
> it's not uncommon in Emacs to have keybindings that are a couple of
> keymaps deep.

Right; it's easy to use either left or right alignment for the bindings. (I 
think we all agree that they should be kept in a separate column to the right 
of the menu strings in either case.) I'm going to experiment with translating 
modifiers and keys to the standard symbols. Not sure how to deal with modifiers 
that are unavailable, such as s-k when no Super modifier is available.

It might be a good idea to normalise how key bindings are displayed on all 
platforms to some extent. For example, <C-return> is better written C-<return> 
or C-RET, <M-backspace> is better as M-<backspace> or M-DEL (although not 
necessarily exactly the same thing, it's a bit muddy).

Of course <> are terrible as angle brackets; we typically want ‹› or ⟨⟩ 
depending on context and purpose.

Attachment: 0001-Right-justify-keys-in-NS-menu-entries-bug-45502.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]