bug#43300: [PATCH] Make M-x show new commands for obsolete aliases

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#43300: [PATCH] Make M-x show new commands for obsolete aliases

Stefan Kangas
Severity: wishlist

Please find attached two patches that tweaks how M-x
(execute-extended-command) works:

1. Show obsolete commands, and give their new name as an annotation.

M-x recently got the capability to show the keybindings for commands in
an annotation (in parenthesis after the command name).  This patch makes
it show new names for obsolete aliases in the same way, instead of just
refusing to show them.  This should help users ease into the new name
less abruptly and disruptively, instead of having it just disappearing
from M-x and be nowhere to be found.

As an added bonus, we could more confidently mark an alias such as
`display-time-world' obsolete (without worrying that it will just be
gone in the next release).

(Yes, if you type out the full name, you can still call it, but chances
are that many users are very reliant on tab completion and will assume
that it's just gone if it doesn't show up.)

2. Show the function that aliases point to in the same way.

0001-Make-M-x-show-obsolete-commands.patch (2K) Download Attachment
0002-Make-M-x-show-what-aliases-point-to.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#43300: [PATCH] Make M-x show new commands for obsolete aliases

Lars Ingebrigtsen
Stefan Kangas <[hidden email]> writes:

> As an added bonus, we could more confidently mark an alias such as
> `display-time-world' obsolete (without worrying that it will just be
> gone in the next release).

Sounds like a good idea.

> 2. Show the function that aliases point to in the same way.

Makes sense.

> * lisp/simple.el (read-extended-command): Don't hide obsolete
> commands.
> (read-extended-command--annotation): Show an annotation for obsolete
> commands that says what their new name is.

I haven't tried the patch, but it looks good to me.

--
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



Reply | Threaded
Open this post in threaded view
|

bug#43300: [PATCH] Make M-x show new commands for obsolete aliases

Stefan Kangas
Lars Ingebrigtsen <[hidden email]> writes:

> I haven't tried the patch, but it looks good to me.

Thanks.  Pushed to master as commit 1b0a922a19 and 06d86b954d.