GNU bug report logs -
#62279
Add an option to restore C-h b "Prefix Command" output
Previous Next
Reported by: Bob Rogers <rogers <at> rgrjr.com>
Date: Sun, 19 Mar 2023 19:27:01 UTC
Severity: normal
Tags: patch
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 62279 in the body.
You can then email your comments to 62279 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#62279
; Package
emacs
.
(Sun, 19 Mar 2023 19:27:01 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Bob Rogers <rogers <at> rgrjr.com>
:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org
.
(Sun, 19 Mar 2023 19:27:01 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Tags: patch
Pursuant to the 'No more "Prefix Command" in C-h m' discussion in the
emacs-devel list.
-- Bob Rogers
http://www.rgrjr.com/
In GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version
3.24.34, cairo version 1.16.0) of 2023-03-13 built on orion
Repository revision: a79d209748857c7d63ac48a7172a89e5ad1e752b
Repository branch: rgr-smtpmail-env-from
Windowing system distributor 'The X.Org Foundation', version 11.0.12003000
System Description: openSUSE Leap 15.4
Configured using:
'configure --with-dbus=no --with-gsettings=no --with-gif=ifavailable
--with-tiff=no --with-gnutls=yes --with-gconf=no'
[describe-bindings-show-prefixes-2.patch (text/patch, attachment)]
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#62279
; Package
emacs
.
(Sun, 19 Mar 2023 19:36:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 62279 <at> debbugs.gnu.org (full text, mbox):
> From: Bob Rogers <rogers <at> rgrjr.com>
> Date: Sun, 19 Mar 2023 12:26:46 -0700
>
> Pursuant to the 'No more "Prefix Command" in C-h m' discussion in the
> emacs-devel list.
Thanks.
> --- a/etc/NEWS
> +++ b/etc/NEWS
> @@ -61,6 +61,11 @@ This allows the user to customize the prompt that is appended by
> This is used for displaying the time and date components of
> 'display-time-mode'.
>
> +** New user option 'describe-bindings-show-prefixes'.
> +If true (the default is false), restores the "Prefix Command" entries
We use "If non-nil", not "If true". And similarly for "false".
> +in 'describe-binding' ('C-h b') output for keys that are prefixes, the
> +details of which are shown later (regardless of this option setting).
This NEWS entry should go to where the change in "C-h b"s behavior is
described in NEWS (on emacs-29 branch), and should be reworded to tell
users how to get the previous behavior back ("To get the previous
behavior back, customize ...").
> +(defcustom describe-bindings-show-prefixes nil
> + "Non-nil shows prefixes in the output buffer of `describe-bindings'."
> + :type 'boolean
> + :group 'help
> + :version "30.1")
This should be "29.1". Since the behavior change was introduced in
Emacs 29, we need to add this option in the same version.
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#62279
; Package
emacs
.
(Sun, 19 Mar 2023 20:00:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 62279 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
From: Eli Zaretskii <eliz <at> gnu.org>
Date: Sun, 19 Mar 2023 21:35:37 +0200
> From: Bob Rogers <rogers <at> rgrjr.com>
> Date: Sun, 19 Mar 2023 12:26:46 -0700
>
> Pursuant to the 'No more "Prefix Command" in C-h m' discussion in the
> emacs-devel list.
Thanks.
> --- a/etc/NEWS
> +++ b/etc/NEWS
> @@ -61,6 +61,11 @@ This allows the user to customize the prompt that is appended by
> This is used for displaying the time and date components of
> 'display-time-mode'.
>
> +** New user option 'describe-bindings-show-prefixes'.
> +If true (the default is false), restores the "Prefix Command" entries
We use "If non-nil", not "If true". And similarly for "false".
Guess I've been thinking too much in Perl lately. ;-}
> +in 'describe-binding' ('C-h b') output for keys that are prefixes, the
> +details of which are shown later (regardless of this option setting).
This NEWS entry should go to where the change in "C-h b"s behavior is
described in NEWS (on emacs-29 branch), and should be reworded to tell
users how to get the previous behavior back ("To get the previous
behavior back, customize ...").
OK. I also subordinated it to the "Keymap descriptions have changed"
entry, since that seemed apt.
> +(defcustom describe-bindings-show-prefixes nil
> + "Non-nil shows prefixes in the output buffer of `describe-bindings'."
> + :type 'boolean
> + :group 'help
> + :version "30.1")
This should be "29.1". Since the behavior change was introduced in
Emacs 29, we need to add this option in the same version.
OK, I wasn't sure Lars' change actually made it into 29.
-- Bob
[describe-bindings-show-prefixes-3.patch (text/x-patch, inline)]
Add an option to restore C-h b "Prefix Command" output
* lisp/help.el (describe-bindings-show-prefixes): Option to enable prefixes.
(describe-map): If true, restore "Prefix Command" output.
* etc/NEWS: Describe the new option.
diff --git a/etc/NEWS b/etc/NEWS
index a5f9f8c6ec1..c4e16221d72 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -3563,6 +3563,10 @@ keymap descriptions have changed. In particular, prefix commands are
not output at all, and instead of "??" for closures/functions,
"[closure]"/"[lambda]" is output.
+*** New user option 'describe-bindings-show-prefixes'.
+To reinstate the "Prefix Command" entries in key binding output for
+keys that are prefixes, set this option non-nil (the default is nil).
+
---
** 'downcase' details have changed slightly.
In certain locales, changing the case of an ASCII-range character may
diff --git a/lisp/help.el b/lisp/help.el
index 83be85b1ee4..3e450fe0763 100644
--- a/lisp/help.el
+++ b/lisp/help.el
@@ -717,6 +717,12 @@ describe-bindings-outline
:group 'help
:version "29.1")
+(defcustom describe-bindings-show-prefixes nil
+ "Non-nil shows prefixes in the output buffer of `describe-bindings'."
+ :type 'boolean
+ :group 'help
+ :version "29.1")
+
(declare-function outline-hide-subtree "outline")
(defun describe-bindings (&optional prefix buffer)
@@ -1699,6 +1705,7 @@ describe-map
(setq vect (cdr vect))
(setq end (caar vect))))
(when (or (not (eq start end))
+ describe-bindings-show-prefixes
;; Don't output keymap prefixes.
(not (keymapp definition)))
(when first
Reply sent
to
Eli Zaretskii <eliz <at> gnu.org>
:
You have taken responsibility.
(Thu, 23 Mar 2023 08:34:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Bob Rogers <rogers <at> rgrjr.com>
:
bug acknowledged by developer.
(Thu, 23 Mar 2023 08:34:02 GMT)
Full text and
rfc822 format available.
Message #16 received at 62279-done <at> debbugs.gnu.org (full text, mbox):
> From: Bob Rogers <rogers <at> rgrjr.com>
> Date: Sun, 19 Mar 2023 12:59:45 -0700
> CC: 62279 <at> debbugs.gnu.org
>
> From: Eli Zaretskii <eliz <at> gnu.org>
> Date: Sun, 19 Mar 2023 21:35:37 +0200
>
> > From: Bob Rogers <rogers <at> rgrjr.com>
> > Date: Sun, 19 Mar 2023 12:26:46 -0700
> >
> > Pursuant to the 'No more "Prefix Command" in C-h m' discussion in the
> > emacs-devel list.
>
> Thanks.
>
> > --- a/etc/NEWS
> > +++ b/etc/NEWS
> > @@ -61,6 +61,11 @@ This allows the user to customize the prompt that is appended by
> > This is used for displaying the time and date components of
> > 'display-time-mode'.
> >
> > +** New user option 'describe-bindings-show-prefixes'.
> > +If true (the default is false), restores the "Prefix Command" entries
>
> We use "If non-nil", not "If true". And similarly for "false".
>
> Guess I've been thinking too much in Perl lately. ;-}
>
> > +in 'describe-binding' ('C-h b') output for keys that are prefixes, the
> > +details of which are shown later (regardless of this option setting).
>
> This NEWS entry should go to where the change in "C-h b"s behavior is
> described in NEWS (on emacs-29 branch), and should be reworded to tell
> users how to get the previous behavior back ("To get the previous
> behavior back, customize ...").
>
> OK. I also subordinated it to the "Keymap descriptions have changed"
> entry, since that seemed apt.
>
> > +(defcustom describe-bindings-show-prefixes nil
> > + "Non-nil shows prefixes in the output buffer of `describe-bindings'."
> > + :type 'boolean
> > + :group 'help
> > + :version "30.1")
>
> This should be "29.1". Since the behavior change was introduced in
> Emacs 29, we need to add this option in the same version.
>
> OK, I wasn't sure Lars' change actually made it into 29.
Thanks, I've now installed this (with some minor changes) on the
emacs-29 branch, and I'm therefore closing this bug.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Thu, 20 Apr 2023 11:24:16 GMT)
Full text and
rfc822 format available.
This bug report was last modified 2 years and 24 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.